Skip to content
This repository has been archived by the owner on Jun 29, 2023. It is now read-only.

Add version field #260

Merged
merged 2 commits into from
Mar 1, 2021
Merged

Add version field #260

merged 2 commits into from
Mar 1, 2021

Conversation

normakm
Copy link
Contributor

@normakm normakm commented Feb 4, 2021

Add "version" field to GELF 1.1 messages and above as specified here:
https://docs.graylog.org/en/latest/pages/gelf.html

Add version field for GELF versions 1.1 and up as specified here:
https://docs.graylog.org/en/latest/pages/gelf.html
> version string (UTF-8)
> GELF spec version – “1.1”; MUST be set by client library.
added unit test for version field
@mp911de mp911de added the type: bug A general bug label Mar 1, 2021
@mp911de mp911de added this to the 1.15.0 milestone Mar 1, 2021
@mp911de mp911de merged commit 2e231c7 into mp911de:main Mar 1, 2021
@mp911de
Copy link
Owner

mp911de commented Mar 1, 2021

Thank you for your contribution. That's merged now.

@mp911de mp911de linked an issue Mar 1, 2021 that may be closed by this pull request
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
type: bug A general bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

GELF message seems to be missing mandatory field "version"
2 participants