Linting/Formatting of middleware folder #107
Open
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This pull request refactors the
RequestTrackingMiddlewareinapp/middleware/request_tracking.pyto improve type safety, update error logging, and clarify documentation. The main changes focus on using more precise type annotations, enhancing error logging, and cleaning up the request handling logic.Type annotation improvements:
call_nextparameter in thedispatchmethod to useCallable[[Request], Awaitable[Response]]for better type safety. [1] [2]responsevariable, clarifying its type asResponse.Error handling and logging enhancements:
logger.errortologger.exceptionto automatically include stack traces for better debugging. Also removed the redundantexc_info=Trueargument.Documentation improvements:
dispatchmethod to use a more standard format with section headers for arguments and return values.