Add RSS feed fetch timeout #588

Merged
Vylpes merged 2 commits from cursor/583-rss-feed-fetch-timeout into feature/epic-528-rss-feeds 2026-10-09 13:55:03 +01:00
Member

Implements RSS feed fetch timeout for #583.

Changes

  • Pass { timeout: 10000 } into rss-parser from RssFeedHelper.FetchFeed (library default is 60s)
  • Keep invalid-URL and fetch/parse errors wrapped as before
  • Add unit coverage for constructor timeout option and timed-out fetch errors

Verification

  • yarn build PASS
  • yarn test PASS (5 suites / 65 tests)

Ready for human review/approval. Does not auto-close the issue (needs testing after merge).

Implements RSS feed fetch timeout for #583. ## Changes - Pass `{ timeout: 10000 }` into `rss-parser` from `RssFeedHelper.FetchFeed` (library default is 60s) - Keep invalid-URL and fetch/parse errors wrapped as before - Add unit coverage for constructor timeout option and timed-out fetch errors ## Verification - `yarn build` PASS - `yarn test` PASS (5 suites / 65 tests) Ready for human review/approval. Does not auto-close the issue (needs testing after merge).
Smithy-bot changed title from WIP: Add RSS feed fetch timeout to Add RSS feed fetch timeout 2026-10-09 12:06:11 +01:00
Vylpes approved these changes 2026-10-09 13:54:39 +01:00
Vylpes merged commit bceb740c21 into feature/epic-528-rss-feeds 2026-10-09 13:55:03 +01:00
Vylpes deleted branch cursor/583-rss-feed-fetch-timeout 2026-10-09 13:55:03 +01:00
Sign in to join this conversation.
No description provided.