Skip to content

feat(sandbox): enforce proxy-aware network routing in sandbox#11113

Merged
viyatb-oai merged 23 commits intomainfrom
codex/viyatb/sandbox-proxy-wiring
Feb 10, 2026
Merged

feat(sandbox): enforce proxy-aware network routing in sandbox#11113
viyatb-oai merged 23 commits intomainfrom
codex/viyatb/sandbox-proxy-wiring

Conversation

@viyatb-oai
Copy link
Collaborator

@viyatb-oai viyatb-oai commented Feb 8, 2026

Summary

  • expand proxy env injection to cover common tool env vars (HTTP_PROXY/HTTPS_PROXY/ALL_PROXY/NO_PROXY families + tool-specific variants)
  • harden macOS Seatbelt network policy generation to route through inferred loopback proxy endpoints and fail closed when proxy env is malformed
  • thread proxy-aware Linux sandbox flags and add minimal bwrap netns isolation hook for restricted non-proxy runs
  • add/refresh tests for proxy env wiring, Seatbelt policy generation, and Linux sandbox argument wiring

gated on [experimental_network] enabled in requirements.toml

@viyatb-oai viyatb-oai force-pushed the codex/viyatb/sandbox-proxy-wiring branch from e3a69d3 to f16c393 Compare February 8, 2026 21:30
@bolinfest bolinfest force-pushed the pr11105 branch 3 times, most recently from a26207b to 954e0d1 Compare February 8, 2026 21:35
@viyatb-oai viyatb-oai changed the title eat(sandbox): enforce proxy-aware network routing in sandbox feat(sandbox): enforce proxy-aware network routing in sandbox Feb 8, 2026
@bolinfest bolinfest force-pushed the pr11105 branch 5 times, most recently from 1c3bade to 714e595 Compare February 9, 2026 02:42
@viyatb-oai viyatb-oai force-pushed the codex/viyatb/sandbox-proxy-wiring branch 2 times, most recently from d2b8198 to c5b31de Compare February 9, 2026 02:48
Base automatically changed from pr11105 to main February 9, 2026 03:32
@viyatb-oai viyatb-oai force-pushed the codex/viyatb/sandbox-proxy-wiring branch from c5b31de to 20006b9 Compare February 9, 2026 03:38
@viyatb-oai viyatb-oai marked this pull request as ready for review February 9, 2026 06:47
Copy link
Contributor

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 8862494880

ℹ️ 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".

Copy link
Collaborator

@nornagon-openai nornagon-openai left a comment

Choose a reason for hiding this comment

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

sandbox changes seem safe in that they are strictly less permissive than previously. didn't review the rest of the PR in depth.

@viyatb-oai viyatb-oai force-pushed the codex/viyatb/sandbox-proxy-wiring branch from e219116 to 7c22f77 Compare February 10, 2026 06:19
@viyatb-oai viyatb-oai enabled auto-merge (squash) February 10, 2026 07:43
@viyatb-oai viyatb-oai merged commit 3391e5e into main Feb 10, 2026
41 of 44 checks passed
@viyatb-oai viyatb-oai deleted the codex/viyatb/sandbox-proxy-wiring branch February 10, 2026 07:44
@github-actions github-actions bot locked and limited conversation to collaborators Feb 10, 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.

3 participants