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

Commit

Permalink
Add some action items for the credentials error message
Browse files Browse the repository at this point in the history
  • Loading branch information
Fauzyy committed Jan 24, 2019
1 parent 5100627 commit f87f74c
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions lib/okta.go
Original file line number Diff line number Diff line change
Expand Up @@ -101,7 +101,7 @@ func NewOktaClient(creds OktaCreds, oktaAwsSAMLUrl string, sessionCookie string,
} else if creds.Domain != "" {
domain = creds.Domain
} else {
return &OktaClient{}, errors.New("either creds.Organization (deprecated) or creds.Domain must be set, and not both")
return &OktaClient{}, errors.New("either creds.Organization (deprecated) or creds.Domain must be set, but not both. To remedy this, re-add your credentials with `aws-okta add`")
}

// url parse & set base
Expand Down Expand Up @@ -526,9 +526,9 @@ func (p *OktaProvider) Retrieve() (sts.Credentials, string, error) {
}

newCookieItem := keyring.Item{
Key: p.OktaSessionCookieKey,
Data: []byte(newSessionCookie),
Label: "okta session cookie",
Key: p.OktaSessionCookieKey,
Data: []byte(newSessionCookie),
Label: "okta session cookie",
KeychainNotTrustApplication: false,
}

Expand Down

0 comments on commit f87f74c

Please sign in to comment.