Skip to content

fix(news): replace dead BIG10 RSS URL, upgrade HTTP feeds to HTTPS #112

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
5ymb01 wants to merge 1 commit into
base: main
Choose a base branch
from
Open

fix(news): replace dead BIG10 RSS URL, upgrade HTTP feeds to HTTPS #112

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions plugins.json
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"version": "1.0.0",
"last_updated": "2026-04-26",
"last_updated": "2026-04-28",
"plugins": [
{
"id": "hello-world",
Expand Down Expand Up @@ -417,7 +417,7 @@
"last_updated": "2026-02-17",
"verified": true,
"screenshot": "",
"latest_version": "1.0.4"
"latest_version": "1.0.5"
},
{
"id": "stock-news",
Expand Down
6 changes: 3 additions & 3 deletions plugins/news/manager.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -50,13 +50,13 @@ class NewsTickerPlugin(BasePlugin):

# Default RSS feeds
DEFAULT_FEEDS = {
'MLB': 'http://espn.com/espn/rss/mlb/news',
'NFL': 'http://espn.go.com/espn/rss/nfl/news',
'MLB': 'https://www.espn.com/espn/rss/mlb/news',
'NFL': 'https://www.espn.com/espn/rss/nfl/news',
'NCAA FB': 'https://www.espn.com/espn/rss/ncf/news',
'NHL': 'https://www.espn.com/espn/rss/nhl/news',
'NBA': 'https://www.espn.com/espn/rss/nba/news',
'TOP SPORTS': 'https://www.espn.com/espn/rss/news',
'BIG10': 'https://www.espn.com/blog/feed?blog=bigten',
'BIG10': 'https://www.espn.com/espn/rss/ncf/news',
Copy link
Copy Markdown

@coderabbitai coderabbitai Bot Apr 29, 2026 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Potential issue | 🟠 Major

BIG10 now points to the same feed as NCAA FB, causing semantic drift and possible duplicate headlines.

At Line 59, BIG10 now resolves to ncf/news, which is already used by NCAA FB (Line 55). If both are enabled, users can get duplicated content and BIG10 no longer represents conference-specific headlines. Please either (a) rename/deprecate BIG10, or (b) keep BIG10 and add URL-level deduping in fetch/update logic plus README alignment.

🤖 Prompt for AI Agents 
Verify each finding against the current code and only fix it if needed.

In `@plugins/news/manager.py` at line 59, The BIG10 entry in the RSS mapping now
points to the same URL as NCAA FB, causing duplicate headlines; fix by either
(A) restoring or removing BIG10: update the mapping key 'BIG10' to the correct
conference-specific RSS URL (or remove it and add a deprecation comment/README
note) so it no longer equals the 'NCAA FB' URL, or (B) keep both keys but add
URL-level deduplication in the news fetch/update code (e.g., inside the
functions that aggregate feeds such as fetch_and_update_headlines or
fetch_feed_urls) to filter identical feed URLs and dedupe items by canonical
URL/guid before inserting, and update the README to reflect the chosen approach.

Copy link
Copy Markdown
Owner

@ChuckBuilds ChuckBuilds May 6, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@5ymb01 is this intended?

Copy link
Copy Markdown

@coderabbitai coderabbitai Bot May 6, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems like the humans are having a chat. I'll hop back into my burrow for now. If you need me again, just tag @coderabbitai in a new comment, and I'll come hopping out!

'NCAA': 'https://www.espn.com/espn/rss/ncaa/news',
'Other': 'https://www.coveringthecorner.com/rss/current.xml'
}
Expand Down
4 changes: 2 additions & 2 deletions plugins/news/manifest.json
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
{
"id": "news",
"name": "News Ticker",
"version": "1.0.4",
"version": "1.0.5",
"description": "Displays scrolling news headlines from RSS feeds including sports news from ESPN, NCAA updates, and custom RSS sources",
"author": "ChuckBuilds",
"category": "content",
Expand All @@ -21,7 +21,7 @@
"plugin_path": "plugins/news",
"versions": [
{
"version": "1.0.4",
"version": "1.0.5",
"released": "2026-02-17",
"ledmatrix_min_version": "2.0.0"
},
Expand Down