Skip to content

Test executor-routed MCP OAuth token exchange - #29656

Merged
jif-oai merged 1 commit into
mainfrom
jif/executor-mcp-oauth-e2e
Jun 25, 2026
Merged

Test executor-routed MCP OAuth token exchange#29656
jif-oai merged 1 commit into
mainfrom
jif/executor-mcp-oauth-e2e

Conversation

@jif-oai

@jif-oai jif-oai commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

Why

#28529 proves OAuth discovery uses the selected executor, but its end-to-end test stops before the callback and token exchange.

What changed

  • add an executor-only mock token endpoint
  • complete the OAuth callback using the authorization URL's state and redirect_uri
  • assert the PKCE token exchange reaches the executor-only endpoint
  • assert the completion notification reports the selected thread and succeeds

Depends on #28529.

@jif-oai
jif-oai requested a review from a team as a code owner June 25, 2026 09:19
Base automatically changed from jif/oauth-mco to main June 25, 2026 09:31
@jif-oai
jif-oai force-pushed the jif/executor-mcp-oauth-e2e branch from 7e9479c to b93d566 Compare June 25, 2026 09:35
@jif-oai
jif-oai enabled auto-merge (squash) June 25, 2026 09:42
@jif-oai
jif-oai merged commit c38b2e9 into main Jun 25, 2026
31 checks passed
@jif-oai
jif-oai deleted the jif/executor-mcp-oauth-e2e branch June 25, 2026 09:45
@github-actions github-actions Bot locked and limited conversation to collaborators Jun 25, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants