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

Log level for errors is "info" #4245

Open
rosemary-wang opened this issue Apr 23, 2024 · 1 comment
Open

Log level for errors is "info" #4245

rosemary-wang opened this issue Apr 23, 2024 · 1 comment

Comments

@rosemary-wang
Copy link

Hi team, I have a question regarding the log level chosen for the generated code. I noticed that info level is used for everything, even for obvious errors such as "failed to decode message". I'm curious about the motivation behind this choice. Thank you!

@johanbrandhorst
Copy link
Collaborator

Hi Rosemary, thanks for your issue! I don't think the log level of these messages have changed since they were first introduced, and I doubt the level was considered much. If you have any thoughts on ways to differentiate different log messages, it might make for a great contribution! The logic for the generated file is mostly here: https://github.com/grpc-ecosystem/grpc-gateway/blob/main/protoc-gen-grpc-gateway/internal/gengateway/template.go.

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

No branches or pull requests

2 participants