From 1a113d6da94445f02d45d0232a11d1ac10a41703 Mon Sep 17 00:00:00 2001 From: Cosmin Cojocar Date: Tue, 14 Apr 2020 10:51:21 +0200 Subject: [PATCH] Turn the color always on when the text format is set Signed-off-by: Cosmin Cojocar --- cmd/gosec/main.go | 10 ++++------ 1 file changed, 4 insertions(+), 6 deletions(-) diff --git a/cmd/gosec/main.go b/cmd/gosec/main.go index 0597550..29608c3 100644 --- a/cmd/gosec/main.go +++ b/cmd/gosec/main.go @@ -120,9 +120,6 @@ var ( // exlude the folders from scan flagDirsExclude arrayFlags - // set color on text format output - flagColor = flag.Bool("color", false, "Enable colored output. Valid for text format") - logger *log.Logger ) @@ -286,8 +283,9 @@ func main() { } // Color flag is allowed for text format - if *flagColor && *flagFormat != "text" { - logger.Fatalf("cannot set color with %s format. Only text format is accepted", *flagFormat) + var color bool + if *flagFormat != "text" { + color = true } failSeverity, err := convertToScore(*flagSeverity) @@ -358,7 +356,7 @@ func main() { } // Create output report - if err := saveOutput(*flagOutput, *flagFormat, *flagColor, flag.Args(), issues, metrics, errors); err != nil { + if err := saveOutput(*flagOutput, *flagFormat, color, flag.Args(), issues, metrics, errors); err != nil { logger.Fatal(err) }