thekrakken / java-grok

Simple API that allows you to easily parse logs and other files
http://grok.nflabs.com/
Other
358 stars 152 forks source link

Issue #36: Data type conversion and compatiblity with Logstash's grok #44

Closed jweygandt closed 8 years ago

jweygandt commented 8 years ago

Modified converter per issue to allow for ":" as a type delimiter. Altered ApacheDataTypeTest to correctly detect errors, and added a test for ":" delimeter, as the original file used ";".

An unintended side effect is that capture names can no longer have the colon (:) character in the name. Updated CaptureTest.

anthonycorbacho commented 8 years ago

Thanks you for the improvement looks great