Graylog2 / Graylog2/graylog2-server

GELF Input falsely complains about invalid timestamp field

Open
#9,878 1 comment 0 reactions 0 assignees View on GitHub
bug inputs triaged
Dominant language
Java
Stars
8.1k
Forks
1.1k
Avg merge
1d 20h
Merged PRs (30d)
217

Description

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]

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.