Skip to content

fix(network-proxy): serve HTTP proxy listener as HTTP/1#14395

Merged
viyatb-oai merged 1 commit intomainfrom
codex/viyatb/http-proxy-http1-listener
Mar 11, 2026
Merged

fix(network-proxy): serve HTTP proxy listener as HTTP/1#14395
viyatb-oai merged 1 commit intomainfrom
codex/viyatb/http-proxy-http1-listener

Conversation

@viyatb-oai
Copy link
Copy Markdown
Collaborator

@viyatb-oai viyatb-oai commented Mar 11, 2026 •
edited
Loading

Summary

  • switch the local HTTP proxy listener from Rama's auto server to explicit HTTP/1 so CONNECT clients skip the version-sniffing pre-read path
  • move rustls crypto-provider bootstrap into the HTTP proxy runner so direct callers do not need hidden global init
  • add a regression test that exercises a plain HTTP/1 CONNECT request against a live loopback listener
@viyatb-oai viyatb-oai force-pushed the codex/viyatb/http-proxy-http1-listener branch from c3d1357 to 00646ee Compare March 11, 2026 20:56
@openai openai deleted a comment from chatgpt-codex-connector bot Mar 11, 2026
rreichel3-oai
rreichel3-oai approved these changes Mar 11, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@rreichel3-oai rreichel3-oai left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!
@viyatb-oai viyatb-oai merged commit 5259e5e into main Mar 11, 2026
31 checks passed
@viyatb-oai viyatb-oai deleted the codex/viyatb/http-proxy-http1-listener branch March 11, 2026 21:35
@github-actions github-actions bot locked and limited conversation to collaborators Mar 11, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

2 participants

@viyatb-oai @rreichel3-oai