t0xa / gelfj

Graylog Extended Log Format (GELF) implementation in Java and log4j appender without any dependencies.
https://github.com/t0xa/gelfj/wiki
Other
187 stars 115 forks source link

UDP buffer size error #89

Closed grzegorz-zur closed 9 years ago

grzegorz-zur commented 9 years ago

When a message is split into multiple UDP datagrams GrayLog fails to decompress it. This is caused by the 'range' buffer being too large by one byte. This fixes the issue. Tested with release 1.1.12.

t0xa commented 9 years ago

Hi,

Thanks a lot for you contribution. Could you please add tests.

Thanks! Anton

When a message is split into multiple UDP datagrams GrayLog fails to decompress it. This is caused by the 'range' buffer being too large by one

byte. This fixes the issue. Tested with release 1.1.12.

You can view, comment on, or merge this pull request online at:

https://github.com/t0xa/gelfj/pull/89 Commit Summary

File Changes

Patch Links:

— Reply to this email directly or view it on GitHub https://github.com/t0xa/gelfj/pull/89.

grzegorz-zur commented 9 years ago

I improved GelfMessageTest.testSendLongMessage so it fails if the Gzip stream is corrupted. See that it will fail without the fix. The same exception would be raised on GrayLog server.

t0xa commented 9 years ago

Thanks Grzegorz! Will release new version closer to the weekend.