search

magnusbaeck / logstash-filter-verifier

Apache License 2.0
192 stars 27 forks source link

Better error reporting on JSON syntax errors #16

Closed breml closed 7 years ago

breml commented 7 years ago

Does include line number and context in error message.

breml commented 7 years ago

I was unsure if it is worth to add an other dependency for a single function. On the other side I did not want to just copy the function (from a licensing point of view) so I think, this is the proper solution.

magnusbaeck commented 7 years ago

Yeah, I don't mind the dependency. Great improvement, thanks!