Closed Bug 1430240 Opened 6 years ago Closed 6 years ago

Port bug 1420622 to SeaMonkey (Remove Feed and Pcast related protocols)

Categories

(SeaMonkey :: Feed Discovery and Preview, enhancement, P1)

enhancement

Tracking

(seamonkey2.60 wontfix, seamonkey2.53 unaffected, seamonkey2.57esr fixed)

RESOLVED FIXED
Future
Tracking Status
seamonkey2.60 --- wontfix
seamonkey2.53 --- unaffected
seamonkey2.57esr --- fixed

People

(Reporter: Gijs, Assigned: frg)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

+++ This bug was initially created as a clone of Bug #1420622 +++

pcast:// and feed:// protocols were never standardised and are very rarely used so won't impact a large number of users however have been a target in the past for security issues.

Bug 1345546 was to add telemetry for feed protocol usage which is likely only as high as it is due to the use of "feed:" prefixing from a few menu items in Firefox.

This work requires the testing of feed related "subscribe to this page" features to ensure that removing the protocol won't break those.


-------------------------


Note that once we remove simple nested URI creation off nsINetUtil (bug 1430237) this code will start breaking otherwise.
Blocks: 1430237
Assignee: nobody → frgrahl
Status: NEW → ASSIGNED
You can test it in 2.57 by subscribing via live bookmarks.

There was only one usage of mSecMan left so I replaced it with the stock urlSecurityCheck call.
Attachment #9007456 - Flags: review?(iann_bugzilla)
Attachment #9007456 - Flags: approval-comm-esr60?
Depends on: 1489307
Comment on attachment 9007456 [details] [diff] [review]
1430240-feeds.patch

r/a=me
Attachment #9007456 - Flags: review?(iann_bugzilla)
Attachment #9007456 - Flags: review+
Attachment #9007456 - Flags: approval-comm-esr60?
Attachment #9007456 - Flags: approval-comm-esr60+
Pushed by frgrahl@gmx.net:
https://hg.mozilla.org/comm-central/rev/b0465ba35457
Port bug 1420622 [Remove Feed and Pcast related protocols] to SeaMonkey. r=IanN
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: