[codex-api] observe auth updates on Responses WebSockets#25952
[codex-api] observe auth updates on Responses WebSockets#25952cooper-oai wants to merge 6 commits into
Conversation
|
@codex review |
|
Codex Review: Didn't find any major issues. Delightful! ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 803027bca1
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
13640ba to
6ba3d54
Compare
803027b to
32cf2ff
Compare
|
@codex review |
|
Codex Review: Didn't find any major issues. Already looking forward to the next diff. ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
6ba3d54 to
1d49c22
Compare
32cf2ff to
18d02d2
Compare
|
@codex review |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 18d02d2577
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
@codex review |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 9704d8d222
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
@codex review |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: d2b67b96e9
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
@codex review |
|
Codex Review: Didn't find any major issues. 🎉 ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
1d49c22 to
2bd99fa
Compare
caa07aa to
b2df930
Compare
|
@codex review |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: b2df930c51
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| let loopback_address = url.host_str().and_then(|host| { | ||
| host.parse::<IpAddr>() | ||
| .ok() | ||
| .filter(IpAddr::is_loopback) | ||
| .and_then(|_| url.port_or_known_default().map(|port| (host, port))) |
There was a problem hiding this comment.
Bypass proxies for localhost WebSocket providers
The local tokio-tungstenite connect path honors HTTP_PROXY/ALL_PROXY unless bypassed, but this new bypass only matches numeric loopback IPs because localhost fails host.parse::<IpAddr>(). A provider configured as http://localhost:11434/... therefore still falls through to connect_async_tls_with_config and can be tunneled through an ambient proxy in enterprise/CI environments, breaking local Responses WebSocket providers even though the comment says explicit loopback providers must not be proxied. Include localhost/name-resolution loopback in this bypass.
Useful? React with 👍 / 👎.
2bd99fa to
1bb047e
Compare
b2df930 to
bba3d95
Compare
Summary
Stack
PR 4 of 15. Depends on #25932. Followed by #25989.
Validation
just fmtjust test -p codex-apijust fix -p codex-apijust bazel-lock-checkgit diff --check