security: iframe sandbox allows same-origin access to parent page

オープン

#3 2026年3月14日に作成コメント 2 件リアクション 0 件担当者 0 名 GitHub で見る

まだ誰も着手していません。

評価

難易度: 3/5
見積もり時間: 1〜2日
初心者へのやさしさ: 45/100

issue の種類: バグ
明瞭さ: おおむね明確
活発さ: 静か
技術スタック: typescript
領域: frontend, security

調査の方向性

Start at apps/app/src/components/generative-ui/widget-renderer.tsx:523 and inspect how the iframe sandbox and postMessage bridge are configured. Test what breaks when allow-same-origin is removed, including any CDN or fetch behavior mentioned in the issue. Done means the sandbox uses only the permissions required by the renderer, the bridge still works, and the intentional security tradeoff is documented if the permission must remain.

索引モデルが issue の本文から書いたものです。

説明

bug

Description

Severity: P0 — Security

In apps/app/src/components/generative-ui/widget-renderer.tsx:523, the iframe sandbox attribute is:

sandbox="allow-scripts allow-same-origin"

The combination of allow-scripts + allow-same-origin allows the iframe to access window.parent and all parent DOM/cookies. While the CSP mitigates remote script loading, agent-generated HTML with inline scripts has full same-origin access to the host page.

Failure mode

XSS via agent-generated content — malicious or hallucinated HTML could read/modify parent page state, steal cookies, or call CopilotKit APIs.

Suggested fix

Remove allow-same-origin from the sandbox. The iframe only needs allow-scripts. The postMessage bridge already works cross-origin. If allow-same-origin is truly required (e.g., for CDN fetches), tighten CSP connect-src and document the tradeoff.

Open question

Is allow-same-origin intentional? What breaks if it's removed?

From self-review finding F01

主要言語: TypeScript
スター: 1.6k
フォーク: 202
PR マージ指標: 30日以内にマージされた PR はありません

コントリビューションガイド

コントリビューションガイドを開く

はじめの一歩

issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
リポジトリをフォークし、ブランチを切って変更します。
issue 番号を参照したプルリクエストを送ります。

CopilotKit/OpenGenerativeUI のほかの issue

Script dedup key truncation may cause collisions オープン

bug

難易度 2/5 1〜3時間初心者へのやさしさ 72/100

CopilotKit/OpenGenerativeUI#37 · 2026年3月24日に作成
Deduplicate progressive rendering pattern between playbook and mechanical skill オープン

enhancement

難易度 2/5 1〜3時間初心者へのやさしさ 72/100

CopilotKit/OpenGenerativeUI#29 · 2026年3月23日に作成
Agent loop stops after first tool call with non-Claude models (GLM-5): generateSandboxedUi never invoked オープン

難易度 4/5 3〜5日初心者へのやさしさ 48/100

CopilotKit/OpenGenerativeUI#87 · コメント 2 件 · 2026年7月18日に作成
Replace custom widgetRenderer with CopilotKit Open Generative UI オープン

難易度 5/5 1週間以上初心者へのやさしさ 35/100

CopilotKit/OpenGenerativeUI#82 · 2026年4月30日に作成
Edit existing HTML instead of regenerating from scratch オープン

難易度 4/5 3〜5日初心者へのやさしさ 45/100

CopilotKit/OpenGenerativeUI#77 · 2026年4月6日に作成

CopilotKit/OpenGenerativeUI の issue をすべて見る

似ている issue

portal pins drifted from wallet main オープン

難易度 2/5 1〜3時間初心者へのやさしさ 72/100

0xMiden/bridge-portal#132 · 2026年9月21日に作成
Cloudflare scaffold recommends the keyless cloudflare/... model but never declares the AI binding it requires オープン

難易度 2/5 1〜3時間初心者へのやさしさ 78/100

withastro/flue#739 · 2026年9月21日に作成
getEcashRowColors throws on the default theme: buttonGradient is an array, blendHexColors expects a string オープン

難易度 2/5 1〜3時間初心者へのやさしさ 82/100

ZeusLN/zeus#4687 · 2026年9月21日に作成
Do not render a phantom EOF line in read output オープン

area:tools bug good first issue help wanted priority:P2

難易度 2/5 1〜3時間初心者へのやさしさ 90/100

TaewoooPark/Motifcode#14 · 2026年9月21日に作成
Antigravity CLI: PostToolUse hook misidentified as PreToolUse due to `toolCall` check, causing protojson error オープン

bug

難易度 2/5 1〜3時間初心者へのやさしさ 84/100

newrelic-experimental/preflight#793 · コメント 1 件 · 2026年9月21日に作成

TypeScript の issue をもっと見る