Skip to content

chore: add minimal proxy egress diagnostics#21220

Merged
viyatb-oai merged 3 commits into
mainfrom
codex/viyatb/minimal-proxy-diagnostics
May 5, 2026
Merged

chore: add minimal proxy egress diagnostics#21220
viyatb-oai merged 3 commits into
mainfrom
codex/viyatb/minimal-proxy-diagnostics

Conversation

@viyatb-oai

@viyatb-oai viyatb-oai commented May 5, 2026

Copy link
Copy Markdown
Collaborator

Why

Recent Auto Review reports show Git traffic hanging through the local proxy on both SSH and HTTPS paths. Today the support bundle does not make it obvious whether a request is stuck before upstream dialing, during the proxy hop, or after the upstream response begins, which slows down root-cause triage.

This adds a small amount of runtime visibility at the existing proxy boundaries without changing routing or policy behavior.

What changed

  • log whether HTTP and CONNECT traffic take the direct or upstream-proxy route
  • log start / success / failure timings for CONNECT, HTTP, and SOCKS5 upstream dials
  • log CONNECT forwarding lifecycle events
  • describe HTTP success at the response-header boundary that is actually observed, rather than implying the full body finished

Verification

  • cargo test -p codex-network-proxy
  • cargo clippy -p codex-network-proxy --all-targets -- -D warnings

viyatb-oai added 3 commits May 5, 2026 10:02
Co-authored-by: Codex noreply@openai.com
Co-authored-by: Codex noreply@openai.com
Co-authored-by: Codex noreply@openai.com
@viyatb-oai viyatb-oai enabled auto-merge (squash) May 5, 2026 17:42
@viyatb-oai viyatb-oai merged commit 172303b into main May 5, 2026
26 checks passed
@viyatb-oai viyatb-oai deleted the codex/viyatb/minimal-proxy-diagnostics branch May 5, 2026 17:51
@github-actions github-actions Bot locked and limited conversation to collaborators May 5, 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