Skip to content
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

Separate request and response logging into other named loggers. #180

Merged
merged 1 commit into from
Jul 13, 2024

Conversation

aclemons
Copy link
Contributor

@aclemons aclemons commented Feb 16, 2024

I'd like to make the logging for aiodynamo louder at times for debugging
or to generally see in application logs when a throttle or a timeout or
something happened. For that, I need to set the log level to debug, but
this also enables request and response payload logging which becomes
very noisy and also smears potentially unwanted data into my log
streams.

I made two other loggers so the levels for those two things can be
set at a different level to all the other logging done by aiodynamo.

@aclemons
Copy link
Contributor Author

Thoughts on this?

I'd like to make the logging for aiodynamo louder at times for debugging
or to generally see in application logs when a throttle or a timeout or
something happened. For that, I need to set the log level to debug, but
this also enables request and response payload logging which becomes
very noisy and also smears potentially unwanted data into my log
streams.

I made two other loggers so the levels for those two things can be
set at a different level to all the other logging done by aiodynamo.
@aclemons aclemons force-pushed the payload-names-loggers branch from 138bcad to 5628360 Compare February 16, 2024 06:27
@dimaqq
Copy link
Contributor

dimaqq commented May 27, 2024

👉🏻 @ojii

@y26805 y26805 requested a review from ojii June 17, 2024 01:39
@ojii ojii merged commit d98a867 into HENNGE:master Jul 13, 2024
33 checks passed
@ojii
Copy link
Contributor

ojii commented Jul 13, 2024

thank you

@aclemons aclemons deleted the payload-names-loggers branch July 16, 2024 00:26
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.

3 participants