Skip to content

Promote foss-sandbox to foss-main: fix MCP OAuth redirect after sign-in - #23

Merged
UsamaSadiq merged 2 commits into
foss-mainfrom
foss-sandbox
Jul 15, 2026
Merged

Promote foss-sandbox to foss-main: fix MCP OAuth redirect after sign-in#23
UsamaSadiq merged 2 commits into
foss-mainfrom
foss-sandbox

Conversation

@UsamaSadiq

Copy link
Copy Markdown
Collaborator

Description

Promotes foss-sandbox changes to foss-main, including the MCP OAuth redirect fix: when an unauthenticated user hits /authorize during MCP OAuth (e.g. from Claude.ai), the full OAuth URL is now preserved as returnToPath through the sign-in flow. Previously the user landed on the homepage after login instead of returning to the OAuth consent page.

UsamaSadiq and others added 2 commits July 15, 2026 21:22
When an unauthenticated user hits /authorize during MCP OAuth,
PageChangeEffect discards the full URL (client_id, redirect_uri,
state, code_challenge) and redirects to /sign-in-up. After login
the user lands on the homepage instead of returning to the OAuth
consent page.

Encode the full /authorize?... URL as returnToPath so the existing
post-login redirect mechanism routes back to the consent flow.
Preserve OAuth authorize URL through sign-in redirect
@UsamaSadiq
UsamaSadiq merged commit c77c060 into foss-main Jul 15, 2026
60 of 66 checks passed
@github-actions

Copy link
Copy Markdown
Fails
🚫

node failed.

Log

Details
�[31mError: �[39m SyntaxError: Unexpected token '<', "<!DOCTYPE "... is not valid JSON
    at JSON.parse (<anonymous>)
�[90m    at parseJSONFromBytes (node:internal/deps/undici/undici:4387:19)�[39m
�[90m    at successSteps (node:internal/deps/undici/undici:7041:27)�[39m
�[90m    at readAllBytes (node:internal/deps/undici/undici:5958:13)�[39m
�[90m    at process.processTicksAndRejections (node:internal/process/task_queues:104:5)�[39m
danger-results://tmp/danger-results-0e073389.json

Generated by 🚫 dangerJS against 8a862d7

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.

1 participant