Closed Bug 341040 Opened 19 years ago Closed 18 years ago

Bad RSS1 detection

Categories

(Firefox Graveyard :: RSS Discovery and Preview, defect)

x86
Windows XP
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: sayrer, Unassigned)

References

()

Details

It should not detect this as a feed.
Our/IE7's heuristic is "anything that's RDF, that uses the namespace prefix "rdf" for RDF, and declares the RSS 1.0 namespace." That's too broad, RSS 1.0 gets a fair amount of reuse in random RDF. We're already insisting that Atom, where there's no reason to assume it will use the default namespace, have at least the document element in the default namespace, and the RSS 1.0 spec actually does require the default namespace, though so obliquely that a lot of people have probably missed it in Syntax: <rdf:RDF xmlns:rdf="http://www.w3.org/1999/02/22-rdf-syntax-ns#" xmlns="http://purl.org/rss/1.0/"> Requirement: Required exactly as shown, aside from any additional namespace declarations So, I'd say for people who can't be bothered to use a mime-type we trust, "<channel" && "http://purl.org/rss/1.0/" && "<rdf:RDF" (in that order, not our current broadest->narrowest) is a better heuristic.
Mmm, if I paid more attention to branch-only checkins, I'd know we need to check for <rdf:RDF first. And since it's nothing but convention to have the <channel> before the <item>s, insisting that <channel be in the first 512 bytes might be too much. xmlns(\s?)=(\s?)('|")http://purl.org/rss/1.0/ maybe?
we now reload when this happens, which is rarely
Status: NEW → RESOLVED
Closed: 18 years ago
Resolution: --- → WONTFIX
Product: Firefox → Firefox Graveyard
You need to log in before you can comment on or make changes to this bug.