Browse Source

Remove false failed errors.

Signed-off-by: Happy2C0de <46957159+Happy2C0de@users.noreply.github.com>
pull/2233/head
Happy2C0de 5 years ago
parent
commit
1608b473eb
  1. 8
      connector/oidc/oidc.go
  2. 2
      connector/oidc/oidc_test.go

8
connector/oidc/oidc.go

@ -299,10 +299,7 @@ func (c *oidcConnector) createIdentity(ctx context.Context, identity connector.I
preferredUsername, found := claims[prefUsername].(string)
if (!found || c.overrideClaimMapping) && c.preferredUsernameKey != "" {
prefUsername = c.preferredUsernameKey
preferredUsername, found = claims[prefUsername].(string)
if !found {
return identity, fmt.Errorf("missing \"%s\" claim", prefUsername)
}
preferredUsername, _ = claims[prefUsername].(string)
}
hasEmailScope := false
@ -319,9 +316,6 @@ func (c *oidcConnector) createIdentity(ctx context.Context, identity connector.I
if (!found || c.overrideClaimMapping) && c.emailKey != "" {
emailKey = c.emailKey
email, found = claims[emailKey].(string)
if !found {
return identity, fmt.Errorf("missing \"%s\" claim", emailKey)
}
}
if !found && hasEmailScope {

2
connector/oidc/oidc_test.go

@ -253,7 +253,7 @@ func TestHandleCallback(t *testing.T) {
},
},
{
name: "customGroupsKeyButGroupsProvidedButOverride",
name: "customGroupsKeyDespiteGroupsProvidedButOverride",
overrideClaimMapping: true,
groupsKey: "cognito:groups",
expectUserID: "subvalue",

Loading…
Cancel
Save