Skip to content

Clarify sandbox networking, env sourcing, and approval defaults#24745

Open
dvdksn wants to merge 7 commits intodocker:mainfrom
dvdksn:docs/sbx-clarifications
Open

Clarify sandbox networking, env sourcing, and approval defaults#24745
dvdksn wants to merge 7 commits intodocker:mainfrom
dvdksn:docs/sbx-clarifications

Conversation

@dvdksn
Copy link
Copy Markdown
Contributor

@dvdksn dvdksn commented Apr 16, 2026

Summary

🤖 Generated with Claude Code

dvdksn and others added 3 commits April 16, 2026 09:59
Variables set in /etc/sandbox-persistent.sh inside a sandbox are
only sourced during interactive shell sessions and agent startup.
Running a command directly with `sbx exec <name> <command>` skips
the login shell, so the persistent environment file is not loaded.
Add a note explaining this distinction and a workaround using a
login shell wrapper.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Sandbox agent templates run without approval prompts by default,
which can be surprising. Add a FAQ entry explaining that the
sandbox isolation layers replace the need for per-action approval,
and show how to re-enable approval prompts by passing the agent's
own CLI flags after the -- separator.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…king

Users may expect that adding a domain to the allow list unblocks
all traffic to that host, including SSH. Add a note to the network
policy page stating that allow rules only apply to HTTP/HTTPS, and
add a troubleshooting entry for SSH with a workaround (use HTTPS
for Git).

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@netlify
Copy link
Copy Markdown

netlify bot commented Apr 16, 2026

Deploy Preview for docsdocker ready!

Name Link
🔨 Latest commit 5aac6d4
🔍 Latest deploy log https://app.netlify.com/projects/docsdocker/deploys/69e0e14faafea70008d07b38
😎 Deploy Preview https://deploy-preview-24745--docsdocker.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

dvdksn and others added 3 commits April 16, 2026 10:06
Replace the incorrect --permission-mode flag example with two
workarounds that actually work: changing the permission mode
inside the session (e.g. /permissions in Claude Code), or building
a custom template with different default launch flags.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Blocked non-HTTP requests look the same as other blocked requests
in sbx policy log output, so the advice to check for a PROXY value
of "network" was not useful. Simplify the section.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Verified that sbx exec runs commands without a shell, so the
persistent env file is not sourced. Wrapping in bash -c is enough
because the file is sourced via BASH_ENV, not login profiles.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@dvdksn dvdksn requested a review from a team April 16, 2026 08:15
@dvdksn dvdksn added the status/review Pull requests that are ready for review label Apr 16, 2026
@dvdksn dvdksn marked this pull request as ready for review April 16, 2026 08:15
Comment thread content/manuals/ai/sandboxes/security/policy.md Outdated
Non-HTTP TCP traffic including SSH can be permitted via IP:port policy
rules (docker/sandboxes#2329). UDP and ICMP remain blocked. Update the
policy page, monitoring table, and troubleshooting entry accordingly.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@dvdksn dvdksn requested a review from robmry April 16, 2026 13:18
@dvdksn dvdksn added this to the sbx/next milestone Apr 16, 2026
@dvdksn dvdksn added the status/do-not-merge Pull requests that are awaiting some event or decision before they can be merged. label Apr 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

status/do-not-merge Pull requests that are awaiting some event or decision before they can be merged. status/review Pull requests that are ready for review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants