Skip to content

Conversation

@Pritesh-30
Copy link
Collaborator

Made changes to avoid stopping early when encountered previously seen entries so newer items later in the feed are not missed.

Fixes: #831

As discussed on CZO, some RSS feeds (such as uptime.com) are ordered oldest-first rather than newest-first. The RSS bot previously assumed reverse chronological ordering and could stop scanning early, causing newer entries later in the feed to be missed.

This change updates the scanning logic so the bot continues iterating through the feed, ensuring newer entries are detected regardless of feed ordering.

How did you test this PR?

  • Tested locally using an oldest-first RSS feed:
    https://status.uptime.com/statuspage/uptime-status/subscribe/rss
  • Verified behavior using the #test here stream
  • First run posted 3 initial entries
  • Second run posted 7 additional newer entries that were previously missed
  • At the third run we had no new feed as all old feed was scanned and stores in hash
Self-review checklist
  • Self-reviewed the changes for clarity and maintainability
    (variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

  • Explains differences from previous plans (e.g., issue description).
  • Highlights technical choices and bugs encountered.
  • Calls out remaining decisions and concerns.
  • Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

  • Each commit is a coherent idea.
  • Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

  • Visual appearance of the changes.
  • Responsiveness and internationalization.
  • Strings and tooltips.
  • End-to-end functionality of buttons, interactions and flows.
  • Corner cases, error conditions, and easily imagined bugs.

@alya
Copy link
Contributor

alya commented Dec 16, 2025

Thanks for the fix! Could you please update your commit message to match the commit style guidelines?

Made changes to avoid stopping early when encountered previously seen
entries so newer items later in the feed are not missed.

Fixes zulip#831
@Pritesh-30
Copy link
Collaborator Author

@alya Yes I have updated the commit message

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.

RSS bot pulls the oldest entries and ignores the newer/latest entries on the feed.

3 participants