Skip to content

Restrict MCP OAuth authorization endpoints to HTTP(S) - #47326

Merged
copyberry[bot] merged 1 commit into
mainfrom
copyberry/codex-internal-to-codex-oss/e5f6b4c5556234c3cc2aa56910486e892c011116
Sep 22, 2026
Merged

copyberry[bot] merged 1 commit into
mainfrom
copyberry/codex-internal-to-codex-oss/e5f6b4c5556234c3cc2aa56910486e892c011116

Conversation

@copyberry

@copyberry copyberry Bot commented Sep 22, 2026 •

Copy link
Copy Markdown

Restrict MCP OAuth authorization endpoints to HTTP(S)

What changed

Reject authorization endpoints with schemes other than http or https before client registration or authorization URL return. Document this constraint for mcpServer/oauth/login.

Testing

Make existing asynchronous tests wait for cleanup: wait for ThreadIdle before injecting user input in the guardian cached-score test, and poll for queue consumption before asserting that dispatched items are gone.

## What changed

Reject authorization endpoints with schemes other than `http` or `https` before client registration or authorization URL return. Document this constraint for `mcpServer/oauth/login`.

## Testing

Make existing asynchronous tests wait for cleanup: wait for `ThreadIdle` before injecting user input in the guardian cached-score test, and poll for queue consumption before asserting that dispatched items are gone.

GitOrigin-RevId: e5f6b4c5556234c3cc2aa56910486e892c011116
@copyberry
copyberry Bot force-pushed the copyberry/codex-internal-to-codex-oss/e5f6b4c5556234c3cc2aa56910486e892c011116 branch from b23c626 to 2c2a42e Compare September 22, 2026 17:31
@copyberry
copyberry Bot merged commit 2c2a42e into main Sep 22, 2026
@copyberry
copyberry Bot deleted the copyberry/codex-internal-to-codex-oss/e5f6b4c5556234c3cc2aa56910486e892c011116 branch September 22, 2026 17:31
@github-actions github-actions Bot locked and limited conversation to collaborators Sep 22, 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.

1 participant