Skip to content
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

Multi-tenancy Documentation - com.nimbusds.jwt.proc.JWTProcessor does not have a setJWTClaimSetJWSKeySelector method #10504

Closed
jmax01 opened this issue Nov 15, 2021 · 2 comments
Assignees
Labels
in: docs An issue in Documentation or samples status: backported An issue that has been backported to maintenance branches type: bug A general bug

Comments

@jmax01
Copy link
Contributor

jmax01 commented Nov 15, 2021

In line 326 of docs/modules/ROOT/pages/servlet/oauth2/resource-server/multitenancy.adoc#L326 there is a reference to a com.nimbusds.jwt.proc.JWTProcessor method called setJWTClaimSetJWSKeySelector.

There is no such method on com.nimbusds.jwt.proc.JWTProcessor there is a setJWTClaimsSetAwareJWSKeySelector method however.

image

To Reproduce
Read the doc

Expected behavior
Replace with the correct call.

@jmax01 jmax01 added status: waiting-for-triage An issue we've not yet triaged type: bug A general bug labels Nov 15, 2021
@sjohnr sjohnr added in: docs An issue in Documentation or samples and removed status: waiting-for-triage An issue we've not yet triaged labels Nov 15, 2021
@jzheaux
Copy link
Contributor

jzheaux commented Nov 15, 2021

Thanks, @jmax01! Are you able to submit a PR with the correction?

@jmax01
Copy link
Contributor Author

jmax01 commented Nov 15, 2021

@jzheaux I can if setJWTClaimsSetAwareJWSKeySelector is the correct method?

jmax01 added a commit to jmax01/spring-security that referenced this issue Nov 15, 2021
Replace setJWTClaimSetJWSKeySelector  with setJWTClaimsSetAwareJWSKeySelector
jzheaux pushed a commit that referenced this issue Nov 16, 2021
jzheaux pushed a commit that referenced this issue Nov 16, 2021
jzheaux added a commit that referenced this issue Nov 16, 2021
@spring-projects-issues spring-projects-issues added the status: backported An issue that has been backported to maintenance branches label Nov 16, 2021
jzheaux added a commit that referenced this issue Nov 16, 2021
jzheaux added a commit that referenced this issue Nov 16, 2021
jzheaux added a commit that referenced this issue Nov 16, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
in: docs An issue in Documentation or samples status: backported An issue that has been backported to maintenance branches type: bug A general bug
Projects
None yet
Development

No branches or pull requests

4 participants