jordansissel / ruby-grok

Pure-ruby implementation of grok.
Apache License 2.0
54 stars 25 forks source link

Added exception catching to regexp match #3

Closed cabeca closed 3 years ago

cabeca commented 12 years ago

Here is my change as reported in:

https://logstash.jira.com/browse/LOGSTASH-436

Best Regards Miguel Cabeça

cabeca commented 12 years ago

Here is a second take on the problem.

rarruda commented 11 years ago

I also have this issue and it would be really nice to have this handled, as I see thousands of these exceptions an hour. Looks like the second patch proposed here to deal with it effectively.

jordansissel commented 11 years ago

I wonder if this problem goes away in logstash 1.1.6 which has much improved text encoding support. Thoughts?

jordansissel commented 11 years ago

I wonder if this problem goes away in logstash 1.1.6 which has much improved text encoding support. Thoughts?

rarruda commented 11 years ago

I will try to build master and see what results I get.

jordansissel commented 11 years ago

You can fetch daily builds here: http://r.logstash.net/jenkins/job/logstash.jar.daily/lastSuccessfulBuild/artifact/build/logstash-1.1.6.dev-monolithic.jar

rarruda commented 11 years ago

As quick feedback. This is then fixed in logstash 1.1.6-dev by using the "encoding" parameter as you suggested. In my case setting it to any valid encoding will make it work.