Skip to content
This repository has been archived by the owner on May 18, 2021. It is now read-only.

Commit

Permalink
Update missing conversion of OktaAccountName (#285)
Browse files Browse the repository at this point in the history
GetSAMLLoginURL was using hard-coded "okta-creds" instead of the lookup
for OktaAccountName, in cases where multiple accounts are used, this can
result in building an incorrect login URL
  • Loading branch information
xarses authored Mar 19, 2020
1 parent 6479115 commit 9787e7c
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion lib/okta.go
Original file line number Diff line number Diff line change
Expand Up @@ -698,7 +698,7 @@ func (p *OktaProvider) Retrieve() (sts.Credentials, string, error) {
}

func (p *OktaProvider) GetSAMLLoginURL() (*url.URL, error) {
item, err := p.Keyring.Get("okta-creds")
item, err := p.Keyring.Get(p.OktaAccountName)
if err != nil {
log.Debugf("couldnt get okta creds from keyring: %s", err)
return &url.URL{}, err
Expand Down

0 comments on commit 9787e7c

Please sign in to comment.