use LogStash::Json for json serialization

logstash-plugins / logstash-output-influxdb

Apache License 2.0

58 stars 77 forks source link

Closed colinsurprenant closed 9 years ago

colinsurprenant commented 9 years ago

fixed json serialization as per #20 and introduced in #8 added initial spec did some cleanups

ph commented 9 years ago

LGTM, tests are running, I do not have influxdb to do an integration test

colinsurprenant commented 9 years ago

corrected the allow_time_override misusage, it is now actually doing what it was supposed to do. it happens that my spec also validates it.

jsvd commented 9 years ago

LGTM

wiibaa commented 9 years ago

@colinsurprenant could you please close https://github.com/elastic/logstash-contrib/pull/116 related to allow_time_override error you just fixed. Thanks