Closed
Bug 320818
Opened 20 years ago
Closed 20 years ago
broken parsing on CDATA sections
Categories
(MailNews Core :: Feed Reader, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: sayrer, Assigned: sayrer)
References
()
Details
(4 keywords)
Attachments
(1 file)
|
1.92 KB,
patch
|
mscott
:
review+
|
Details | Diff | Splinter Review |
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
|
Assignee | |
Updated•20 years ago
|
OS: Windows 2000 → All
|
|
Assignee | |
Comment 1•20 years ago
|
||
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)
|
||
Updated•20 years ago
|
Attachment #206525 -
Flags: review?(mscott) → review+
|
|
||
Comment 2•20 years ago
|
||
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
|
|
Assignee | |
Updated•20 years ago
|
Whiteboard: [checkin needed]
|
|
||
Updated•20 years ago
|
Whiteboard: [checkin needed]
|
|
||
Comment 3•20 years ago
|
||
fixed on the trunk. leaving open so i can get this into thunderbird 2.0 after it has baked.
|
|
||
Updated•20 years ago
|
Status: NEW → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
|
|
||
Comment 4•20 years ago
|
||
1.8.0.2 is technically the first release with this Thunderbird fix.
Keywords: fixed1.8.0.1 → fixed1.8.0.2
|
|
||
Comment 5•19 years ago
|
||
verified fixed 1.8.1.3 using Mozilla/5.0 (Windows; U; Windows NT 5.2; en-US; rv:1.8.1.3) Gecko/20070326 Thunderbird/2.0.0.0 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
Keywords: verified1.8.1.3
|
||
Comment 6•18 years ago
|
||
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.
Description
•