fix: wait for async command exit (#1326)
This prevents the accumulation of zombie processes when using async (&) event commands. Also log async command failures.
This commit is contained in:
parent
381f09087a
commit
6d5ceae8b4
|
@ -102,6 +102,14 @@ func (r *Runner) exec(raw, evt, path, dst string, user *users.User) error {
|
||||||
|
|
||||||
if !blocking {
|
if !blocking {
|
||||||
log.Printf("[INFO] Nonblocking Command: \"%s\"", strings.Join(command, " "))
|
log.Printf("[INFO] Nonblocking Command: \"%s\"", strings.Join(command, " "))
|
||||||
|
defer func() {
|
||||||
|
go func() {
|
||||||
|
err := cmd.Wait()
|
||||||
|
if err != nil {
|
||||||
|
log.Printf("[INFO] Nonblocking Command \"%s\" failed: %s", strings.Join(command, " "), err)
|
||||||
|
}
|
||||||
|
}()
|
||||||
|
}()
|
||||||
return cmd.Start()
|
return cmd.Start()
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue