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

未关闭

#3 创建于 2026年3月14日 2 条评论 0 个 reaction 已指派 0 人在 GitHub 查看

还没有人认领这个 Issue。

评估

难度: 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 下留言说明你要接手 —— 这能避免两个人做同样的事。
Fork 仓库，在一个分支上完成修改。
提交 Pull Request，并在描述里引用这个 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 一周以上新手友好度 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

[MagmaFusion] rapports qualité - contenus des rubriques en anglais manquants 未关闭

难度 2/5 1-3 小时新手友好度 65/100

InseeFr/Bauhaus#1634 · 2026年9月23日创建
[Bug]: SSH host picker duplicates configured aliases and known_hosts targets 未关闭

难度 2/5 1-3 小时新手友好度 75/100

pingdotgg/t3code#13270 · 2026年9月23日创建
resetSseResponseWait and noteRequestProgress silently fail to re-arm the 60s relay timeout if progress tokens arrive as strings 未关闭

bug v2

难度 2/5 1-3 小时新手友好度 75/100

modelcontextprotocol/inspector#2458 · 1 条评论 · 2026年9月23日创建
Resources: Update work of changsha 未关闭

难度 1/5 1 小时以内新手友好度 75/100

railmapgen/rmp-gallery#4068 · 2026年9月23日创建
sass-1.105.0.tgz: 1 vulnerabilities (highest severity is: 7.5) 未关闭

Mend: dependency security vulnerability status: needs triage 🕵️‍♀️

难度 2/5 1-3 小时新手友好度 70/100

carbon-design-system/ibm-products#9907 · 2026年9月23日创建

更多 TypeScript Issue