Revert RSS mime to text/xml

[Denis-de-Bernardy]

Same issue as here:

​http://comox.textdrive.com/pipermail/wp-testers/2009-May/011534.html

On a Mac, you can no longer browse a feed in a browser. Leaving aside that it's annoying when you try to debug a patch that was applied to feed-related files, I'm not entirely certain that things end up working as expected with an XSL stylesheet.

Is there any room to revert this change?

[sivel]

Don't you mean revert to text/xml?

[Denis-de-Bernardy]

indeed. :)

[sivel]

Have you confirmed that this is a problem in Safari or on a Mac? According to ​http://developer.apple.com/internet/safari/faq.html#anchor22 they use application/rss+xml as an example for telling Safari about RSS feeds.

I would test now but my Mac is temporarily out of commission.

[Denis-de-Bernardy]

Yeah. It ends up triggering downloads when you try to view the HTML in Camino. And in safari, it goes straight into RSS Menu.

The safari behavior hasn't changed one bit, but the Camino (and presumably FF, since it's the same engine) issue is very real. Just like the reporter, my ~/Download folder is cluttered with localhost-* files. :-)

patch fixes this.

[ryan]

I think we've gone through this before and had to stick with text/xml to accommodate old browsers. Modern Safari and Firefox have no problems with application types.

[peaceablewhale]

Using text/xml with the charset parameter should be fine. +1 to the patch, unless dropping support to older browsers is acceptable.

[Denis-de-Bernardy]

Replying to ryan:

I think we've gone through this before and had to stick with text/xml to accommodate old browsers. Modern Safari and Firefox have no problems with application types.

not on a mac :-(

or rather, neither has issue with the application/rss+xml mime, it's just that Safari will send it straight to Mail or RSS Menu, and Camino/FF will download it as it would any app...

[ryan]

Hmmm, on Mac both FF and Safari display the feed for me and offer to add it to a reader. In FF, once you pick a default reader I think it always sends there instead of displaying. This happens for me regardless of the type.

Regardless, sounds like we're good to revert.

[ryan]

(In [11334]) Use text/xml for rss feeds for best browser compat. Props Denis-de-Bernardy. fixes #9810

[peaceablewhale]

Reopening this report as the current patch breaks feed autodiscovery. New patch will be attached shortly.

[peaceablewhale]

New patch that does not affect feed autodiscovery

[peaceablewhale]

A new patch has been uploaded.

[ryan]

(In [11358]) Fix feed autodiscovery. Props peaceablewhale. fixes #9810