Skip to content

Commit 96d82ec

Browse files
rafaelrenanpachecoeleftherias
authored andcommitted
Fix docs that cause unchecked assignment and NPE
1 parent 8b8267e commit 96d82ec

File tree

2 files changed

+6
-4
lines changed

2 files changed

+6
-4
lines changed

docs/manual/src/docs/asciidoc/_includes/reactive/oauth2/resource-server.adoc

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -464,10 +464,11 @@ static class GrantedAuthoritiesExtractor
464464
implements Converter<Jwt, Collection<GrantedAuthority>> {
465465
466466
public Collection<GrantedAuthority> convert(Jwt jwt) {
467-
Collection<String> authorities = (Collection<String>)
468-
jwt.getClaims().get("mycustomclaim");
467+
Collection<?> authorities = (Collection<?>)
468+
jwt.getClaims().getOrDefault("mycustomclaim", Collections.emptyList());
469469
470470
return authorities.stream()
471+
.map(Object::toString)
471472
.map(SimpleGrantedAuthority::new)
472473
.collect(Collectors.toList());
473474
}

docs/manual/src/docs/asciidoc/_includes/servlet/oauth2/oauth2-resourceserver.adoc

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -496,10 +496,11 @@ static class GrantedAuthoritiesExtractor
496496
implements Converter<Jwt, Collection<GrantedAuthority>> {
497497
498498
public Collection<GrantedAuthority> convert(Jwt jwt) {
499-
Collection<String> authorities = (Collection<String>)
500-
jwt.getClaims().get("mycustomclaim");
499+
Collection<?> authorities = (Collection<?>)
500+
jwt.getClaims().getOrDefault("mycustomclaim", Collections.emptyList());
501501
502502
return authorities.stream()
503+
.map(Object::toString)
503504
.map(SimpleGrantedAuthority::new)
504505
.collect(Collectors.toList());
505506
}

0 commit comments

Comments
 (0)