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
Issue 5719 - remove Pretty JSON and make it user configurable #5766
Conversation
Signed-off-by: George Tebrean <george@web3labs.com>
|
Signed-off-by: George Tebrean <george@web3labs.com>
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.
does this CLI option cover both HTTP and websockets?
Signed-off-by: George Tebrean <george@web3labs.com>
Yes because is setting the format in |
Signed-off-by: George Tebrean <george@web3labs.com>
Signed-off-by: George Tebrean <george@web3labs.com>
Signed-off-by: Sally MacFarlane <macfarla.github@gmail.com>
Docs change required - add a note to docs that all HTTP and Websockets responses will be compact JSON by default, and the |
…edger#5766) * add --pretty-json-enabled * update changelog Signed-off-by: George Tebrean <george@web3labs.com> * Update CLI option name in CHANGELOG.md Signed-off-by: Sally MacFarlane <macfarla.github@gmail.com> --------- Signed-off-by: George Tebrean <george@web3labs.com> Signed-off-by: Sally MacFarlane <macfarla.github@gmail.com> Co-authored-by: Sally MacFarlane <macfarla.github@gmail.com>
…edger#5766) * add --pretty-json-enabled * update changelog Signed-off-by: George Tebrean <george@web3labs.com> * Update CLI option name in CHANGELOG.md Signed-off-by: Sally MacFarlane <macfarla.github@gmail.com> --------- Signed-off-by: George Tebrean <george@web3labs.com> Signed-off-by: Sally MacFarlane <macfarla.github@gmail.com> Co-authored-by: Sally MacFarlane <macfarla.github@gmail.com>
…edger#5766) * add --pretty-json-enabled * update changelog Signed-off-by: George Tebrean <george@web3labs.com> * Update CLI option name in CHANGELOG.md Signed-off-by: Sally MacFarlane <macfarla.github@gmail.com> --------- Signed-off-by: George Tebrean <george@web3labs.com> Signed-off-by: Sally MacFarlane <macfarla.github@gmail.com> Co-authored-by: Sally MacFarlane <macfarla.github@gmail.com> Signed-off-by: garyschulte <garyschulte@gmail.com>
…edger#5766) * add --pretty-json-enabled * update changelog Signed-off-by: George Tebrean <george@web3labs.com> * Update CLI option name in CHANGELOG.md Signed-off-by: Sally MacFarlane <macfarla.github@gmail.com> --------- Signed-off-by: George Tebrean <george@web3labs.com> Signed-off-by: Sally MacFarlane <macfarla.github@gmail.com> Co-authored-by: Sally MacFarlane <macfarla.github@gmail.com>
PR description
Added --pretty-json-enabled as CLI option as it was asked in the issue mentioned bellow.
Here I attached an example with the output of a RPC request when Besu was ran with --pretty-json-enabled (first case) and without the argument in second case:
Fixed Issue(s)
Fixes #5712