Skip to content

Support readable files below sandbox denies#28966

Closed
winston-openai wants to merge 8 commits into
dev/winston/mitm-ca-bundle-helpersfrom
dev/winston/mitm-sandbox-readable-carvebacks
Closed

Support readable files below sandbox denies#28966
winston-openai wants to merge 8 commits into
dev/winston/mitm-ca-bundle-helpersfrom
dev/winston/mitm-sandbox-readable-carvebacks

Conversation

@winston-openai

@winston-openai winston-openai commented Jun 18, 2026

Copy link
Copy Markdown
Contributor

Why

Managed MITM must expose the active CA bundle to a sandboxed command without making the proxy directory, private key, or stale sibling bundles readable.

What

  • add read-only file carvebacks beneath denied parents for Seatbelt and bubblewrap
  • keep each carveback scoped to the active bundle for that command
  • close Linux cleanup-parent bundle descriptors before untrusted code starts
  • cover the split deny/read behavior on macOS and Linux

Stack

Validation

  • just test -p codex-sandboxing
  • just test -p codex-linux-sandbox --no-tests=pass

…to dev/winston/mitm-sandbox-readable-carvebacks
…to dev/winston/mitm-sandbox-readable-carvebacks
@winston-openai winston-openai marked this pull request as ready for review June 19, 2026 00:17

Copy link
Copy Markdown
Contributor Author

Closing as superseded by the focused #29013 and #29014 stack. The replacement isolates the persisted MITM private key and preserves file-backed custom CAs present at Codex startup, with zero codex-core changes. Per-command bundle materialization, generic sandbox carvebacks, and additional shell-snapshot plumbing are intentionally deferred because they are not required for this scoped completion work.

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