search

logstash-plugins / logstash-filter-grok

Grok plugin to parse unstructured (log) data into something structured.
https://www.elastic.co/guide/en/logstash/current/plugins-filters-grok.html
Apache License 2.0
124 stars 98 forks source link

fix fieldref assignment to not assume mutable object #62

Closed colinsurprenant closed 8 years ago

colinsurprenant commented 8 years ago

this is related to problem described in elastic/logstash#4140 and also relates to elastic/logstash#4191 we cannot assume anymore that field reference getter returns a mutable object.

tracked in elastic/logstash#4264

sorry for the whitespaces diffs :(

colinsurprenant commented 8 years ago

@jsvd followup review request please!

jsvd commented 8 years ago

minor comment otherwise LGTM

colinsurprenant commented 8 years ago

thanks, will fix comment and merge!

colinsurprenant commented 8 years ago

pushed version 2.0.3