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

Update the OAuth2 jwt and opaque Resource Server documentation #15362

Conversation

arey
Copy link
Contributor

@arey arey commented Jul 4, 2024

Prefer usage of the Lambda DSL.
The OAuth2ResourceServerConfigurer::opaqueToken and OAuth2ResourceServerConfigurer::jwt methods are deprecated since Spring Security 6.1.
With Spring Security 6.3, it looks like both methods are not working.

…he Lambda DSL

The OAuth2ResourceServerConfigurer::opaqueToken() and ::jwt() methods are deprecated since Spring Security 6.1
@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Jul 4, 2024
@jzheaux
Copy link
Contributor

jzheaux commented Jul 9, 2024

Thanks, @arey, I think this is a good idea.

Can you clarify what you mean by "it looks like both methods are not working"? What is it that isn't working for you?

@jzheaux jzheaux added in: docs An issue in Documentation or samples type: enhancement A general enhancement and removed status: waiting-for-triage An issue we've not yet triaged labels Jul 9, 2024
@jzheaux jzheaux self-assigned this Jul 9, 2024
@arey
Copy link
Contributor Author

arey commented Jul 9, 2024

Hi @jzheaux
Please disregard my last comment : With Spring Security 6.3, it looks like both methods are not working.
Instead of displaying a warning following the use of the deprecated method, IntelliJ displays an error. But the code compiles fine.
Saying that, I'd like to see the target method referenced in the documentation.

@jzheaux jzheaux added this to the 6.4.0-M1 milestone Jul 9, 2024
@jzheaux jzheaux merged commit f184d13 into spring-projects:main Jul 9, 2024
4 checks passed
@jzheaux
Copy link
Contributor

jzheaux commented Jul 9, 2024

Thanks, @arey! This is now merged into main.

@arey
Copy link
Contributor Author

arey commented Jul 10, 2024

You're welcome. Thank you for all your work on Spring Security @jzheaux

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 type: enhancement A general enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants