Text logger does not support fully logfmt
mantzas opened this issue · 0 comments
mantzas commented
Is your feature request related to a problem? Please describe
There are benefits to have our text logs following a key/value representation which allows for better parsing.
Text logging needs also to escape invalid characters in order to not break the log line.
We should also keep the performance impact in mind.
Some posts about logfmt: https://brandur.org/logfmt
Describe the solution
- Introduce fully logfmt in our logs
- Escape invalid text
- Benchmark JSON vs logfmt in Patron