Log level for errors is "info"

[rosemary-wang]

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]

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.

[rajeshkarnena]

Hello @johanbrandhorst @rosemary-wang, can I help with a PR for this?

I also noticed that a few log statements in the package runtime did not follow the correct log level, e.g., this one and this one. Is that intentional?

[rosemary-wang]

i haven't started working on this so feel free to take it

[johanbrandhorst]

Feel free to open a PR

[rajeshkarnena]

Here you go, please review #4327