Graylog2 / go-gelf

GELF library for Go
http://godoc.org/github.com/Graylog2/go-gelf/gelf
MIT License
103 stars 64 forks source link

Fix gelf uncompressed scheme #3

Closed dqminh closed 8 years ago

dqminh commented 8 years ago

The previous code (#2) was based on https://github.com/lusis/gelfd/commit/79b893ada7867eee1eb78b94cc97cf0199aac32b but apparently the scheme was changed for GelfMessage.java i.e. no more magic header.

This removes the magic header and also optimize the ReadMessage code a bit i.e. no more temporary buffer to store the uncompressed json.

dqminh commented 8 years ago

ping @mariussturm

tested this manually with a container running graylog2 locally. Let me know if this looks right.

mariussturm commented 8 years ago

Yeah, looks good. Will also do some testing later in the week. Thanks again!