Skip to content

Harden Markdown report rendering - #19

Merged
tjdonley merged 1 commit into
mainfrom
fix/markdown-report-escaping
Apr 26, 2026
Merged

tjdonley merged 1 commit into
mainfrom
fix/markdown-report-escaping

Conversation

@tjdonley

Copy link
Copy Markdown
Owner

Summary

  • Escape untrusted report fields before writing Markdown so issues, evidence, recommendations, and cost rows cannot inject links, HTML, images, reference definitions, or table breaks.
  • Preserve JSON payloads while using safe inline-code fences and safe generated artifact link targets.
  • Add regression coverage for malicious Markdown, HTML, and autolink input.

Verification

  • uv run pytest tests/test_report.py
  • uv run pytest
  • uv build
  • git diff --check

@tjdonley
tjdonley merged commit 38afad9 into main Apr 26, 2026
1 check passed
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