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(sources): fix unexpected timestamp field #21558

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

fix(sources): fix unexpected timestamp field #21558

wants to merge 1 commit into from
+9 −3

Commits on Oct 19, 2024

  1. fix(sources): fix unexpected timestamp field 

    When log namespacing is enabled, `exec` and `http_server` sources include
a redundant `timestamp` field when decoding is enabled. When decoding
is not enabled, they put the content under a _"message"_ key, instead
of directly in the root object.

In the case of `exec`, the issue was caused by passing a hardcoded
`LogNamespace` value to the decoder. Fixed by passing the parsed value.

In the case of `http_server`, the decoder ignored the global
`LogNamespace`. Fixed by passing the correctly parsed value to the
decoder.

Fixes vectordotdev#20404.
    @rwakulszowa
    rwakulszowa committed Oct 19, 2024
    Configuration menu
    Copy the full SHA
    355c9b4 View commit details
    Browse the repository at this point in the history