feat: use old.reddit.com for RSS feeds and subreddit icons#63
Merged
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
This PR addresses Reddit's rate limiting issues (Sentry issue TUVIX-API-H) by switching from www.reddit.com to old.reddit.com for all RSS feed URLs and subreddit icon API requests. The old.reddit.com domain is known to have more lenient rate limiting for RSS feed access.
Key Changes:
- Updated Reddit discovery service to use
old.reddit.comfor dynamically generated RSS feed URLs (both subreddit and user feeds) - Updated subreddit icon fetching to use
old.reddit.comAPI endpoint - Updated hardcoded Reddit News feed suggestion to use
old.reddit.com
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| packages/tricorder/src/services/reddit-discovery.ts | Modified RSS feed URL generation for subreddits and users, and updated the subreddit icon API endpoint to use old.reddit.com instead of preserving the original domain |
| packages/app/src/components/app/feed-suggestions.tsx | Updated the hardcoded Reddit News feed URL in the suggested feeds list to use old.reddit.com |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Remove unused baseUrl variable that was causing TypeScript compilation error. The feed URLs are hardcoded to use old.reddit.com for better RSS reliability, so the parsed base URL is not needed. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
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
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.
Description
Fixes TUVIX-API-H. The issue was that: Sequential feed fetching without inter-request delays or rate limiting triggers Reddit's strict 429/403 rate limit blocks.
This fix was generated by Seer in Sentry, triggered by Kyle TechSquidTV. 👁️ Run ID: 4971195
Not quite right? Click here to continue debugging with Seer.
Type of Change
Related Issues
Fixes #7077487175
Relates to #
Changes Made
Testing
pnpm test)pnpm type-check)pnpm lint)Screenshots/Videos
Documentation
Checklist
Additional Notes