fix(cli): report-formats panics with uppercase input closes Checkmarx…

…#4455 (Checkmarx#4457)
noamd-legit · Nov 3, 2021 · 8daf925 · 8daf925
1 parent 87b1ff7
commit 8daf925
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/internal/console/helpers/helpers.go b/internal/console/helpers/helpers.go
@@ -218,6 +218,7 @@ func GenerateReport(path, filename string, body interface{}, formats []string, p
 	defer progressBar.Close()
 
 	for _, format := range formats {
+		format = strings.ToLower(format)
 		if err = reportGenerators[format](path, filename, body); err != nil {
 			log.Error().Msgf("Failed to generate %s report", format)
 			break

diff --git a/internal/console/scan.go b/internal/console/scan.go
@@ -5,6 +5,7 @@ import (
 	"os"
 	"os/signal"
 	"path/filepath"
+	"strings"
 	"syscall"
 
 	"github.com/Checkmarx/kics/internal/console/flags"
@@ -99,7 +100,7 @@ func run(cmd *cobra.Command) error {
 
 func updateReportFormats() {
 	for _, format := range flags.GetMultiStrFlag(flags.ReportFormatsFlag) {
-		if format == "all" {
+		if strings.EqualFold(format, "all") {
 			flags.SetMultiStrFlag(flags.ReportFormatsFlag, consoleHelpers.ListReportFormats())
 			break
 		}