Skip to content

fix(sandboxing): propagate managed DNS policy#31646

Closed
viyatb-oai wants to merge 1 commit into
codex/viyatb/linux-managed-dns-routingfrom
codex/viyatb/linux-managed-dns
Closed

fix(sandboxing): propagate managed DNS policy#31646
viyatb-oai wants to merge 1 commit into
codex/viyatb/linux-managed-dns-routingfrom
codex/viyatb/linux-managed-dns

Conversation

@viyatb-oai

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

Copy link
Copy Markdown
Contributor

Summary

  • pass the launch-time domain policy to the Linux sandbox helper only when managed-network enforcement, local binding, proxy ports, and a nonempty allowlist are all active
  • insert the serialized policy before the sandbox command separator without exposing it to the user command
  • keep the existing behavior for disabled enforcement and incomplete managed-network contexts

Testing

  • just test -p codex-network-proxy -p codex-sandboxing -p codex-exec-server-protocol -p codex-exec-server (576 passed)
  • just fix -p codex-network-proxy -p codex-sandboxing -p codex-exec-server-protocol -p codex-exec-server -p codex-core -p codex-linux-sandbox
  • Linux end-to-end coverage is included in feat(linux-sandbox): route DNS through managed proxy #31644

Stack: 3 of 3, on #31644.

Closes #22387.

Co-authored-by: Codex noreply@openai.com
@viyatb-oai viyatb-oai force-pushed the codex/viyatb/linux-managed-dns branch from 955d79c to d2db54f Compare July 8, 2026 21:59
@viyatb-oai viyatb-oai changed the base branch from codex/viyatb/linux-managed-dns-activation to codex/viyatb/linux-managed-dns-routing July 8, 2026 22:03
@viyatb-oai viyatb-oai force-pushed the codex/viyatb/linux-managed-dns-routing branch from d7aae07 to 3a15264 Compare July 8, 2026 22:06
@viyatb-oai viyatb-oai closed this Jul 8, 2026
@viyatb-oai viyatb-oai force-pushed the codex/viyatb/linux-managed-dns branch from d2db54f to d7aae07 Compare July 8, 2026 22:06
@viyatb-oai viyatb-oai reopened this Jul 8, 2026
@viyatb-oai viyatb-oai closed this Jul 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant