Skip to content

chore: harden markdown renderer sanitization#2595

Merged
mburri merged 4 commits into
mainfrom
chore/harden-markdown-renderer-sanitization
Jul 9, 2026
Merged

chore: harden markdown renderer sanitization#2595
mburri merged 4 commits into
mainfrom
chore/harden-markdown-renderer-sanitization

Conversation

@mburri

@mburri mburri commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

Remove rehype-raw and add an explicit sanitize allowlist to the markdown
renderer. Previously, raw HTML in input could produce unintended live DOM
elements. The fix covers all render surfaces: chart title, description,
dashboard meta, text blocks, and annotation tooltips.


  • I added a CHANGELOG entry
  • I made a self-review of my own code
  • I wrote tests for the changes (if applicable)
  • I wrote configurator and chart config migrations (if applicable)

@vercel

vercel Bot commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
visualization-tool Ready Ready Preview, Comment Jul 9, 2026 11:04am

Request Review

@mburri mburri requested a review from hupf July 8, 2026 14:06
@hupf hupf changed the base branch from main to chore/dev-containers July 9, 2026 07:05
Comment thread app/components/markdown.tsx
@mburri mburri force-pushed the chore/dev-containers branch from ff69e62 to 26241a0 Compare July 9, 2026 09:49
Base automatically changed from chore/dev-containers to main July 9, 2026 10:52
@mburri mburri force-pushed the chore/harden-markdown-renderer-sanitization branch from 45193d1 to a43c275 Compare July 9, 2026 10:58
@mburri mburri merged commit c171349 into main Jul 9, 2026
12 checks passed
@mburri mburri deleted the chore/harden-markdown-renderer-sanitization branch July 9, 2026 11:21
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.

2 participants