Skip to content

[MCP Portals] Document OAuth callback URLs - #33915

Merged
kennyj42 merged 3 commits into
cloudflare:productionfrom
sha-e-512:shahed/mcp-438-dashboard-callback-url
Oct 2, 2026
Merged

kennyj42 merged 3 commits into
cloudflare:productionfrom
sha-e-512:shahed/mcp-438-dashboard-callback-url

Conversation

@sha-e-512

@sha-e-512 sha-e-512 commented Oct 1, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • Document the dashboard callback URL
  • Clarify which callback URLs need to be added to the upstream OAuth provider's redirect URI allowlist
  • Specify when each callback URL is used; to summarize:
  1. DCR w/o shared callback @ registration-time: dashboard callback URL
  2. DCR w/ shared callback @ registration-time: dashboard callback URL
  3. DCR w/o shared callback @ connection-time: portal-scoped callback URL
  4. DCR w/ shared callback @ connection-time: shared callback URL
  5. Manual: shared callback URL, always

Documentation checklist

@sha-e-512
sha-e-512 force-pushed the shahed/mcp-438-dashboard-callback-url branch from 7d08804 to 269ad77 Compare October 1, 2026 19:57
@sha-e-512
sha-e-512 marked this pull request as ready for review October 1, 2026 20:00
@sha-e-512
sha-e-512 requested review from a team and asamborski as code owners October 1, 2026 20:00
@cloudflare-docs-bot

cloudflare-docs-bot Bot commented Oct 1, 2026 •

Copy link
Copy Markdown
Contributor

AI Review

✅ Reviewed d6701dc · 0 findings · incremental

Resolved (1) · Dismissed (0)
  • Resolved: Semicolon joins two independent clauses · Verified at head via read_repo_file: line 242 reads "...before authorizing the upstream MCP server. Otherwise, authorization may fail." The semicolon was replaced with a period, splitting the independent clauses into two sentences as the core-content rule requires.
Commands
  • /review Run a review now.
  • /full-review Review the entire PR diff.
  • /disable-auto-review Stop automatic reviews.
  • /rebase Rebase against production.

@kennyj42
kennyj42 enabled auto-merge (squash) October 1, 2026 20:12
auto-merge was automatically disabled October 1, 2026 21:09

Head branch was pushed to by a user without write access

@sha-e-512
sha-e-512 force-pushed the shahed/mcp-438-dashboard-callback-url branch from 269ad77 to d6701dc Compare October 1, 2026 21:09
@kennyj42
kennyj42 enabled auto-merge (squash) October 2, 2026 14:11
@kennyj42
kennyj42 merged commit 1805e7c into cloudflare:production Oct 2, 2026
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants