Skip to content

Conversation

Chu3laMan
Copy link

@Chu3laMan Chu3laMan commented Sep 23, 2025

Description

Updates the "Testing JWT Authentication" sections in both servlet and webflux documentation to match the consistent structure and style used by other OAuth 2.0 testing sections.

Changes

  • Title updates: Changed section titles from "Testing JWT Authentication" to "Testing OAuth 2.0 Resource Servers with JWT"

  • Added controller examples: Shows when SecurityMockMvcRequestPostProcessors#jwt() and SecurityMockServerConfigurers#mockJwt() are necessary (when controllers expect @AuthenticationPrincipal Jwt)

  • Improved introductions: Added consistent motivation paragraphs similar to other OAuth 2.0 sections
    Enhanced clarity: Better explains the specific use case for JWT testing helpers

Fixes: #10028

Signed-off-by: Houssam El Mansouri <[email protected]>
@Chu3laMan Chu3laMan force-pushed the feature/10028-Align-JWT-Testing-Documentation branch from 848da13 to 53573ab Compare September 23, 2025 12:22
@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Sep 23, 2025
@Chu3laMan
Copy link
Author

@jzheaux can you have a look?

Copy link
Contributor

@jzheaux jzheaux left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi, @Chu3laMan, thanks for looking to improve the docs!

I'm not yet sure what you are wanting to achieve here, and maybe you can help me. The docs you added are duplicates of paragraphs that follow in both sections. Also, since JWT is covered first, phrases like "Similar to opaque token" can be confusing since it hasn't yet been covered.

Can you tell me more what you are wanting to get out of the rearrangement and then I'm happy to help get the docs to that point with you?

@jzheaux jzheaux added status: waiting-for-feedback We need additional information before we can continue and removed status: waiting-for-triage An issue we've not yet triaged labels Oct 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

status: waiting-for-feedback We need additional information before we can continue

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Align JWT Testing Documentation

3 participants