-
Notifications
You must be signed in to change notification settings - Fork 79
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Added support for extra attributes for logging.LogRecord #22
Added support for extra attributes for logging.LogRecord #22
Conversation
Would it be better to use an |
@tclancy I have implemented your future-proofing idea! |
Thanks, would it be possible to also add a test/ tests to prove this new behavior works as intended? |
d24d826
to
b4b02af
Compare
Note that the failing test is mentioned in #24 that is an existing regression in the test - it would be nice for this to be merged still. |
Does this need to be updated now or is it obsolete? |
@tclancy Remerged master with test fix, so now everything is ready to go. |
This allows access to request and/or response objects within each LogRecord by default, as well as allowing for easy inheritance configuration with the extra dict construction pulled out into its own method.
Note allows log filters to be able to print out information specific to individual requests, such as user.