-
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? 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
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: