Skip to content

Commit

Permalink
fix: find the chosen mfa option in the mfaOptions
Browse files Browse the repository at this point in the history
  • Loading branch information
russell committed Sep 21, 2023
1 parent 98bc113 commit cad42a9
Showing 1 changed file with 11 additions and 5 deletions.
16 changes: 11 additions & 5 deletions pkg/provider/okta/okta.go
Original file line number Diff line number Diff line change
Expand Up @@ -979,16 +979,22 @@ func verifyMfa(oc *Client, oktaOrgHost string, loginDetails *creds.LoginDetails,
duoMfaOptions = append(duoMfaOptions, "Passcode")
}

duoMfaOption := 0
duoMfaOption := -1

if loginDetails.DuoMFAOption == "Duo Push" {
duoMfaOption = 1
} else if loginDetails.DuoMFAOption == "Passcode" {
duoMfaOption = 2
if loginDetails.DuoMFAOption != "" {
for i, v := range duoMfaOptions {
if loginDetails.DuoMFAOption == v {
duoMfaOption = i
}
}
} else {
duoMfaOption = prompter.Choose("Select a DUO MFA Option", duoMfaOptions)
}

if duoMfaOption == -1 {
return "", errors.Errorf("error unsupported MFA option '%s'", loginDetails.DuoMFAOption)
}

if duoMfaOptions[duoMfaOption] == "Passcode" {
//get users DUO MFA Token
token = prompter.StringRequired("Enter passcode")
Expand Down

0 comments on commit cad42a9

Please sign in to comment.