Adds: logger in case of flag parsing fails (#1115)

* adds: logger in case of flag parsing fails

* fix: error check

Co-authored-by: Suvarna Rokade <suvarna.rokade@accurics.com>

pkg/cli/register.go

@@ -21,6 +21,7 @@ import (
 	"io/ioutil"
 	"log"
 	"os"
+	"strings"
 
 	"github.com/accurics/terrascan/pkg/config"
 	"github.com/accurics/terrascan/pkg/logging"
@@ -40,6 +41,9 @@ func Execute() {
 	rootCmd.PersistentFlags().StringVarP(&OutputType, "output", "o", "human", "output type (human, json, yaml, xml, junit-xml, sarif, github-sarif)")
 	rootCmd.PersistentFlags().StringVarP(&ConfigFile, "config-path", "c", "", "config file path")
 
+	//Added init here in case flag parsing failed we should log which flag was incorrect.
+	logging.Init(LogType, LogLevel)
+
 	// Function to execute before processing commands
 	cobra.OnInitialize(func() {
 		// Set up the logger
@@ -77,6 +81,11 @@ func Execute() {
 	}
 
 	if err := rootCmd.Execute(); err != nil {
+		// check if the error is related to flag argument missing and
+		// log it before terminating the process so user gets idea about the incorrect flag value
+		if strings.Contains(err.Error(), "flag needs an argument") {
+			zap.S().Error("error while executing command ", zap.Error(err))
+		}
 		os.Exit(1)
 	}
 }