Skip to content

Commit 08fa272

Browse files
chanbinmejzheaux
authored andcommitted
Remove authoritiesClaimName Field
This commit simplfies the logic in JwtGrantedAuthoritiesConverter to no longer need the authoritiesClaimName field. Signed-off-by: chanbinme <[email protected]>
1 parent 164cc46 commit 08fa272

File tree

1 file changed

+3
-6
lines changed

1 file changed

+3
-6
lines changed

oauth2/oauth2-resource-server/src/main/java/org/springframework/security/oauth2/server/resource/authentication/JwtGrantedAuthoritiesConverter.java

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,7 @@ public final class JwtGrantedAuthoritiesConverter implements Converter<Jwt, Coll
5353

5454
private String authoritiesClaimDelimiter = DEFAULT_AUTHORITIES_CLAIM_DELIMITER;
5555

56-
private String authoritiesClaimName;
56+
private Collection<String> authoritiesClaimNames = WELL_KNOWN_AUTHORITIES_CLAIM_NAMES;
5757

5858
/**
5959
* Extract {@link GrantedAuthority}s from the given {@link Jwt}.
@@ -102,14 +102,11 @@ public void setAuthoritiesClaimDelimiter(String authoritiesClaimDelimiter) {
102102
*/
103103
public void setAuthoritiesClaimName(String authoritiesClaimName) {
104104
Assert.hasText(authoritiesClaimName, "authoritiesClaimName cannot be empty");
105-
this.authoritiesClaimName = authoritiesClaimName;
105+
this.authoritiesClaimNames = Collections.singletonList(authoritiesClaimName);
106106
}
107107

108108
private String getAuthoritiesClaimName(Jwt jwt) {
109-
if (this.authoritiesClaimName != null) {
110-
return this.authoritiesClaimName;
111-
}
112-
for (String claimName : WELL_KNOWN_AUTHORITIES_CLAIM_NAMES) {
109+
for (String claimName : this.authoritiesClaimNames) {
113110
if (jwt.hasClaim(claimName)) {
114111
return claimName;
115112
}

0 commit comments

Comments
 (0)