CLI utility to pipe JSON logs through to pretty print and colorize them #449
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
rs
requested changes
Jul 16, 2022
rs
requested changes
Jul 27, 2022
cmd/prettylog/README.md
Outdated
| Humans are visual beings so the preferred format should be ordinary `Console` format with colors if possible. | ||
|
|
||
| Zerolog ships with `ConsoleWriter` which does exactly that. But modifying your code to get pretty printed and colorized | ||
| logs is somewhat difficult. |
There was a problem hiding this comment.
A simpler solution to the problem described here is to detect if the stdout is a terminal and automatically enable console mode if it is. This is a trick I may add as conditional writer in the lib.
I still think this tool is useful when you need to tail zerolog produced output in prod for instance, or in settings when you need both JSON and pretty output.
Could you change the text of the readme to remove the part about why you need this tool so it does not come as the recommended solution to solve this problem?
There was a problem hiding this comment.
Yes - of course. I updated the README.md - now it's just a description and usage.
Sorry for taking so long - was on vacation during the summer :)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Implemented a small utility tool to use when developing an application using zerolog as mentioned in #316.
The only thing that is untested is
Windowsinstructions specifically commands that redirectstderrtostdout:Can someone verify this is working? I only have access to a Linux machine currently.