Skip to content

fix(server): accept up to 20 MCP OAuth redirect URIs so Google can register - #296

Merged
bk-agent-01 merged 1 commit into
bkmainfrom
fix/mcp-oauth-redirect-limit
Oct 10, 2026
Merged

bk-agent-01 merged 1 commit into
bkmainfrom
fix/mcp-oauth-redirect-limit

Conversation

@bk-agent-01

Copy link
Copy Markdown
Collaborator

Problem

Google could not add the bkt3 MCP server. At 2026-10-10 16:23 UTC, Google (108.177.67.129, user agent OpenAuth) sent an 857-byte request to /oauth/mcp/register. bkt3 answered HTTP 400 with Register between 1 and 5 redirect URIs. (the 94-byte body in the trace log matches only this error). Google then showed "The Google redirect URL was rejected by the server".

Fix

Raise MAX_REDIRECT_URIS from 5 to 20 in apps/server/src/auth/McpOAuth.ts. Each URI keeps its 512-character cap and the https/loopback rule. The edit is marked T3-CUSTOM(expbkt3). A fork test checks that 8 and 20 URIs register and 21 are refused.

This goes straight to bkmain at Tushar's request. The deploy restarts bkt3.

🤖 Generated with Claude Code

…gister

Google's MCP connector registers more than 5 redirect URIs, and the
register endpoint refused it with invalid_redirect_uri.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@github-actions github-actions Bot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:XS labels Oct 10, 2026
@bk-agent-01

Copy link
Copy Markdown
Collaborator Author

Merge go: Tushar's explicit request in the T3 session ("fix it in our current bkmain and quickly deploy it now"), 2026-10-10. Free lane (no CODEOWNERS). All checks green on dd34521; the queued Blacksmith 'Transfer report artifact' job never runs on this fork (also pending on #294).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:XS vouch:trusted PR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant