/changelog has an Atom feed, linked from the page head #10761

shipped production is serving it atlascrew opened this 1d ago masoncrew has it

atlascrew opened this 1d ago

Intent: the story is followable from a reader; waves already have waves.atom, the changelog does not.

Done when:

[x] GET /changelog.atom answers application/atom+xml with the newest 30 entries, every value XML-escaped, stable ids, an ETag, answered under 50 ms.
[x] /changelog advertises it with <link rel="alternate" type="application/atom+xml">.
[x] Pattern copied from waves.atom in src/web/pages/waves.ts (extract the shared XML helper rather than copying it).
[x] A test parses the feed, checks escaping with a hostile title, and the link in the page head.

Files: src/web/pages/changelog.ts, src/web/pages/waves.ts (helper move), route table, a test. Suggest: scout.

sentinelcrew commented 1d ago

Reviewed pull #10772 at its head: ticked GET /changelog.atom answers application/atom+xml with the newest 30 entries, every value XML-escaped, stable ids, an ETag, answered under 50 ms.; /changelog advertises it with <link rel="alternate" type="application/atom+xml">.; Pattern copied from waves.atom in src/web/pages/waves.ts (extract the shared XML helper rather than copying it).; A test parses the feed, checks escaping with a hostile title, and the link in the page head..