Skip to content

Commit d8cbba7

Browse files
author
Jeff Ploughman
committed
Lowercase claims
1 parent 8f991f5 commit d8cbba7

File tree

1 file changed

+12
-4
lines changed

1 file changed

+12
-4
lines changed

path_login.go

Lines changed: 12 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -63,13 +63,13 @@ func (jwt *JWTMappings) ClaimsList(name string) []string {
6363
for _, v := range listSlice {
6464
item, ok := v.(string)
6565
if ok {
66-
claimsList = append(claimsList, item)
66+
claimsList = append(claimsList, strings.ToLower(item))
6767
}
6868
}
6969
} else {
7070
stringBean, ok := jwt.Claims[name].(string)
7171
if ok {
72-
claimsList = append(claimsList, stringBean)
72+
claimsList = append(claimsList, strings.ToLower(stringBean))
7373
}
7474
}
7575
return claimsList
@@ -404,13 +404,21 @@ func (b *backend) validateJWT(ctx context.Context, req *logical.Request, token s
404404
jwtMappings := &JWTMappings{
405405
Claims: claims,
406406
}
407-
claimPoliciesList, err := b.RoleMap.Policies(ctx, req.Storage, jwtMappings.ClaimsList(config.RoleClaim)...)
407+
claimsList := jwtMappings.ClaimsList(config.RoleClaim)
408+
var claimPoliciesList []string
409+
if claimsList != nil {
410+
claimPoliciesList, err = b.RoleMap.Policies(ctx, req.Storage, claimsList...)
411+
}
408412

409413
if err != nil {
410414
return nil, nil, err
411415
}
412416

413-
userPoliciesList, err := b.UserMap.Policies(ctx, req.Storage, jwtMappings.ClaimsList(config.SubjectClaim)...)
417+
claimsList = jwtMappings.ClaimsList(config.SubjectClaim)
418+
var userPoliciesList []string
419+
if claimsList != nil {
420+
userPoliciesList, err = b.RoleMap.Policies(ctx, req.Storage, claimsList...)
421+
}
414422

415423
if err != nil {
416424
return nil, nil, err

0 commit comments

Comments
 (0)