threatgrid / jqpipe-go

A Go Wrapper for JQ
BSD 2-Clause "Simplified" License
12 stars 8 forks source link

Let 'jq' call run to completion #8

Open jmelchio opened 5 years ago

jmelchio commented 5 years ago

We've found that when running the Kill() on the process before the Wait() we quite often run into the unexplained jq failure. After some playing with it and enhancing the error message we noticed that the program had an io.EOF error and should have exited normally. We suspect that in some cases the program gets killed prematurely so that instead of Success(), it gets Killed() as its process status when things otherwise finish without problems. After removing the Kill() we don't see the unexplained errors anymore.