Merge pull request #82 from Versent/refactor_simpify_configure

refactor(Configure) Simplify the MFA configuration.
Versent · Nov 5, 2017 · db8951b · db8951b
2 parents 7d8b14a + dd24d84
commit db8951b
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 4 deletions.
diff --git a/input.go b/input.go
@@ -28,9 +28,15 @@ func PromptForConfigurationDetails(idpAccount *cfg.IDPAccount) error {
 
 	mfas := MFAsByProvider.Mfas(idpAccount.Provider)
 
-	idpAccount.MFA, err = promptForSelection("\nPlease choose an MFA you would like to use:\n", mfas)
-	if err != nil {
-		return errors.Wrap(err, "error selecting provider file")
+	// only prompt for MFA if there is more than one option
+	if len(mfas) > 1 {
+		idpAccount.MFA, err = promptForSelection("\nPlease choose an MFA you would like to use:\n", mfas)
+		if err != nil {
+			return errors.Wrap(err, "error selecting provider file")
+		}
+
+	} else {
+		idpAccount.MFA = mfas[0]
 	}
 
 	fmt.Println("")

diff --git a/saml2aws.go b/saml2aws.go
@@ -21,7 +21,7 @@ type ProviderList map[string][]string
 var MFAsByProvider = ProviderList{
 	"ADFS":      []string{"None", "VIP"},
 	"ADFS2":     []string{"None"},
-	"Ping":      []string{"PingID"},
+	"Ping":      []string{"Auto"}, // automatically detects PingID
 	"JumpCloud": []string{"None"},
 	"Okta":      []string{"Auto"}, // automatically detects DUO, SMS and ToTP
 	"KeyCloak":  []string{"Auto"}, // automatically detects ToTP