Skip to content

Disable MCP stdio transport by default#238

Merged
RhysSullivan merged 1 commit intomainfrom
fix/mcp-stdio-disabled-by-default
Apr 14, 2026
Merged

Disable MCP stdio transport by default#238
RhysSullivan merged 1 commit intomainfrom
fix/mcp-stdio-disabled-by-default

Conversation

@RhysSullivan
Copy link
Copy Markdown
Owner

@RhysSullivan RhysSullivan commented Apr 14, 2026

Summary

Stdio MCP sources spawn a subprocess that inherits the parent process.env, is fine for local dev - doesn't impact cloud since we're on workers but worth disabling

Stdio MCP sources spawn a subprocess that inherits the parent
`process.env` and accept attacker-controlled `command`/`args`/`cwd`,
which is equivalent to arbitrary code execution and secret
exfiltration on any server-side deployment.

- Add `dangerouslyAllowStdioMCP` option to `mcpPlugin()` (default
  false), gated at `resolveConnectorInput` so add/refresh/source-manager
  paths are all covered by one check.
- Add `createMcpSourcePlugin({ allowStdio })` React factory that hides
  the Remote/Stdio tab and filters stdio presets when disabled.
- Local app opts in on both the server and UI. Cloud keeps the default
  (disabled) on both sides.
@cloudflare-workers-and-pages
Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Updated (UTC)
❌ Deployment failed
View logs
executor-cloud 245cbf8 Apr 14 2026, 05:36 PM

@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new Bot commented Apr 14, 2026

Open in StackBlitz

@executor/sdk

npm i https://pkg.pr.new/RhysSullivan/executor/@executor/sdk@238

@executor/plugin-file-secrets

npm i https://pkg.pr.new/RhysSullivan/executor/@executor/plugin-file-secrets@238

@executor/plugin-google-discovery

npm i https://pkg.pr.new/RhysSullivan/executor/@executor/plugin-google-discovery@238

@executor/plugin-graphql

npm i https://pkg.pr.new/RhysSullivan/executor/@executor/plugin-graphql@238

@executor/plugin-keychain

npm i https://pkg.pr.new/RhysSullivan/executor/@executor/plugin-keychain@238

@executor/plugin-mcp

npm i https://pkg.pr.new/RhysSullivan/executor/@executor/plugin-mcp@238

@executor/plugin-onepassword

npm i https://pkg.pr.new/RhysSullivan/executor/@executor/plugin-onepassword@238

@executor/plugin-openapi

npm i https://pkg.pr.new/RhysSullivan/executor/@executor/plugin-openapi@238

@executor/plugin-workos-vault

npm i https://pkg.pr.new/RhysSullivan/executor/@executor/plugin-workos-vault@238

commit: 245cbf8

@RhysSullivan RhysSullivan merged commit a99a81d into main Apr 14, 2026
5 of 7 checks passed
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.

1 participant