set default apiServer, allow to run init and version subcommands without arguments

Author: OriHoch

cmd/command.go

@@ -57,6 +57,18 @@ func commandRun(cmd *cobra.Command, command SchemaCommand) {
 			}
 		}
 	}
+	numErrors := 0
+	if apiClientid == "" {
+		fmt.Printf("ERROR: --%s flag is required\n", flagApiClientid)
+		numErrors += 1
+	}
+	if apiSecret == "" {
+		fmt.Printf("ERROR: --%s flag is required\n", flagApiSecret)
+		numErrors += 1
+	}
+	if numErrors > 0 {
+		os.Exit(exitCodeInvalidFlags)
+	}
 	if command.Run.Cmd == "getList" {
 		commandRunGetList(cmd, command, false, false, nil, "")
 	} else if command.Run.Cmd == "post" {

cmd/global_flags.go

@@ -54,22 +54,15 @@ func loadGlobalFlags() {
 		}
 	}
 	numFailures := 0
-	if apiServer = strings.TrimSpace(viper.GetString(flagApiServerKey)); apiServer == "" {
-		numFailures += 1
-		fmt.Printf("ERROR: --%s flag is required\n", flagApiServer)
-	} else if debug {
-		_, _ = fmt.Fprintf(os.Stderr, "%s = %s\n", flagApiServerKey, apiServer)
+	apiServer = strings.TrimSpace(viper.GetString(flagApiServerKey))
+	apiClientid = strings.TrimSpace(viper.GetString(flagApiClientidKey))
+	apiSecret = strings.TrimSpace(viper.GetString(flagApiSecretKey))
+	if apiServer == "" {
+		apiServer = "https://cloudcli.cloudwm.com"
 	}
-	if apiClientid = strings.TrimSpace(viper.GetString(flagApiClientidKey)); apiClientid == "" {
-		numFailures += 1
-		fmt.Printf("ERROR: --%s flag is required\n", flagApiClientid)
-	} else if debug {
+	if debug {
+		_, _ = fmt.Fprintf(os.Stderr, "%s = %s\n", flagApiServerKey, apiServer)
 		_, _ = fmt.Fprintf(os.Stderr, "%s = %s\n", flagApiClientidKey, apiClientid)
-	}
-	if apiSecret = strings.TrimSpace(viper.GetString(flagApiSecretKey)); apiSecret == "" {
-		numFailures += 1
-		fmt.Printf("ERROR: --%s flag is required\n", flagApiSecret)
-	} else if debug {
 		_, _ = fmt.Fprintf(os.Stderr, "%s = %s\n", flagApiSecretKey, apiSecret)
 	}
 	format = strings.TrimSpace(viper.GetString(flagFormatKey))