When adding a new feed (via Home › Administer › Content management › Feed aggregator), provide support for the “feed://” URL scheme in addition to “http://”.

The “feed://” URL scheme is used internally on Mac OS X and commonly generated by the Safari browser when clicking on the “RSS” icon to view feeds in this browser. When a feed is subsequently viewed in Safari, its “feed://” URL — not an “http://” URL — is displayed in the browser’s location field as the current location. Therefore, it becomes easily copied for pasting into Drupal’s administration screens — and could potentially be pulled via scripts, bookmarklets, or other means.

The Aggregator module dutifully responds to these URLs with the error below. For example, when trying to submit the feed of “feed://example.com/feed” it returns:

“The URL feed://example.com/feed is invalid. Please enter a fully-qualified URL, such as http://www.example.com/feed.xml.”

It would save a manual step for users of Safari — and perhaps other apps on Mac OS X — if the Aggregator module would support and handle “feed://” URLs directly. Perhaps it can be a configuration choice for the Aggregator to treat them as standard HTTP feed URLs. If it were to be a configuration option, I would suggest that it be enabled by default. Translations from the “feed://” to “http://” scheme could be logged in the Drupal logs when they happen, and an on-screen notice could be provided on the feed entry screen that says that the URL was translated successfully (or not) after it is submitted in the form.

CommentFileSizeAuthor
#11 499774_11.patch1.08 KBjbrauer
#6 499774_6.patch1.11 KBjbrauer
#2 499774.patch1.11 KBjbrauer

Comments

sun.core’s picture

Version: 7.x-dev » 8.x-dev
jbrauer’s picture

Component: aggregator.module » base system
Status: Active » Needs review
StatusFileSize
new1.11 KB

Here's a two-line patch that adds support for accepting and using feed:// URL's. Although it's most useful in aggregator in core it's really base functionality from common.inc that needs to be altered.

gábor hojtsy’s picture

Version: 8.x-dev » 7.x-dev

I've seen many people bump into this. Not sure that this should be as far off as 8.x. We've added in missing MIME types even to Drupal 6.x, so fully compatible "protocols", like feed:// would be fine to add to 7.x right away I believe.

jbrauer’s picture

Title: Support entry of "feed://" URLs when adding feeds to the feed aggregator » Support use of feed:// URLs as synonyms for http:// in core

Updating title since this affects more than the aggregator module.

aspilicious’s picture

- (?:ftp|https?):\/\/ # Look for ftp, http, or https schemes
+ (?:ftp|https?|feed):\/\/ # Look for ftp, http, or https schemes

comment needs a little work

jbrauer’s picture

StatusFileSize
new1.11 KB

@aspilicious thanks! Updated patch attached.

Nick Lewis’s picture

Status: Needs review » Reviewed & tested by the community

This patch is extremely small, the changes straightforward, and its one small step to making drupal less annoying to the humans to use it. Assuming tests pass RTBC

dries’s picture

Status: Reviewed & tested by the community » Fixed

Committed to CVS HEAD. Thanks.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

jbrauer’s picture

Version: 7.x-dev » 6.x-dev
Status: Closed (fixed) » Patch (to be ported)

we should do this for D6 as well

jbrauer’s picture

Status: Patch (to be ported) » Reviewed & tested by the community
StatusFileSize
new1.08 KB

Patch for Drupal 6

gábor hojtsy’s picture

Status: Reviewed & tested by the community » Fixed

Committed, thanks.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.