Skip to content

Escape Atom feed content - #81

Merged
NeverBehave merged 1 commit into
masterfrom
codex/fix-feed-escaping
Sep 22, 2026
Merged

NeverBehave merged 1 commit into
masterfrom
codex/fix-feed-escaping

Conversation

@NeverBehave

Copy link
Copy Markdown
Member

Summary

  • escape dynamic metadata and entry titles in both Atom feeds
  • escape rendered HTML content before embedding it in Atom XML
  • address the Copilot finding from Add ramblings section聽#80 and the same pre-existing issue in the main post feed

Testing

  • Eleventy production build
  • xmllint validation for both Atom feeds and sitemap
  • temporary regression fixture containing ampersand and less-than characters

Copilot AI lite review requested due to automatic review settings September 22, 2026 18:07
@NeverBehave
NeverBehave merged commit 18f3a1c into master Sep 22, 2026
2 checks passed

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot review overview

馃數 Needs a closer look

Moderate findings remain because URL values in both feeds are not XML-escaped.

Review effort: Lite
Findings: None

What changed in this PR

Updates both Atom feed templates to XML-escape dynamic metadata, titles, and rendered content.

Changes:

  • Escapes feed metadata and entry titles.
  • Escapes rendered HTML content within Atom XML.
File Summary
src/鈥媐eed.njk Updates escaping for the main Atom feed.
src/鈥媟amblings-feed.njk Updates escaping for the ramblings Atom feed.

馃挕 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

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