Skip to content

Constrain OpenAPI server variable origins#1104

Closed
RhysSullivan wants to merge 1 commit into
mainfrom
fix/openapi-hosted-origin-guard
Closed

Constrain OpenAPI server variable origins#1104
RhysSullivan wants to merge 1 commit into
mainfrom
fix/openapi-hosted-origin-guard

Conversation

@RhysSullivan

@RhysSullivan RhysSullivan commented Jun 23, 2026

Copy link
Copy Markdown
Owner

What changed

  • Validate OpenAPI server variable overrides before invocation.
  • Reject unbounded server variables that change the request origin.
  • Allow origin changes only when the selected variable value is explicitly enum-bounded by the spec.

Why

A broad OpenAPI server template such as https://{host} let a tool caller redirect authenticated invocations to arbitrary hosts.

Status

Skipped from the DeepSec batch merge in #1106. Left as a standalone draft for follow-up because this can affect legitimate tenant-host OpenAPI specs.

Validation

  • Previous targeted validation passed before the batch decision.
  • Needs a follow-up review of tenant-host compatibility before merge.

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Jun 23, 2026

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 Preview URL Updated (UTC)
✅ Deployment successful!
View logs
executor-marketing ba997aa Commit Preview URL

Branch Preview URL
Jun 23 2026, 06:18 PM

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Jun 23, 2026

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 successful!
View logs
executor-cloud ba997aa Jun 23 2026, 06:20 PM

@github-actions

github-actions Bot commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

Cloudflare preview

Torn down — the PR is closed.

@pkg-pr-new

pkg-pr-new Bot commented Jun 23, 2026

Copy link
Copy Markdown

Open in StackBlitz

@executor-js/cli

npm i https://pkg.pr.new/@executor-js/cli@1104

@executor-js/config

npm i https://pkg.pr.new/@executor-js/config@1104

@executor-js/execution

npm i https://pkg.pr.new/@executor-js/execution@1104

@executor-js/sdk

npm i https://pkg.pr.new/@executor-js/sdk@1104

@executor-js/codemode-core

npm i https://pkg.pr.new/@executor-js/codemode-core@1104

@executor-js/runtime-quickjs

npm i https://pkg.pr.new/@executor-js/runtime-quickjs@1104

@executor-js/plugin-file-secrets

npm i https://pkg.pr.new/@executor-js/plugin-file-secrets@1104

@executor-js/plugin-graphql

npm i https://pkg.pr.new/@executor-js/plugin-graphql@1104

@executor-js/plugin-keychain

npm i https://pkg.pr.new/@executor-js/plugin-keychain@1104

@executor-js/plugin-mcp

npm i https://pkg.pr.new/@executor-js/plugin-mcp@1104

@executor-js/plugin-onepassword

npm i https://pkg.pr.new/@executor-js/plugin-onepassword@1104

@executor-js/plugin-openapi

npm i https://pkg.pr.new/@executor-js/plugin-openapi@1104

executor

npm i https://pkg.pr.new/executor@1104

commit: ba997aa

@RhysSullivan RhysSullivan force-pushed the fix/graphql-hosted-egress-guard branch from 12d79c6 to 36facc5 Compare June 23, 2026 17:00
@RhysSullivan RhysSullivan force-pushed the fix/openapi-hosted-origin-guard branch from f46a78f to e747baa Compare June 23, 2026 17:00
@RhysSullivan RhysSullivan force-pushed the fix/graphql-hosted-egress-guard branch from 36facc5 to cdacfd7 Compare June 23, 2026 17:07
@RhysSullivan RhysSullivan force-pushed the fix/openapi-hosted-origin-guard branch from e747baa to 8f3481d Compare June 23, 2026 17:07
@RhysSullivan RhysSullivan force-pushed the fix/graphql-hosted-egress-guard branch from cdacfd7 to 9c24ce4 Compare June 23, 2026 17:36
@RhysSullivan RhysSullivan force-pushed the fix/openapi-hosted-origin-guard branch from 8f3481d to 9e508f9 Compare June 23, 2026 17:36
@RhysSullivan RhysSullivan force-pushed the fix/graphql-hosted-egress-guard branch 2 times, most recently from f2e5bc2 to 5865d0c Compare June 23, 2026 18:11
@RhysSullivan RhysSullivan changed the base branch from fix/graphql-hosted-egress-guard to main June 23, 2026 18:12
@RhysSullivan RhysSullivan force-pushed the fix/openapi-hosted-origin-guard branch from 9e508f9 to ba997aa Compare June 23, 2026 18:12
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