723469 - RSS feeds are not properly recognized [rss2 feeds with isPermaLink=false and no link url only]

Status: RESOLVED FIXED

(MailNews Core :: Feed Reader, defect)

Description

[Matt]

User Agent: Mozilla/5.0 (Windows NT 6.1; rv:10.0) Gecko/20100101 Firefox/10.0
Build ID: 20120129021758

Steps to reproduce:

I pressed "Get Messages" to update my RSS feeds. 


Actual results:

Some feeds are not updating, and one is being reported as an invalid feed in the add-on bar at the bottom of the window. I checked the web site for the problematic RSS feeds, and there are new items available. I then reverted to Thunderbird 9.0.1 and the feeds are behaving properly again.

Comment 1

[Matt]

The two feeds that I know for sure had content available that wasn't being recognized are as follows:
http://podcastrss.play.it/mully-and-hanley-show_mp3_32.xml
http://podcastrss.play.it/the-mcneil-and-spiegel-show_mp3_32.xml

The feed that was being reported as invalid is https://www.audi-mediaservices.com/publish/ms/content/en/public.rss2.gid-oeffentlichkeit.xml

I am running Thunderbird in Windows 7 (32 bit), and the extensions I have installed are Add-On Compatibility Reporter, Calendar Tweaks, Folderpane Tools, Lightning, LightningButton, LookOut and NoGlass.

Comment 2

[Hashem Masoud]

(In reply to Matt from comment #0)
> Some feeds are not updating, and one is being reported as an invalid feed in
> the add-on bar at the bottom of the window. I checked the web site for the
> problematic RSS feeds, and there are new items available. I then reverted to
> Thunderbird 9.0.1 and the feeds are behaving properly again.

So this issue happened just after the upgrade? Is there anything in the error console?

Comment 3

[Matt]

Yes, it happened after the upgrade. After reverting to TB 9.0.1 everything worked properly again. After trying TB 10 again, even in safe mode, the problem persists. The error console is showing the following errors:

Error: Unable to report telemetry.
Source File: resource://gre/modules/PlacesDBUtils.jsm
Line: 975

www.audi-mediaservices.com : server does not support RFC 5746, see CVE-2009-3555

pop.secureserver.net : server does not support RFC 5746, see CVE-2009-3555

Error: not well-formed
Source File: https://www.audi-mediaservices.com/publish/ms/content/en/public.rss2.gid-oeffentlichkeit.xml
Line: 371, Column: 222
Source Code:
 <c:image_link_alt_00 type="string">Franziska Moennig, chair of the Presiding Committee of the „Automotive Circle International”, gave the trophy EuroCarBody Award to plant manager Albrecht Reimold  and the trophy Doors & Closures Innovation Award to Dr. Klaus Koglin, head of the technology development (to the right). Frank Martin, development door systems (to the left) and Hartmut Elsässer, project controlling A6-production, received the certifications.   </c:image_link_alt_00>

Comment 4

[:aceman]

Well, giving that link to IE8 yields this error:
Whitespace is not allowed at this location.
 Line: 378 Character: 222

 <c:image_link_alt_00 type="string">Franziska Moennig, chair of the Presiding Committee of the „Automotive Circle International”, gave the trophy EuroCarBody Award to plant manager Albrecht Reimold  and the trophy Doors & Closures Innovation Award to Dr. Klaus Koglin, head of the technology development (to the right). Frank Martin, development door systems (to the left) and Hartmut Elsässer, project controlling A6-production, received the certifications.   </c:image_link_alt_00>
 
So there could be a real problem in that feed...
Is it legal to use & unescaped as &amp; entity?

Comment 5

[alta88]

although all 3 of these feeds (rss2) are sort of poorly constructed, only the last is cleanly handled by Tb as genuinely broken beyond salvage.

the first 2, despite failing to validate on a bad enclosure length syntax, cause Tb to choke/hang earlier for invalid reasons.  Tb attempts to create a content-base header from a null value; this should be a url.  the feeds specifically don't give a url, so Tb's assumption needs to be fixed/handled.  (the guid is a url but isPermaLink is false, and there is no link tag, so Tb assumes null).

Comment 6

[Matt]

After doing some more looking, the Audi RSS feed is invalid and has issues, so that one should be ignored here. I tried TB 10.0.1 and the problem still persists for the other 2 feeds. They still work perfectly well in TB 9.0.1, so something in the way TB handles RSS feeds has changed.

Comment 7

[alta88]

Attachment: patch

actually, the error is on trying to set the BASE href.  this is a regression from Bug 606818, where a null went through some hoops to become a "".

Comment 9

[Mark Banner (:standard8)]

Checked in:

http://hg.mozilla.org/comm-central/rev/7bb995ee9fa3
http://hg.mozilla.org/releases/comm-aurora/rev/79790980748a
http://hg.mozilla.org/releases/comm-beta/rev/45d5ebb045fe