Skip to content

fix(opencode): allow blob frames in embedded UI CSP - #119

Open
kvnloo wants to merge 1 commit into
devfrom
tip/51636-blob-frames-csp
Open

kvnloo wants to merge 1 commit into
devfrom
tip/51636-blob-frames-csp

Conversation

@kvnloo

@kvnloo kvnloo commented Sep 27, 2026

Copy link
Copy Markdown
Owner

Tip leaf (fork-only diversify) — CLEAN

Closes #118

Maps origin tip anomalyco#51636 (not touched / not promoted).

What

Allow same-origin blob iframes in the embedded UI CSP via frame-src 'self' blob: so blob-backed frames are not blocked.

Proof (fail→pass)

  • GREEN: tip product + tip tests → 3 pass (packages/opencode/test/server/ui-csp.test.ts)
  • NEG: base product + tip tests → 0 pass / 3 fail (missing frame-src 'self' blob:)
  • RESTORE: tip product restored → 3 pass

Notes

  • github_writes_origin=0 — kvnloo fork only
  • Codex promote: parked
  • Tip SHA (origin): d02f9c52c9e9
  • Fork tip SHA: b269d0ce9c17656ebba61ee962693fa638c7edbc

@kvnloo kvnloo added the grok bot Posted by Grok Bot OSS review dump label Sep 27, 2026
@github-actions

Copy link
Copy Markdown

This PR doesn't fully meet our contributing guidelines and PR template.

What needs to be fixed:

  • PR description is missing required template sections. Please use the PR template.

Please edit this PR description to address the above within 2 hours, or it will be automatically closed.

If you believe this was flagged incorrectly, please let a maintainer know.

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

Labels

grok bot Posted by Grok Bot OSS review dump

Projects

None yet

Development

Successfully merging this pull request may close these issues.

tip: allow blob frames in embedded UI CSP (#51636)

1 participant