Skip to content

docs(security): make enable_socket: false the headline control for untrusted stdio servers (#1704) - #1793

Merged
Dumbris merged 1 commit into
mainfrom
docs/1704-enable-socket-louder
Oct 26, 2026
Merged

Dumbris merged 1 commit into
mainfrom
docs/1704-enable-socket-louder

Conversation

@Dumbris

@Dumbris Dumbris commented Oct 26, 2026

Copy link
Copy Markdown
Member

Follow-up to #1704 from james-crowley's verification matrix: with the admin socket on, a stdio server under none or sandbox isolation can mint itself an agent token with no key, in both modes. The docs mentioned it in passing; this PR makes enable_socket: false the headline control.

  • docs/configuration.md and docs/configuration/config-file.md: the enable_socket row states the trust boundary and when to turn it off.
  • docs/features/sandbox-isolation.md: a caution box at the top and the socket setting first in "What to do".
  • docs/features/security-quarantine.md: new limit "It gates definitions, not processes".
  • Both docker-isolation pages: the same wording.

Docs only, no code.

…o-child token-minting path (#1704)

A stdio server under none or sandbox isolation can mint agent tokens over the
admin socket with no key. The configuration reference, the sandbox page, the
quarantine limits and the isolation pages now name enable_socket: false as
the setting that matters outside docker isolation.
@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying mcpproxy-docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: 11b8595
Status: ✅  Deploy successful!
Preview URL: https://a81e0e54.mcpproxy-docs.pages.dev
Branch Preview URL: https://docs-1704-enable-socket-loud.mcpproxy-docs.pages.dev

View logs

@Dumbris
Dumbris merged commit 72660ff into main Oct 26, 2026
52 of 57 checks passed
@codecov-commenter

Copy link
Copy Markdown

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants