Add generic getClaim() method in ClaimAccessor

[jgrandja]

We should add a convenience (default) method in ClaimAccessor, as follows:

default <T> T getClaim(String claim) {
    return (T) this.getClaims().get(claim);
}

Related to comment

[ThomasVitale]

Hi @jgrandja, if needed I would be available to work on this issue

[jgrandja]

@ThomasVitale That would be great! It should be pretty straight forward - similar to what we already have in OAuth2AuthorizationRequest.getAttribute() (in master). Let me know if you have any questions. Thanks.

[ThomasVitale]

Hi @jgrandja, I created a PR for the issue, what do you think?

[codemasterover9000]

Hi, we just upgraded to Spring Boot from 2.1.1 to 2.2.0. Now we have a failing test for our custom jwtAuthenticationConverter. It seems the implementation of method Jwt.getClaimAsStringList() has changed.

We had a case test where the authorization server sends a space separated list of authorities. In that case getClaimAsStringList() returned null and we did the parsing ourselves. Now it returns a list with the string value of the claim as its only entry.

This makes it harder to distinguish between an actual list and a space separated string value. Was this intentional?

[jgrandja]

@codemasterover9000 It's best to log a new issue instead of commenting on a closed issue. If you're still having problems, please log a new issue with as much detail as possible.