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

Commit 9787e7c

Browse files
authored
Update missing conversion of OktaAccountName (#285)
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
1 parent 6479115 commit 9787e7c

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

lib/okta.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -698,7 +698,7 @@ func (p *OktaProvider) Retrieve() (sts.Credentials, string, error) {
698698
}
699699

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

0 commit comments

Comments
 (0)