feeds: absolute URLs in entry content, short feed title, one feed in <head> - #87
Merged
Merged
Conversation
…<head> Follows the feed checklist in kevincox.ca's "RSS Feed Best Practices": - New feed_polish plugin (feed_generated signal): resolves every relative href/src in feed entry content and summaries against the entry permalink. The live feed had 21 relative links (e.g. /tools/iam-blast-radius/, iam-blast-radius-architecture-problem.html) that many readers resolve wrongly. Fragments, mailto:, tel: and data: are left alone. Site pages are not touched. - FEED_TITLE = 'RivasSec': feeds get a short title (category feeds keep the " - <category>" suffix); SITENAME stays the page <title>. The old feed title also still advertised Kubernetes. - Only the Atom feed is advertised in <head>; the RSS file is still built and linked in the footer, so existing RSS subscribers are unaffected. - Plugin imports Pelican lazily so its helpers run under the stdlib unittest workflow; 7 new tests (49 total pass). Verified with a publishconf build: 22 entries, 0 relative href/src left in Atom or RSS, titles "RivasSec" / "RivasSec - DevSecOps", one feed link in head. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01W7mZgHQBLP7AB6kHoBFk3y
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Applies the parts of Kevin Cox's "RSS Feed Best Practices" checklist that the live feed missed.
Changes
feed_polishplugin (feed_generatedsignal). It resolves every relativehref/srcin feed entry content and summaries against the entry permalink. The live Atom feed had 21 relative links (for example/tools/iam-blast-radius/andiam-blast-radius-architecture-problem.html) that many readers resolve wrongly. Fragments,mailto:,tel:anddata:links are left alone, and site pages are not touched.FEED_TITLE = 'RivasSec': feeds get a short title, and category feeds keep their- <category>suffix.SITENAMEstays the page<title>. The old feed title also still listed Kubernetes.<head>: only Atom is advertised. The RSS file is still built and linked in the footer, so existing RSS subscribers are unaffected.unittestworkflow. 7 new tests; 49 pass locally.Verified
pelican content -s publishconf.pybuild:RivasSecandRivasSec - DevSecOps<head>has one feed link (Atom)<title>unchangedNot in this PR (Cloudflare settings)
application/atom+xml/application/rss+xml; they are currentlyapplication/xml.🤖 Generated with Claude Code
https://claude.ai/code/session_01W7mZgHQBLP7AB6kHoBFk3y