uken / fluent-plugin-elasticsearch

Apache License 2.0
890 stars 310 forks source link

do not overwrite @timestamp in data stream if it already exists in the record #968

Closed dschaaff closed 2 years ago

dschaaff commented 2 years ago

closes #967

If the log record already contains the field @timestamp, then the data stream output plugin should not overwrite the field.

(check all that apply)

dschaaff commented 2 years ago

@cosmo0920 I'll look into adding a test.

dschaaff commented 2 years ago

@cosmo0920 I've added a test if you don't mind reviewing when you have time. Thanks!

rjbma commented 2 years ago

Just ran into the problem outlined in #967 and noticed this PR fixes it. Any news on when this is getting released?

Thanks!