Ensure we log into logrus on command error

`urfave/cli` now takes upon itself to log the error returned by the command action directly. This means that by default the `--log` option was ignored upon error. This commit ensure that `urfave/cli.ErrWriter` will use logrus Signed-off-by: Kenfe-Mickael Laventure <[email protected]>
opencontainers · Sep 29, 2016 · 48a10f1 · 48a10f1
1 parent 3597b7b
commit 48a10f1
Showing 1 changed file with 12 additions and 0 deletions.
diff --git a/main.go b/main.go
@@ -129,7 +129,19 @@ func main() {
 		}
 		return nil
 	}
+	// If the command returns an error, cli takes upon itself to print
+	// the error on cli.ErrWriter and exit.
+	// Use our own writer here to ensure the log gets sent to the right location.
+	cli.ErrWriter = &FatalWriter{}
 	if err := app.Run(os.Args); err != nil {
 		fatal(err)
 	}
 }
+
+type FatalWriter struct {
+}
+
+func (FatalWriter) Write(p []byte) (n int, err error) {
+	logrus.Error(string(p))
+	return len(p), nil
+}