-
Notifications
You must be signed in to change notification settings - Fork 6.1k
Add constructor to JwtAuthenticationToken that takes a principal name #6865
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
I'd like to submit a PR for this issue. Taking a look at Sorry if silly question, first time around here :) |
@HaydenMeloche if I understand correctly, you will be mimicking the As far the You might be able to provide a PR for this since it is tagged with I am not part of |
@HaydenMeloche welcome! Yes, @rhamedy's analysis is correct, and yes, it'd be great to see a PR from you. |
I believe this issue can be closed now. |
Closed via f84ab3a |
OAuth2IntrospectionAuthenticationToken
takes a name as a constructor parameter and otherwise defaults to the token's SUB claim for the principal name.JwtAuthenticationToken
, on the other hand, provides no way to customize this.It'd be nice to do the same for
JwtAuthenticationToken
:This encourages composition over inheritance while also keeping the application from having to create a custom class to override this default behavior.
The text was updated successfully, but these errors were encountered: