Skip to content

sbx: note that org policy overrides kit network rules#25231

Merged
dvdksn merged 1 commit into
docker:mainfrom
dvdksn:sbx-kit-org-policy-precedence
Jun 3, 2026
Merged

sbx: note that org policy overrides kit network rules#25231
dvdksn merged 1 commit into
docker:mainfrom
dvdksn:sbx-kit-org-policy-precedence

Conversation

@dvdksn
Copy link
Copy Markdown
Contributor

@dvdksn dvdksn commented Jun 2, 2026

Description

Documents a gap in the Docker Sandboxes (sbx) docs: network rules defined in a kit (allowedDomains / deniedDomains) don't apply when organization governance is active. In that case only organization rules are evaluated, so kit-defined allows and denies are ignored — including domains a kit allows for the agent to reach. This precedence behavior wasn't documented anywhere.

Changes:

  • customize/kits.md — Add an [!IMPORTANT] callout in the "Control network access" section (where a kit author defines network rules) explaining that org governance takes precedence, with a link to Policy precedence.
  • governance/concepts.md — The "Precedence" section previously mentioned only local rules. Both bullets now account for kit-defined network rules: they apply alongside local rules when there's no org governance, and are not evaluated (shown inactive in sbx policy ls) when org governance is active.

Related issues

n/a

🤖 Generated with Claude Code

Kit-defined network rules (allowedDomains/deniedDomains) are ignored when
organization governance is active, since only org rules are evaluated. This
wasn't documented anywhere. Add an IMPORTANT callout in the kit
"Control network access" section and extend the governance precedence section
to account for kit rules.

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

netlify Bot commented Jun 2, 2026

Deploy Preview for docsdocker ready!

Name Link
🔨 Latest commit cb8aeed
🔍 Latest deploy log https://app.netlify.com/projects/docsdocker/deploys/6a1ea18cb05b4b0008ebedb4
😎 Deploy Preview https://deploy-preview-25231--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 dvdksn requested a review from a team June 2, 2026 09:27
@dvdksn dvdksn added the status/review Pull requests that are ready for review label Jun 2, 2026
Copy link
Copy Markdown

@docker-agent docker-agent left a comment

Choose a reason for hiding this comment

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

Assessment: 🟢 APPROVE

Small, well-scoped documentation PR that fills a genuine gap: kit-defined network rules being silently ignored when organization governance is active.

Changes reviewed:

  • customize/kits.md: Adds a correctly-formatted [!IMPORTANT] callout with accurate information and a valid cross-link to the precedence section.
  • governance/concepts.md: Expands both precedence bullets to mention kit-defined network rules alongside local rules — internally consistent and accurate.

No style guide violations, no markdown issues, no missing redirects, no AI-isms detected. The PR is clean.

@dvdksn dvdksn merged commit d9bf784 into docker:main Jun 3, 2026
17 checks passed
@dvdksn dvdksn deleted the sbx-kit-org-policy-precedence branch June 3, 2026 06:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/ai status/review Pull requests that are ready for review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants