Merge pull request #12 from appknox/risk-threshold-flag

Change flag risk_threshold to risk-threshold
appknox · Sep 16, 2019 · f705cca · f705cca
2 parents b3da6c8 + 358374a
commit f705cca
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/README.md b/README.md
@@ -107,7 +107,7 @@ $ appknox files 4
 
 - **Upload a file and do cicheck based on the risk threshold**
 
-$ appknox upload ~/Downloads/mfva.apk | xargs appknox cicheck --risk_threshold low
+$ appknox upload ~/Downloads/mfva.apk | xargs appknox cicheck --risk-threshold low
 
 2.3 MiB / 2.3 MiB [==========================================================| 00:00 ] 226.28 KiB/s
 Static Scan Progress:  100 % [==========================================================| ]

diff --git a/cmd/cicheck.go b/cmd/cicheck.go
@@ -28,7 +28,7 @@ var cicheckCmd = &cobra.Command{
 			fmt.Fprintln(os.Stderr, err)
 			os.Exit(1)
 		}
-		riskThreshold, _ := cmd.Flags().GetString("risk_threshold")
+		riskThreshold, _ := cmd.Flags().GetString("risk-threshold")
 		riskThresholdLower := strings.ToLower(riskThreshold)
 		var riskThresholdInt int
 		switch riskThresholdStr := riskThresholdLower; riskThresholdStr {
@@ -52,5 +52,5 @@ var cicheckCmd = &cobra.Command{
 func init() {
 	RootCmd.AddCommand(cicheckCmd)
 	cicheckCmd.Flags().StringP(
-		"risk_threshold", "r", "low", "Risk threshold to fail the command. Available options: low, medium, high")
+		"risk-threshold", "r", "low", "Risk threshold to fail the command. Available options: low, medium, high")
 }