search

Graylog2 / graylog2-server

Free and open log management
https://www.graylog.org
Other
7.35k stars 1.06k forks source link

GELF Input falsely complains about invalid timestamp field #9878

Open mpfz0r opened 3 years ago

mpfz0r commented 3 years ago

Receiving a GELF message with a timestamp as string will trigger a validation warrning:

https://github.com/Graylog2/graylog2-server/blob/master/graylog2-server/src/main/java/org/graylog2/inputs/codecs/GelfCodec.java#L273

However, these messages can be processed just fine: https://github.com/Graylog2/graylog2-server/blob/master/graylog2-server/src/main/java/org/graylog2/inputs/codecs/GelfCodec.java#L111

I suggest we unify the validation and parsing methods.

While there, we might also take a look at https://github.com/Graylog2/graylog2-server/issues/8416

[#z 4471]

hilocz commented 7 months ago

Still in 5.2.4