Skip to content

fix: reject missing issuer in authorization server metadata by default - #1054

Merged
DaleSeo merged 1 commit into
mainfrom
jamadeo/require-issuer-by-default
Jul 27, 2026
Merged

fix: reject missing issuer in authorization server metadata by default#1054
DaleSeo merged 1 commit into
mainfrom
jamadeo/require-issuer-by-default

Conversation

@jamadeo

@jamadeo jamadeo commented Jul 27, 2026

Copy link
Copy Markdown
Contributor

#1051 made rmcp tolerant of missing 'issuer' fields in metadata. This PR changes the default, such that clients will need to opt in to the tolerant behavior.

Motivation and Context

How Has This Been Tested?

Breaking Changes

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

@jamadeo
jamadeo requested a review from a team as a code owner July 27, 2026 15:52
@github-actions github-actions Bot added T-core Core library changes T-transport Transport layer changes labels Jul 27, 2026
@jamadeo
jamadeo force-pushed the jamadeo/require-issuer-by-default branch from 9d68844 to 1256db3 Compare July 27, 2026 16:13

@DaleSeo DaleSeo left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks!

@DaleSeo
DaleSeo merged commit 125dbfd into main Jul 27, 2026
22 checks passed
@DaleSeo
DaleSeo deleted the jamadeo/require-issuer-by-default branch July 27, 2026 16:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

T-core Core library changes T-transport Transport layer changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants