Skip to content

Conversation

stdevi
Copy link

@stdevi stdevi commented Oct 3, 2025

Changes in this PR

Adding jwtDecoder to the McpServerOAuth2Configurer to set NimbusJwtDecoder.

@Kehrlann
Copy link
Collaborator

Kehrlann commented Oct 3, 2025

@stdevi thanks for contributing.

Before going further, what are you trying to accomplish with this exactly exactly?

@stdevi
Copy link
Author

stdevi commented Oct 3, 2025

@stdevi thanks for contributing.

Before going further, what are you trying to accomplish with this exactly exactly?

I have a project where I would like to use custom rest operations for the resource server. However, this implementation does not allow configuring it. I was hoping to expose jwtDecoderCustomizer to achieve that.

@Kehrlann
Copy link
Collaborator

Kehrlann commented Oct 3, 2025

Let’s inject a full JwtDecoder instead of an customizer.

@stdevi stdevi changed the title feat: add jwtDecoderCustomizer to McpServerOAuth2Configurer feat: add jwtDecoder to McpServerOAuth2Configurer Oct 4, 2025
@stdevi
Copy link
Author

stdevi commented Oct 4, 2025

@Kehrlann should I use NimbusJwtDecoder or rather JwtDecoder interface? If we use JwtDecoder, then the validateAudienceClaim won't be enforced.

@Kehrlann
Copy link
Collaborator

Kehrlann commented Oct 7, 2025

ValidateAudienceClaim is a good point.
I'm away this week, let me pick this back up next week.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants