Skip to content

Honor configured proxy routes for auth refreshes - #34655

Merged
copyberry[bot] merged 1 commit into
mainfrom
copyberry/codex-internal-to-codex-oss/7d54ab3219939a49921c51faf08cd4c2eabca51c
Jul 22, 2026
Merged

Honor configured proxy routes for auth refreshes#34655
copyberry[bot] merged 1 commit into
mainfrom
copyberry/codex-internal-to-codex-oss/7d54ab3219939a49921c51faf08cd4c2eabca51c

Conversation

@copyberry

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

Copy link
Copy Markdown

Honor configured proxy routes for auth refreshes

Why

ChatGPT token refresh requests need to follow the same configured routing policy as other authentication traffic, including environments that use the system proxy.

What changed

  • Require an AuthRouteConfig throughout login, logout, token refresh, personal access token, and agent identity flows.
  • Build auth HTTP clients directly from that configuration instead of falling back to a default HTTP client when routing configuration is absent.

Testing

  • Add an integration test that refreshes a token through a cached system-proxy route and verifies the proxy receives the expected request.
@bolinfest copyberry
Honor configured proxy routes for auth refreshes (#34655)
d4fcb28 
## Why

ChatGPT token refresh requests need to follow the same configured routing policy as other authentication traffic, including environments that use the system proxy.

## What changed

- Require an `AuthRouteConfig` throughout login, logout, token refresh, personal access token, and agent identity flows.
- Build auth HTTP clients directly from that configuration instead of falling back to a default HTTP client when routing configuration is absent.

## Testing

- Add an integration test that refreshes a token through a cached system-proxy route and verifies the proxy receives the expected request.

GitOrigin-RevId: 7d54ab3219939a49921c51faf08cd4c2eabca51c
@copyberry
copyberry Bot force-pushed the copyberry/codex-internal-to-codex-oss/7d54ab3219939a49921c51faf08cd4c2eabca51c branch from 0e76dab to d4fcb28 Compare July 22, 2026 02:27
@copyberry
copyberry Bot merged commit d4fcb28 into main Jul 22, 2026
27 of 32 checks passed
@copyberry
copyberry Bot deleted the copyberry/codex-internal-to-codex-oss/7d54ab3219939a49921c51faf08cd4c2eabca51c branch July 22, 2026 02:28
@github-actions github-actions Bot locked and limited conversation to collaborators Jul 22, 2026
@mh0pe
mh0pe temporarily deployed to issue-triage July 22, 2026 02:31 — with GitHub Actions Inactive
@mh0pe
mh0pe temporarily deployed to issue-triage July 22, 2026 02:31 — with GitHub Actions Inactive
@mh0pe
mh0pe temporarily deployed to issue-triage July 22, 2026 02:31 — with GitHub Actions Inactive
@mh0pe
mh0pe temporarily deployed to issue-triage July 22, 2026 02:33 — with GitHub Actions Inactive
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

7 participants

@mh0pe @jabrison1989 @jimicze @wenqifu @zhang8630 @andradericardop @bolinfest