Skip to content

fix: re-register after auth server change#1011

Merged
alexhancock merged 1 commit into
mainfrom
fix/conformance-auth-migration-dispatch
Jul 21, 2026
Merged

fix: re-register after auth server change#1011
alexhancock merged 1 commit into
mainfrom
fix/conformance-auth-migration-dispatch

Conversation

@DaleSeo

@DaleSeo DaleSeo commented Jul 18, 2026

Copy link
Copy Markdown
Member

Motivation and Context

This is a quick follow-up to #1008. It routed auth/authorization-server-migration through the generic auth flow, which made its dedicated conformance path unreachable. Restoring that path exposed a second issue in #998: after clearing the DCR credentials tied to the previous authorization server, initialize_from_store returned an issuer-mismatch error instead of allowing registration with the new server. This completes the authorization-server migration flow and adds a focused regression test.

How Has This Been Tested?

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-core Core library changes T-transport Transport layer changes labels Jul 18, 2026
@DaleSeo DaleSeo self-assigned this Jul 18, 2026
@DaleSeo
DaleSeo marked this pull request as ready for review July 18, 2026 14:00
@DaleSeo
DaleSeo requested a review from a team as a code owner July 18, 2026 14:00
@DaleSeo
DaleSeo force-pushed the fix/conformance-auth-migration-dispatch branch from e83fd8b to 204114a Compare July 21, 2026 14:01
@github-actions github-actions Bot added T-CI Changes to CI/CD workflows and configuration T-config Configuration file changes labels Jul 21, 2026
@alexhancock

Copy link
Copy Markdown
Contributor

Only thing I can see is that now callers can no longer distinguish "AS changed, creds discarded" from "no creds present" as both now return Ok(false) but seems ok

@alexhancock
alexhancock merged commit 4e36fd9 into main Jul 21, 2026
21 checks passed
@alexhancock
alexhancock deleted the fix/conformance-auth-migration-dispatch branch July 21, 2026 16:14
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-transport Transport layer changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants