Skip to content

fix(rss): RSS feed respects sitemap-disabled pages#3318

Closed
srgg wants to merge 1 commit into
HugoBlox:mainfrom
srgg:fix/rss-respect-sitemap-disabled-pages
Closed

fix(rss): RSS feed respects sitemap-disabled pages#3318
srgg wants to merge 1 commit into
HugoBlox:mainfrom
srgg:fix/rss-respect-sitemap-disabled-pages

Conversation

@srgg
Copy link
Copy Markdown
Contributor

@srgg srgg commented Apr 7, 2026

πŸš€ What type of change is this?

  • [x ] πŸ› Bug fix (A non-breaking change that fixes an issue)
  • ✨ New feature (A non-breaking change that adds functionality)
  • πŸ’… Style change (A change that only affects formatting, visuals, or styling)
  • πŸ“š Documentation update (Changes to documentation only)
  • 🧹 Refactor or chore (A code change that neither fixes a bug nor adds a feature)
  • πŸ’₯ Breaking change (A fix or feature that would cause existing functionality to not work as expected)

🎯 What is the purpose of this change?

The default RSS feed in Hugo Blox Kit does not respect the frontmatter flag sitemap.disable. As a result, excluded pages can appear in the RSS feed, which is inconsistent with the sitemap and may unintentionally expose content.

This change updates the RSS layout to exclude sitemap-disabled pages, ensuring the feed only contains indexable content.


πŸ“Έ Screenshots or Screencast (if applicable)

Not applicable

ℹ️ Documentation Check

  • [ x] No, this change does not require a documentation update.
  • Yes, I have updated the documentation accordingly (or will in a follow-up PR).

πŸ“œ Contributor Agreement

Thank you for your contribution!

@srgg srgg requested a review from gcushen as a code owner April 7, 2026 19:52
@netlify
Copy link
Copy Markdown

netlify Bot commented Apr 7, 2026

βœ… Deploy Preview for academic-demo canceled.

Name Link
πŸ”¨ Latest commit 577788e
πŸ” Latest deploy log https://app.netlify.com/projects/academic-demo/deploys/69d568eeec2aba0007c60015

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 7, 2026

Wow, your first PR! Welcome to the community! πŸŽ‰

Thank you for this contribution to open source and open research. It makes a huge impact for the thousands of innovators building with Hugo Blox.

If you're wondering about next steps, please read our Contributor Guide for coding standards, how to run the project locally, and how to get help.

We hope this is just the start of your journey with us. Let's build the future together! Join us on Discord to connect with the team and community.

Awesome work, we'll take a look soon! ✨

@srgg srgg force-pushed the fix/rss-respect-sitemap-disabled-pages branch from 484edee to 577788e Compare April 7, 2026 20:28
@gcushen
Copy link
Copy Markdown
Collaborator

gcushen commented Apr 8, 2026

Thanks for the PRs and for identifying that private pages can leak into the RSS feed - that's a real gap we'll fix.

However, HugoBlox uses its own private: true frontmatter param (rather than Hugo's newly added sitemap.disable inspired by private) to control page visibility across all surfaces (sitemap, RSS, 404 suggestions, search engine indexing). This is by design - it's a simpler, flatter param that's more intuitive for our audience.

We'll fix the RSS feed to respect private (which was missing). Supporting Hugo's suggested sitemap.disable as an additional mechanism would be a new feature rather than a bug fix, and we'd want to evaluate that separately as that would add unnecessary bloat and confusion for our community.

@gcushen gcushen closed this Apr 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants