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.

Sign up for GitHub

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

Jump to bottom

fix: change api access log level to debug #4876

Open

martinconic wants to merge 1 commit into master from api-access-info-logs

+1 −1

Contributor

martinconic commented Oct 23, 2024 •

edited

Loading

Checklist

I have read the coding guide.
My change requires a documentation update, and I have done it.
I have added tests to cover my changes.
I have filled out the description and linked the related issues.

Description

Closes #4845

Open API Spec Version Changes (if applicable)

Motivation and Context (Optional)

Related Issue (Optional)

Screenshots (if appropriate):


          fix: change api access log level to debug

cc41250

Member

istae commented Oct 23, 2024

i dont think we should be logging the API acess at the debug level

Contributor Author

martinconic commented Oct 23, 2024 •

edited

Loading

i dont think we should be logging the API acess at the debug level

Have you checked #4845 ? Should we not merge this?

martinconic requested review from janos, istae and acha-bill

October 24, 2024 07:49

janos approved these changes

View reviewed changes

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

Labels

None yet