search

logstash-plugins / logstash-output-riemann

Apache License 2.0
5 stars 13 forks source link

Rework Riemann event construction #24

Closed ninaspitfire closed 4 years ago

ninaspitfire commented 4 years ago

The previous implementation had problems when the "map_fields" and "riemann_event" configuration options compete to set the value of a Riemann event field. "riemann_event" should win, but was not winning.

Additionally, casting operations during event contruction could result in strings being sent to Riemann where numbers are required. The "ttl" field, in particular, had this problem.

Fixes: #22 Fixes: #23

ninaspitfire commented 4 years ago

With pleasure!

Thanks so much for the quick review, João.

jsvd commented 4 years ago

Version 3.0.6 has been published! https://rubygems.org/gems/logstash-output-riemann/versions/3.0.6