Correctly handle [nested][field][references] for target and columns settings.

logstash-plugins / logstash-filter-csv

Apache License 2.0

15 stars 41 forks source link

Correctly handle [nested][field][references] for target and columns settings. #52

Closed jordansissel closed 7 years ago

jordansissel commented 7 years ago

This fixes two similar bugs where a field reference like [foo][bar] would be wrapped incorrectly as [[foo][bar]] causing incorrect fields to be set in the event.

The affected settings were target and columns.

Added tests to cover the bug fixes.

Fixes #24

jordansissel commented 7 years ago

@andrewvc I'm pretty sure there are tests for what you described, can you take a look? I'm not sure what is missing and could use your guidance.

jordansissel commented 7 years ago

@andrewvc Can you take another look at this?

elasticsearch-bot commented 7 years ago

Jordan Sissel merged this into the following branches!

Branch	Commits
master	7899801e628f54d29d15c23e4be5040d1ac1fa37, 1faf28ad493c92a2fe280fc21fac58a29dceb707