search

logstash-plugins / logstash-filter-mutate

Apache License 2.0
16 stars 75 forks source link

Don't create null fields if lowercase/uppercase a missing field #66

Closed jimmyjones2 closed 8 years ago

jimmyjones2 commented 8 years ago

Don't create null fields if trying to lowercase/uppercase a field that is missing from the event

Fixes #65

elasticsearch-release commented 8 years ago

Jenkins standing by to test this. If you aren't a maintainer, you can ignore this comment. Someone with commit access, please review this and clear it for Jenkins to run; then say 'jenkins, test it'.

jimmyjones2 commented 8 years ago

@purbon Can you have a look please?

jimmyjones2 commented 8 years ago

@suyograo Can you have a look please?

andrewvc commented 8 years ago

Thanks for the contribution. Apologies for the extreme delay. I've rebased this PR and added specs here: https://github.com/logstash-plugins/logstash-filter-mutate/pull/83

I'm going to close this PR since it isn't mergeable due to rebase issues and we'll pick up / merge the one I opened (with your commit in it).

jimmyjones2 commented 8 years ago

@andrewvc Thanks!