Skip to content

Add a catch-all error handler using ProblemDetails - #390

Draft
rwb27 wants to merge 1 commit into
mainfrom
catch-all-error-handling
Draft

Add a catch-all error handler using ProblemDetails#390
rwb27 wants to merge 1 commit into
mainfrom
catch-all-error-handling

Conversation

@rwb27

@rwb27 rwb27 commented Aug 3, 2026

Copy link
Copy Markdown
Collaborator

This may not make it out of draft - it was intended to reduce the ASGI noise in the logs, but it doesn't do that: a stack trace still gets logged to the uvicorn log, and nothing elsewhere.

…tails format.

Unfortunately this does not do anything helpful with logging.
@barecheck

barecheck Bot commented Aug 3, 2026

Copy link
Copy Markdown

Barecheck - Code coverage report

Total: 97.25%

Your code coverage diff: -0.06% ▾

Uncovered files and lines
FileLines
src/labthings_fastapi/server/__init__.py261-262, 277, 281, 337, 351-354

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant