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

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