Skip to content

fix: use issuer for JWT client audience#1031

Merged
DaleSeo merged 1 commit into
mainfrom
fix/client-credentials-jwt-audience
Jul 23, 2026
Merged

fix: use issuer for JWT client audience#1031
DaleSeo merged 1 commit into
mainfrom
fix/client-credentials-jwt-audience

Conversation

@DaleSeo

@DaleSeo DaleSeo commented Jul 23, 2026

Copy link
Copy Markdown
Member

Motivation and Context

The client credentials JWT conformance scenario rejected assertions whose audience defaulted to the token endpoint. Using the authorization server issuer aligns the SDK with the client authentication profile, while retaining the explicit audience override for compatibility.

How Has This Been Tested?

Enabled the auth/client-credentials-jwt conformance suite.

Breaking Changes

None

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update

Checklist

  • I have read the MCP Documentation
  • My code follows the repository's style guidelines
  • New and existing tests pass locally
  • I have added appropriate error handling
  • I have added or updated documentation as needed

@github-actions github-actions Bot added T-documentation Documentation improvements T-dependencies Dependencies related changes T-CI Changes to CI/CD workflows and configuration T-config Configuration file changes T-core Core library changes T-transport Transport layer changes labels Jul 23, 2026
@DaleSeo
DaleSeo force-pushed the fix/client-credentials-jwt-audience branch from ed66b02 to cec7c1a Compare July 23, 2026 16:19
@DaleSeo
DaleSeo marked this pull request as ready for review July 23, 2026 17:27
@DaleSeo
DaleSeo requested a review from a team as a code owner July 23, 2026 17:27
@DaleSeo DaleSeo self-assigned this Jul 23, 2026
alexhancock
alexhancock previously approved these changes Jul 23, 2026
@DaleSeo
DaleSeo force-pushed the fix/client-credentials-jwt-audience branch from cec7c1a to a0020de Compare July 23, 2026 18:19
@alexhancock
alexhancock self-requested a review July 23, 2026 18:24
@DaleSeo
DaleSeo merged commit 190ea44 into main Jul 23, 2026
21 checks passed
@DaleSeo
DaleSeo deleted the fix/client-credentials-jwt-audience branch July 23, 2026 18:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

T-CI Changes to CI/CD workflows and configuration T-config Configuration file changes T-core Core library changes T-dependencies Dependencies related changes T-documentation Documentation improvements T-transport Transport layer changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants