1643716 - Remove outdated rss feeds from SeaMonkey

Status: RESOLVED FIXED

(SeaMonkey :: Feed Discovery and Preview, task)

Description

[Frank-Rainer Grahl (:frg)]

We have a few no longer existing rss sites in suite/locales/en-US/chrome/browser/region.properties

Comment 1

[Frank-Rainer Grahl (:frg)]

Attachment: 1643716-1-rss-cleanup-2534.patch

digg and aol are gone. Feedly seems to have changed the api. Only yahoo left. Others may be added at a later date if they support an api.

[Approval Request Comment]
Regression caused by (bug #): --
User impact if declined: rss reader selections fails for some entries.
Testing completed (on m-c, etc.): 2.53.4
Risk to taking this patch (and alternatives if risky): trivial
String changes made by this patch: 1 scheme change 3 removals.

Comment 2

[Frank-Rainer Grahl (:frg)]

Attachment: 1643716-2-rss-cleanup-2534.patch

Drive by scheme https upgrades. Can go into another bug if desired.

Comment 3

[Ian Neal]

Comment on attachment 9154587 [details] [diff] [review]
1643716-2-rss-cleanup-2534.patch

[Triage Comment]
LGTM r/a=me

Comment 4

[Ian Neal]

(In reply to Frank-Rainer Grahl (:frg) from comment #1)

Created attachment 9154586 [details] [diff] [review]
1643716-1-rss-cleanup-2534.patch

digg and aol are gone. Feedly seems to have changed the api. Only yahoo left. Others may be added at a later date if they support an api.

Potential replacement Feedly API is:
https://feedly.com/i/discover/sources/search/leo/%s

Other potential online feed readers:
https://www.inoreader.com?add_feed=%s
https://www.newsblur.com/?url=%s

Did look at feeder.co but not sure there is an API for adding a feed

Comment 5

[Ian Neal]

Also from other locales:
From cs (both seem to be alive):
https://theoldreader.com/feeds/subscribe?url=%s
https://www.g2reader.com/su?u=%s

From fr (seems alive):
https://www.netvibes.com/subscribe.php?url=%s

From ja (feedly one seems better than previous comment, but feedwatcher seems ja only and perhaps broken):
https://feedly.com/i/subscription/feed/%s
https://feedwatcher.net/main/Addfeed?url=%s

Other locales either have some of the above, same as en-US or additional broken ones e.g. Google and Bloglines

Comment 6

[Ian Neal]

Comment on attachment 9154586 [details] [diff] [review]
1643716-1-rss-cleanup-2534.patch

f+ for the moment as we have an updated uri for Feedly and I think there are some additional options that could be added inoreader and maybe netvibes.

Comment 7

[Frank-Rainer Grahl (:frg)]

Attachment: 1643716-3-rss-cleanup-2534.patch

Add back Feedly with a different url and add Inoreader as a start. If requested we can add others later.

[Approval Request Comment]
Regression caused by (bug #): --
User impact if declined: no choices
Testing completed (on m-c, etc.): 2.53.4
Risk to taking this patch (and alternatives if risky): trivial
String changes made by this patch: Adding browser.contentHandlers.types.*

The selection code code seems to have an old bug. If you select a web reader it will always be choosen unless you reset browser.feeds.handler.default, Fixing this is for another bug.

Comment 8

[Frank-Rainer Grahl (:frg)]

Attachment: 1643716-1-rss-cleanup-2534.patch

[Approval Request Comment]
Regression caused by (bug #): --
User impact if declined: non working rss reader sites can be selected.
Testing completed (on m-c, etc.): 2.53.4
Risk to taking this patch (and alternatives if risky): broken already.
String changes made by this patch: rowser.contentHandlers.types.*

Comment 9

[Frank-Rainer Grahl (:frg)]

Attachment: 1643716-2-rss-cleanup-2534.patch

Part 2 rebased after changing part 1. r/a+ from IanN retained.

Comment 10

[Ian Neal]

Comment on attachment 9157963 [details] [diff] [review]
1643716-1-rss-cleanup-2534.patch

[Triage Comment]
LGTM r/a=me

Comment 11

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/a18f6dd87362
Part 1: Replace no longer working rss services in SeaMonkey. r=IanN
https://hg.mozilla.org/comm-central/rev/57ace8b7daf7
Part 2: Upgrade regional http links to https. r=IanN

Comment 12

[Frank-Rainer Grahl (:frg)]

https://gitlab.com/seamonkey-project/seamonkey-2.53-comm/-/commit/b3f57e035ba6a4c61d97d7c245ed96e5689abb08
Part 1: Replace no longer working rss services in SeaMonkey. r=IanN a=IanN
https://gitlab.com/seamonkey-project/seamonkey-2.53-comm/-/commit/c8592d74b9d945e7fbcc74b374688c7d6deaea47
Part 2: Upgrade regional http links to https. r=IanN a=IanN
Target 2.53.4