Skip to content

MCP: accept ChatGPT's per-connector OAuth callback - #177

Merged
nessed merged 1 commit into
mainfrom
claude/new-session-j5d3jb
Oct 4, 2026
Merged

nessed merged 1 commit into
mainfrom
claude/new-session-j5d3jb

Conversation

@nessed

@nessed nessed commented Oct 4, 2026

Copy link
Copy Markdown
Owner

ChatGPT's "Create custom MCP server" dialog showed "Some app settings were rejected" for https://akada.vercel.app/api/mcp with OAuth.

Dynamic registration only accepted ChatGPT's old fixed callback. The plugin dialog gives each connector its own https://chatgpt.com/connector/oauth/<id>, so /api/mcp/register answered invalid_redirect_uri.

  • _shared.ts: ChatGPT matches the /connector/oauth/<id> shape, the old fixed callback, and platform.openai.com/apps-manage/oauth
  • register/route.ts: takes several redirect URIs, every one must be a known callback from the same client
  • test + MCP_SETUP.md updated

Typecheck, lint and oauth.test.ts pass locally.

🤖 Generated with Claude Code

https://claude.ai/code/session_01MLHMzQJgKySRBgFiKapP9Y


Generated by Claude Code

…dialog can register

The plugin dialog gives each connector its own https://chatgpt.com/connector/oauth/<id>
redirect. Registration only knew the older fixed one, so ChatGPT reported
'Some app settings were rejected'. Registration now also takes the review tool's
callback and several URIs at once, as long as every one is a known callback.

Co-Authored-By: Claude Sonnet 5.5 <[email protected]>
Claude-Session: https://claude.ai/code/session_01MLHMzQJgKySRBgFiKapP9Y
@vercel

vercel Bot commented Oct 4, 2026 •

Copy link
Copy Markdown

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

Project Deployment Actions Updated
akada Ready Ready Preview Oct 4, 2026 5:49pm UTC

This branch was successfully deployed

1 active deployment
Preview — e1f28e66 Deployed Oct 4, 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