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.

Sign up for GitHub

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

Jump to bottom

subscriber: fix missing event fields in Compact formatter #1755

Merged
merged 3 commits into from Nov 29, 2021
Merged

subscriber: fix missing event fields in Compact formatter #1755

merged 3 commits into from Nov 29, 2021

Commits on Nov 29, 2021

  1. subscriber: fix missing event fields in Compact formatter 

    The line of code that prints the event's fields (including its message)
was accidentally deleted in 0a16972,
while backporting PR #1696 from `master` (where the `Compact`) formatter
implementation is significantly different.

This branch puts it back.

Fixes #1741
    @hawkw
    hawkw committed Nov 29, 2021
    Copy the full SHA
    d7a2e42 View commit details
    Browse the repository at this point in the history

  2. missing whitespace after target 

    Signed-off-by: Eliza Weisman <eliza@buoyant.io>
    @hawkw
    hawkw committed Nov 29, 2021
    Copy the full SHA
    5dc6de0 View commit details
    Browse the repository at this point in the history

  3. subscriber: add tests for default formatter 

    Signed-off-by: Eliza Weisman <eliza@buoyant.io>
    @hawkw
    hawkw committed Nov 29, 2021
    Copy the full SHA
    0c85d9c View commit details
    Browse the repository at this point in the history