From 9787e7c58ed8e6347da37149bb192f2280011736 Mon Sep 17 00:00:00 2001
From: Andrew Woodward <xarses@gmail.com>
Date: Thu, 19 Mar 2020 09:59:14 -0700
Subject: [PATCH] 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
---
 lib/okta.go | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lib/okta.go b/lib/okta.go
index 38a9d74d..98b531aa 100644
--- a/lib/okta.go
+++ b/lib/okta.go
@@ -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