Add flags PoC

auth0 · Mar 12, 2021 · fa419aa · fa419aa
1 parent c95e71a
commit fa419aa
Show file tree

Hide file tree

Showing 2 changed files with 99 additions and 40 deletions.
diff --git a/internal/cli/apps.go b/internal/cli/apps.go
@@ -13,10 +13,38 @@ import (
 )
 
 const (
-	appID          = "id"
-	appName        = "name"
-	appType        = "type"
-	appDescription = "description"
+	appID   = "id"
+	appType = "type"
+)
+
+var (
+	appName = Flag{
+		Name:          "Name",
+		LongForm:      "name",
+		ShortForm:     "n",
+		DefaultValue:  "",
+		Help:          "Name of the application.",
+		IsRequired:    true,
+		IsInteractive: true,
+	}
+	appDescription = Flag{
+		Name:          "Description",
+		LongForm:      "description",
+		ShortForm:     "d",
+		DefaultValue:  "",
+		Help:          "Description of the application. Max character count is 140.",
+		IsRequired:    false,
+		IsInteractive: true,
+	}
+	appAuthMethod = Flag{
+		Name:          "Auth Method",
+		LongForm:      "auth-method",
+		ShortForm:     "a",
+		DefaultValue:  "",
+		Help:          "Defines the requested authentication method for the token endpoint. Possible values are 'None' (public application without a client secret), 'Post' (application uses HTTP POST parameters) or 'Basic' (application uses HTTP Basic).",
+		IsRequired:    false,
+		IsInteractive: false,
+	}
 )
 
 func appsCmd(cli *cli) *cobra.Command {
@@ -182,21 +210,14 @@ func createAppCmd(cli *cli) *cobra.Command {
 		Short: "Create a new application",
 		Long: `Create a new application:
 
-auth0 apps create --name myapp --type [native|spa|regular|m2m]
+auth0 apps create --Name myapp --type [native|spa|regular|m2m]
 `,
 		PreRun: func(cmd *cobra.Command, args []string) {
 			prepareInteractivity(cmd)
 		},
 		RunE: func(cmd *cobra.Command, args []string) error {
-			if shouldPrompt(cmd, appName) {
-				input := prompt.TextInput(
-					appName, "Name:",
-					"Name of the application. You can change the name later in the application settings.",
-					true)
-
-				if err := prompt.AskOne(input, &flags); err != nil {
-					return fmt.Errorf("An unexpected error occurred: %w", err)
-				}
+			if err := appName.Ask(cmd, &flags.Name); err != nil {
+				return err
 			}
 
 			if shouldPrompt(cmd, appType) {
@@ -215,12 +236,8 @@ auth0 apps create --name myapp --type [native|spa|regular|m2m]
 				}
 			}
 
-			if shouldPrompt(cmd, appDescription) {
-				input := prompt.TextInput(appDescription, "Description:", "Description of the application.", false)
-
-				if err := prompt.AskOne(input, &flags); err != nil {
-					return fmt.Errorf("An unexpected error occurred: %w", err)
-				}
+			if err := appDescription.Ask(cmd, &flags.Description); err != nil {
+				return err
 			}
 
 			a := &management.Client{
@@ -257,20 +274,20 @@ auth0 apps create --name myapp --type [native|spa|regular|m2m]
 		},
 	}
 
-	cmd.Flags().StringVarP(&flags.Name, "name", "n", "", "Name of the application.")
+	appName.RegisterString(cmd, &flags.Name)
 	cmd.Flags().StringVarP(&flags.Type, "type", "t", "", "Type of application:\n"+
 		"- native: mobile, desktop, CLI and smart device apps running natively.\n"+
 		"- spa (single page application): a JavaScript front-end app that uses an API.\n"+
 		"- regular: Traditional web app using redirects.\n"+
 		"- m2m (machine to machine): CLIs, daemons or services running on your backend.")
-	cmd.Flags().StringVarP(&flags.Description, "description", "d", "", "Description of the application. Max character count is 140.")
+	appDescription.RegisterString(cmd, &flags.Description)
 	cmd.Flags().StringSliceVarP(&flags.Callbacks, "callbacks", "c", nil, "After the user authenticates we will only call back to any of these URLs. You can specify multiple valid URLs by comma-separating them (typically to handle different environments like QA or testing). Make sure to specify the protocol (https://) otherwise the callback may fail in some cases. With the exception of custom URI schemes for native apps, all callbacks should use protocol https://.")
 	cmd.Flags().StringSliceVarP(&flags.AllowedOrigins, "origins", "o", nil, "Comma-separated list of URLs allowed to make requests from JavaScript to Auth0 API (typically used with CORS). By default, all your callback URLs will be allowed. This field allows you to enter other origins if necessary. You can also use wildcards at the subdomain level (e.g., https://*.contoso.com). Query strings and hash information are not taken into account when validating these URLs.")
 	cmd.Flags().StringSliceVarP(&flags.AllowedWebOrigins, "web-origins", "w", nil, "Comma-separated list of allowed origins for use with Cross-Origin Authentication, Device Flow, and web message response mode.")
 	cmd.Flags().StringSliceVarP(&flags.AllowedLogoutURLs, "logout-urls", "l", nil, "Comma-separated list of URLs that are valid to redirect to after logout from Auth0. Wildcards are allowed for subdomains.")
-	cmd.Flags().StringVarP(&flags.AuthMethod, "auth-method", "a", "", "Defines the requested authentication method for the token endpoint. Possible values are 'None' (public application without a client secret), 'Post' (application uses HTTP POST parameters) or 'Basic' (application uses HTTP Basic).")
+	appAuthMethod.RegisterString(cmd, &flags.AuthMethod)
 	cmd.Flags().StringSliceVarP(&flags.Grants, "grants", "g", nil, "List of grant types supported for this application. Can include code, implicit, refresh-token, credentials, password, password-realm, mfa-oob, mfa-otp, mfa-recovery-code, and device-code.")
-	mustRequireFlags(cmd, appName, appType)
+	mustRequireFlags(cmd, appName.LongForm, appType)
 
 	return cmd
 }
@@ -296,7 +313,7 @@ func updateAppCmd(cli *cli) *cobra.Command {
 		Short: "Update an application",
 		Long: `Update an application:
 
-auth0 apps update <id> --name myapp --type [native|spa|regular|m2m]
+auth0 apps update <id> --Name myapp --type [native|spa|regular|m2m]
 `,
 		PreRun: func(cmd *cobra.Command, args []string) {
 			prepareInteractivity(cmd)
@@ -316,12 +333,8 @@ auth0 apps update <id> --name myapp --type [native|spa|regular|m2m]
 				inputs.ID = args[0]
 			}
 
-			if shouldPromptWhenFlagless(cmd, appName) {
-				input := prompt.TextInput(appName, "Name:", "Name of the application", true)
-
-				if err := prompt.AskOne(input, &inputs); err != nil {
-					return fmt.Errorf("An unexpected error occurred: %w", err)
-				}
+			if err := appName.AskUpdate(cmd, &inputs.Name); err != nil {
+				return err
 			}
 
 			if shouldPromptWhenFlagless(cmd, appType) {
@@ -340,12 +353,8 @@ auth0 apps update <id> --name myapp --type [native|spa|regular|m2m]
 				}
 			}
 
-			if shouldPromptWhenFlagless(cmd, appDescription) {
-				input := prompt.TextInput(appDescription, "Description:", "Description of the application.", false)
-
-				if err := prompt.AskOne(input, &inputs); err != nil {
-					return fmt.Errorf("An unexpected error occurred: %w", err)
-				}
+			if err := appDescription.AskUpdate(cmd, &inputs.Description); err != nil {
+				return err
 			}
 
 			if shouldPromptWhenFlagless(cmd, "CallbacksString") {
@@ -437,18 +446,18 @@ auth0 apps update <id> --name myapp --type [native|spa|regular|m2m]
 		},
 	}
 
-	cmd.Flags().StringVarP(&inputs.Name, "name", "n", "", "Name of the application.")
+	appName.RegisterString(cmd, &inputs.Name)
 	cmd.Flags().StringVarP(&inputs.Type, "type", "t", "", "Type of application:\n"+
 		"- native: mobile, desktop, CLI and smart device apps running natively.\n"+
 		"- spa (single page application): a JavaScript front-end app that uses an API.\n"+
 		"- regular: Traditional web app using redirects.\n"+
 		"- m2m (machine to machine): CLIs, daemons or services running on your backend.")
-	cmd.Flags().StringVarP(&inputs.Description, "description", "d", "", "Description of the application. Max character count is 140.")
+	appDescription.RegisterString(cmd, &inputs.Description)
 	cmd.Flags().StringSliceVarP(&inputs.Callbacks, "callbacks", "c", nil, "After the user authenticates we will only call back to any of these URLs. You can specify multiple valid URLs by comma-separating them (typically to handle different environments like QA or testing). Make sure to specify the protocol (https://) otherwise the callback may fail in some cases. With the exception of custom URI schemes for native apps, all callbacks should use protocol https://.")
 	cmd.Flags().StringSliceVarP(&inputs.AllowedOrigins, "origins", "o", nil, "Comma-separated list of URLs allowed to make requests from JavaScript to Auth0 API (typically used with CORS). By default, all your callback URLs will be allowed. This field allows you to enter other origins if necessary. You can also use wildcards at the subdomain level (e.g., https://*.contoso.com). Query strings and hash information are not taken into account when validating these URLs.")
 	cmd.Flags().StringSliceVarP(&inputs.AllowedWebOrigins, "web-origins", "w", nil, "Comma-separated list of allowed origins for use with Cross-Origin Authentication, Device Flow, and web message response mode.")
 	cmd.Flags().StringSliceVarP(&inputs.AllowedLogoutURLs, "logout-urls", "l", nil, "Comma-separated list of URLs that are valid to redirect to after logout from Auth0. Wildcards are allowed for subdomains.")
-	cmd.Flags().StringVarP(&inputs.AuthMethod, "auth-method", "a", "", "Defines the requested authentication method for the token endpoint. Possible values are 'None' (public application without a client secret), 'Post' (application uses HTTP POST parameters) or 'Basic' (application uses HTTP Basic).")
+	appAuthMethod.RegisterString(cmd, &inputs.AuthMethod)
 	cmd.Flags().StringSliceVarP(&inputs.Grants, "grants", "g", nil, "List of grant types supported for this application. Can include code, implicit, refresh-token, credentials, password, password-realm, mfa-oob, mfa-otp, mfa-recovery-code, and device-code.")
 
 	return cmd

diff --git a/internal/cli/flags.go b/internal/cli/flags.go
@@ -0,0 +1,50 @@
+package cli
+
+import (
+	"fmt"
+
+	"github.com/auth0/auth0-cli/internal/prompt"
+	"github.com/spf13/cobra"
+)
+
+type Flag struct {
+	Name          string
+	LongForm      string
+	ShortForm     string
+	DefaultValue  string
+	Help          string
+	IsRequired    bool
+	IsInteractive bool
+}
+
+func (f *Flag) Ask(cmd *cobra.Command, value interface{}) error {
+	return ask(cmd, f, value, false)
+}
+
+func (f *Flag) AskUpdate(cmd *cobra.Command, value interface{}) error {
+	return ask(cmd, f, value, true)
+}
+
+func (f *Flag) RegisterString(cmd *cobra.Command, value *string) {
+	cmd.Flags().StringVarP(value, f.LongForm, f.ShortForm, f.DefaultValue, f.Help)
+}
+
+func ask(cmd *cobra.Command, f *Flag, value interface{}, isUpdate bool) error {
+	var shouldAsk bool
+
+	if isUpdate {
+		shouldAsk = shouldPromptWhenFlagless(cmd, f.LongForm)
+	} else {
+		shouldAsk = shouldPrompt(cmd, f.LongForm)
+	}
+
+	if f.IsInteractive && shouldAsk {
+		input := prompt.TextInput("", fmt.Sprintf("%s:", f.Name), f.Help, f.IsRequired)
+
+		if err := prompt.AskOne(input, value); err != nil {
+			return fmt.Errorf("An unexpected error occurred: %w", err)
+		}
+	}
+
+	return nil
+}