-
Notifications
You must be signed in to change notification settings - Fork 41.2k
Improve documented logging property descriptions and default values #41933
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Comments
This is covered in a tip in Custom Log Configuration:
If you pass the value straight into Logback, in |
The default value isn't really correct because it differs depending on the logging system. We're going to remove the default value from the metadata JSON file. |
Hello, now that I'm able to set a custom default for
logging.pattern.console
i noticed that the documentation of the default value seems to be slightly incorrect.By Default Log messages look like this:
The documentation says about
logging.pattern.console
:Appender pattern for output to the console. Supported only with the default Logback setup.
Default:
%clr(%d{${LOG_DATEFORMAT_PATTERN:-yyyy-MM-dd'T'HH:mm:ss. SSSXXX}}){faint} %clr(${LOG_LEVEL_PATTERN:-%5p}) %clr(${PID:- }){magenta} %clr(---){faint} %clr([%15.15t]){faint} %clr(%-40.40logger{39}){cyan} %clr(:){faint} %m%n${LOG_EXCEPTION_CONVERSION_WORD:-%wEx}
But ff I copy paste this value set this as the default the log looks like this:
I after some tests I think this is the correct default:
%clr(%d{${LOG_DATEFORMAT_PATTERN:yyyy-MM-dd'T'HH:mm:ss.SSSXXX}}){faint} %clr(${LOG_LEVEL_PATTERN:%5p}) %clr(${PID:- }){magenta} %clr(---){faint} %clr([%15.15t]){faint} %clr(%-40.40logger{39}){cyan} %clr(:){faint} %m%n${LOG_EXCEPTION_CONVERSION_WORD:%wEx}
this will produce the same log as if there is not custom config.
The text was updated successfully, but these errors were encountered: