search

logstash-plugins / logstash-filter-mutate

Apache License 2.0
16 stars 75 forks source link

Don't create null fields if trying to lowercase/uppercase a field tha… #83

Closed andrewvc closed 8 years ago

andrewvc commented 8 years ago

Rebase based on https://github.com/logstash-plugins/logstash-filter-mutate/pull/66 that adds tests / works with more modern stuff.

jsvd commented 8 years ago

LGTM after addressing Magnus' remark

jsvd commented 8 years ago

And I'd suggest adding a test for the boolean case.

elasticsearch-bot commented 8 years ago

Andrew Cholakian merged this into the following branches!

Branch Commits
master 7f0a1be1e25340d28cdf6f00c94d6671fa1872c2, e9139ac11d5c395663ee4ea715537079f1d1ff7c