Skip to content

Comments

fix: include tenant path in auth server issuer claim#152

Merged
pcarleton merged 2 commits intomodelcontextprotocol:mainfrom
Kehrlann:main
Feb 20, 2026
Merged

fix: include tenant path in auth server issuer claim#152
pcarleton merged 2 commits intomodelcontextprotocol:mainfrom
Kehrlann:main

Conversation

@Kehrlann
Copy link
Contributor

@Kehrlann Kehrlann commented Feb 17, 2026

Add the path component ("/tenant1") to the issuer claim of the Authorization Server Metadata document.

Motivation and Context

Fixes gh-140

How Has This Been Tested?

Using this change in the conformance tests for the Java SDK.

Breaking Changes

n/a

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

Additional context

- Fixes modelcontextprotocolgh-140

Signed-off-by: Daniel Garnier-Moiroux <git@garnier.wf>
@pkg-pr-new
Copy link

pkg-pr-new bot commented Feb 17, 2026

Open in StackBlitz

npx https://pkg.pr.new/modelcontextprotocol/conformance/@modelcontextprotocol/conformance@152

commit: 94468d2

@pcarleton pcarleton enabled auto-merge (squash) February 20, 2026 14:31
@pcarleton pcarleton merged commit 50c3cf1 into modelcontextprotocol:main Feb 20, 2026
4 checks passed
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.

AS issuer validation failure

2 participants