Skip to content
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

[Feature Request] Pretty print Payload data #806

Open
yuandrew opened this issue Aug 29, 2024 · 0 comments
Open

[Feature Request] Pretty print Payload data #806

yuandrew opened this issue Aug 29, 2024 · 0 comments
Assignees
Labels
enhancement New feature or request

Comments

@yuandrew
Copy link
Contributor

Is your feature request related to a problem? Please describe.

As a part of Payload, there is data field with a vec of bytes, depending on the size of that field, the logs can get very noisy with all of this data.

Spencer already previously wrote a pretty printer,

impl Display for Payload {
. Because we generate the Payload struct automatically, prost generates a default Debug trait that doesn’t use this pretty print.

Describe the solution you'd like

Either only Display print in our logs, or create a custom Debug implementation that uses the pretty printing logic Spencer wrote.

Additional context

@yuandrew yuandrew added the enhancement New feature or request label Aug 29, 2024
@yuandrew yuandrew self-assigned this Aug 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant