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
core: Improve log messages consistency #1510
Conversation
@@ -31,7 +31,7 @@ | |||
{lager, | |||
[{handlers, | |||
[ | |||
{lager_console_backend, info}, | |||
{lager_console_backend, [info, {lager_default_formatter, [time, " [", severity, "] ", {site, [site, " "], ""}, {module, [module, ":", line, " "], ""}, message, "\n"]}]}, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Using Logstash conditional formatting.
927297c
to
20f1c43
Compare
@mworrell Please review. |
{lager_console_backend, info}, | ||
{lager_file_backend, [{file, "priv/log/error.log"}, {level, error}]}, | ||
{lager_console_backend, [info, {lager_default_formatter, [time, " [", severity, "] ", {site, [site, " "], ""}, {module, [module, ":", line, " "], ""}, message, "\n"]}]}, | ||
{lager_file_backend, [{file, "priv/log/error.log"}, {level, error}, {formatter, lager_default_formatter}]}, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Todo: check how this influences error.log
output. We want lager metadata in error|console.log
too.
20f1c43
to
b26d9c3
Compare
* Call lager functions directly so parse transform works. * Remove usage of lager:log, which bypasses parse transform. * Remove all custom message formatting: have one standard in the lager configuration.
b26d9c3
to
297eb9d
Compare
* Call lager functions directly so parse transform works. * Remove usage of lager:log, which bypasses parse transform. * Remove all custom message formatting: have one standard in the lager configuration. (cherry picked from commit cb8ccb1)
* Call lager functions directly so parse transform works. * Remove usage of lager:log, which bypasses parse transform. * Remove all custom message formatting: have one standard in the lager configuration. (cherry picked from commit cb8ccb1)
* Fix lager:error calls throwing a FORMAT ERROR due to invalid number of parameters. * Remove site name from log messages (see also #1510).
* Fix lager:error calls throwing a FORMAT ERROR due to invalid number of parameters. * Remove site name from log messages (see also #1510).
Description
Fix #1507.
error.log
andconsole.log
:Checklist