Skip to content

Advertise dev OAuth issuer from MCP discovery - #211

Merged
rgarcia merged 1 commit into
mainfrom
hypeship/dev-mcp-oauth-discovery
Sep 22, 2026
Merged

rgarcia merged 1 commit into
mainfrom
hypeship/dev-mcp-oauth-discovery

Conversation

@rgarcia

@rgarcia rgarcia commented Sep 22, 2026

Copy link
Copy Markdown
Contributor

Summary

Route dev MCP OAuth discovery to the canonical dev authorization server after the auth hostname moved to the staging API. Preserve the resource identity and the existing production mapping; other preview and local hosts continue to use their own origins.

Cover direct requests, requests routed through Next with a public Host, near-match hostnames, and the HTTP discovery route.

Validation

  • bun test — 664 passed.
  • Production build passed with placeholder build-time client IDs and Clerk keys.
  • Prettier check passed on changed files (repository-wide check flags the pre-existing AGENTS.md formatting).

This PR changes metadata only; it does not deploy or modify the authorization server.

@vercel

vercel Bot commented Sep 22, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
mcp Ready Ready Preview Sep 22, 2026 9:07pm UTC

@masnwilliams masnwilliams left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Reviewed the discovery mapping and tests. The dev issuer selection is contained in the existing discovery module, preserves production and other host behavior, and introduces no structural quality regression.

@rgarcia
rgarcia merged commit fe48705 into main Sep 22, 2026
10 checks passed
@rgarcia
rgarcia deleted the hypeship/dev-mcp-oauth-discovery branch September 22, 2026 21:25

This branch was successfully deployed

1 active deployment
Preview 194e6bd8 Deployed Sep 22, 2026 by vercel[bot]
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