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

fix(new_relic sink): Put log API attributes in separate structure #21313

Merged
merged 2 commits into from
Sep 18, 2024
Merged

fix(new_relic sink): Put log API attributes in separate structure #21313

merged 2 commits into from
Sep 18, 2024

Commits on Sep 18, 2024

  1. fix(new_relic sink): Put log API attributes in separate structure 

    The New Relic log API specifies that all attributes other than `message` and
`timestamp` should go into an `attributes` field, which may contain any JSON
object data except for arrays. This change reworks the logs API model to move
all attributes from the log event into this attributes field as well as using
structures to encode fields with fixed names instead of using dynamic maps.

The tests have also been updated to compare the overall structure of the output
data model so as to ensure that the resulting encoding generated by `serde`
matches what we is specified in the New Relic API documentation.
    @bruceg
    bruceg committed Sep 18, 2024
    Configuration menu
    Copy the full SHA
    b53db4a View commit details
    Browse the repository at this point in the history

  2. Add change log entry

    @bruceg
    bruceg committed Sep 18, 2024
    Configuration menu
    Copy the full SHA
    63133a0 View commit details
    Browse the repository at this point in the history