Skip to content

Comments

bypass gelf library level mapping#29

Open
ivan-mjch wants to merge 1 commit intoemsearcy:masterfrom
ivan-mjch:master
Open

bypass gelf library level mapping#29
ivan-mjch wants to merge 1 commit intoemsearcy:masterfrom
ivan-mjch:master

Conversation

@ivan-mjch
Copy link

This gelf output plugin maps syslog levels to gelf library levels (which I think are ruby logger levels)
and then the gelf library maps them back to syslog levels. This patch avoids the conversion,
storing them as plain integers defines an identity mapping for the gelf library to use.

This patch should avoid the loss of message level information when it is 0 1 or 5.
It works on our systems.

@emsearcy
Copy link
Owner

emsearcy commented Jan 5, 2017

I'd prefer if this could just get fixed upstream as per graylog-labs/gelf-rb#52 so we use their library in clean fashion, rather than have to work around it...

emsearcy/gelf-rb@65dd04e

@ivan-mjch
Copy link
Author

I didn't see that.

Personally, I would ask for support for using numeric syslog levels directly,
as mapping it back and forth still seems a bit silly, but yes, that should work.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants