Test cases: http://atomtests.philringnalda.com/tests/item/title/html-cdata.atom http://atomtests.philringnalda.com/tests/item/title/text-cdata.atom This is pretty bad, it can choke up parsing of other data as well. Also seems to affect RSS2: http://franklinmint.fm/2005/12/18/pathological-rss2.xml
Created attachment 206525 [details] [diff] [review] fix CDATA parsing for AtomIETF This passes all of the testcases in http://atomtests.philringnalda.com/tests/item/title/titles.opml. as reported here: http://www.intertwingly.net/wiki/pie/TitleConformanceTests We could probably use tests for all formats on this kind of thing, but that's another bug.
Attachment #206525 - Flags: review?(mscott)
after this has baked on the trunk for a little while, I'll look at moving this onto the appropriate branch(es).
Target Milestone: --- → Thunderbird2.0
fixed on the trunk. leaving open so i can get this into thunderbird 2.0 after it has baked.
Status: NEW → RESOLVED
Last Resolved: 13 years ago
Keywords: fixed126.96.36.199, fixed1.8.1, regression
Resolution: --- → FIXED
188.8.131.52 is technically the first release with this Thunderbird fix.
Keywords: fixed184.108.40.206 → fixed220.127.116.11
verified fixed 18.104.22.168 using Mozilla/5.0 (Windows; U; Windows NT 5.2; en-US; rv:22.214.171.124) Gecko/20070326 Thunderbird/126.96.36.199 Mnenhy/0.7.5.0 ID:2007032620 - the testcase from comment#o is working fine for me - i can`t reproduce the bug, so adding the verified keyword
Should I have reopened this bug instead of bug 277243? I don't _think_ so, but am not confident enough to skip asking here!
Component: RSS → Feed Reader
Product: Thunderbird → MailNews Core
Target Milestone: Thunderbird2.0 → ---
You need to log in before you can comment on or make changes to this bug.