Why is the launch message logged as a warning? #2529
-
I installed the latest version (0.5.0-rc.3) et I saw that the launch message was logged as a warning: I believe it should be an info instead as all logs above are infos as well? Or is there a condition to display a warning? |
Beta Was this translation helpful? Give feedback.
Replies: 2 comments 1 reply
-
I think it was changed to make sure something is displayed with the default settings, which are not displaying INFO level stuff on a release build. There were multiple false bugreports complaining because the servert didn't output anything when starting up. On the other hand people were also complaining about this causing unnecessary alerts because it's treated as a WARN when nothing bad happened really. BTW Originally I think every launch message was displayed as WARN, and there were a PR to change them all the INFO here: #1828 Then it got partially reverted in this commit, I can't find the related issues / discussions, but see the commit message for more info: So I think the current behaviour is intended to be a "middle ground" between everything being WARN and everything being INFO. But it looks like you can't please everyone either way 🤷♂️ |
Beta Was this translation helpful? Give feedback.
-
Maybe a middle ground would be to display everything in DEBUG except the launch log which would be in INFO. And only INFO logs would be displayed in release. Or maybe it's not that simple to do? |
Beta Was this translation helpful? Give feedback.
There are many INFO level log statements that we don't want to emit during release, however, so this would be even more onerous for those who don't want this single log message.
I don't think there's any good solution as it stands. Presumably once we switch over to tracing, it'll be easier to have arbitrary log levels, and launch information can live in its own level.