logstash-plugins / logstash-filter-translate

Translate filter for Logstash
Apache License 2.0
21 stars 47 forks source link

Add better support for JSON. Also json (yaml, csv) in default value. #22

Closed guyboertje closed 6 years ago

guyboertje commented 8 years ago

Use Logstash::Json.

Re-organise the dictionary loading by file extension to save a reference to the loader method, so it can be reused to load the default value or fall back to plain string loading.

Use deep_clone to fix bug because a pointer to the dictionary was stored in the event - any mutation of the value would also mutate the value in the dictionary.

guyboertje commented 8 years ago

Jenkins please test this

guyboertje commented 8 years ago

The test fail on travis and jenkins for different reasons and both suck.

Travis is using logstash-core 2.3.0.snapshot1-java Jenkins is using logstash-core 2.1.1

guyboertje commented 8 years ago

Jenkins please test this

guyboertje commented 8 years ago

please jenkins, test this

guyboertje commented 8 years ago

please jenkins test this