Skip to content

Use configured HTTP clients for all MCP OAuth requests - #35814

Merged
copyberry[bot] merged 1 commit into
mainfrom
copyberry/codex-internal-to-codex-oss/1345f56bd794626533133447597a5908e57b9b48
Jul 28, 2026
Merged

Use configured HTTP clients for all MCP OAuth requests#35814
copyberry[bot] merged 1 commit into
mainfrom
copyberry/codex-internal-to-codex-oss/1345f56bd794626533133447597a5908e57b9b48

Conversation

@copyberry

@copyberry copyberry Bot commented Jul 28, 2026 •
edited
Loading

Copy link
Copy Markdown

Use configured HTTP clients for all MCP OAuth requests

What changed

  • Require callers to provide an HTTP client for MCP OAuth discovery and login, removing the separate direct reqwest path.
  • Use the shared http and url types throughout the MCP client and drop its direct reqwest dependency.
  • Preserve configured MCP headers when routing OAuth discovery through the provided client.

Testing

  • Add coverage that routed OAuth discovery forwards configured headers.
@celia-oai copyberry
Use configured HTTP clients for all MCP OAuth requests (#35814)
709283b 
## What changed

- Require callers to provide an HTTP client for MCP OAuth discovery and login, removing the separate direct `reqwest` path.
- Use the shared `http` and `url` types throughout the MCP client and drop its direct `reqwest` dependency.
- Preserve configured MCP headers when routing OAuth discovery through the provided client.

## Testing

- Add coverage that routed OAuth discovery forwards configured headers.

GitOrigin-RevId: 1345f56bd794626533133447597a5908e57b9b48
@copyberry
copyberry Bot force-pushed the copyberry/codex-internal-to-codex-oss/1345f56bd794626533133447597a5908e57b9b48 branch from 772d78c to 709283b Compare July 28, 2026 19:11
@copyberry
copyberry Bot merged commit 709283b into main Jul 28, 2026
25 of 43 checks passed
@copyberry
copyberry Bot deleted the copyberry/codex-internal-to-codex-oss/1345f56bd794626533133447597a5908e57b9b48 branch July 28, 2026 19:12
@github-actions github-actions Bot locked and limited conversation to collaborators Jul 28, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

4 participants

@daveladouceur @WeizhongWang9 @lancy69 @celia-oai