[prev in list] [next in list] [prev in thread] [next in thread]
List: kde-bugs-dist
Subject: [Bug 118793] Problematic characters break feed parsing
From: Pau Capdevila <pau.capdevila () gmail ! com>
Date: 2006-01-31 22:07:52
Message-ID: 20060131220752.4689.qmail () ktown ! kde ! org
[Download RAW message or body]
------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.
http://bugs.kde.org/show_bug.cgi?id=118793
------- Additional Comments From pau.capdevila gmail com 2006-01-31 23:07 -------
I really admire you.
I hope one day I'll be half a programmer as you.
Thank you very much indeed!
Pau
On 31 Jan 2006 17:05:54 -0000, Frank Osterfeld <frank.osterfeld kdemail net>
wrote:
[bugs.kde.org quoted mail]
I really admire you.<br><br>I hope one day I'll be half a programmer as you.<br><br>Thank you \
very much indeed!<br><br>Pau<br><br><div><span class="gmail_quote">On 31 Jan 2006 17:05:54 \
-0000, <b class="gmail_sendername">Frank Osterfeld </b> <<a href="mailto:frank.osterfeld \
kdemail net">frank osterfeld kdemail net</a>> wrote:</span><blockquote class="gmail_quote" \
style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: \
1ex;">
------- You are receiving this mail because: -------<br>You reported the bug, or are watching \
the reporter.<br><br><a \
href="http://bugs.kde.org/show_bug.cgi?id=118793">http://bugs.kde.org/show_bug.cgi?id=118793</a><br>frank.osterfeld
kdemail net changed:<br><br> \
What |Removed \
|Added<br>----------------------------------------------------------------------------<br> \
Status|UNCONFIRMED \
|RESOLVED <br> \
Resolution| & \
nbsp; |FIXED<br><br><br><br>------- \
Additional Comments From frank.osterfeld kdemail net 2006-01-31 18:05 -------<br>SVN \
commit 504290 by osterfeld:<br><br>RSS parser: ignore unknown or invalid version attribute \
value in the <rss> tag and <br>just assume RSS 2.0. The older formats are compatible to \
2.0, so this should work.<br>(at least better than refusing to parse the feeds)<br>BUG: \
118793<br><br><br><br> M +9 -0 ChangeLog<br> \
M +2 -1 src/librss/document.cpp <br><br><br>--- \
branches/KDE/3.5/kdepim/akregator/ChangeLog #504289:504290<br> @ -2,6 +2,15 <br> \
===================<br> (c) 2004-2006 the Akregator authors.<br><br>+Changes after \
1.2.1:<br>+----------------------------- <br>+<br>+Bug fixes:<br>+<br>+ 2006/01/31 RSS parser: \
ignore unknown or invalid version attribute value in the <rss> tag \
and<br>+ just assume RSS \
2.0. The older formats are compatible to 2.0, so this should work. \
<br>+ (at least better \
than refusing to parse the feeds) (#118793) -fo<br>+<br> Changes after 1.2:<br> \
-----------------------------<br><br>--- \
branches/KDE/3.5/kdepim/akregator/src/librss/document.cpp #504289:504290 <br> @ -110,7 \
+110,8 <br> \
d->version = v0_93;<br> else if (attr == \
QString::fromLatin1("0.94"))<br> \
d->version = v0_94;<br>- else if \
(attr.startsWith("2.0") || attr == QString::fromLatin1("2")) // <a \
href="http://www.breuls.org/rss">http://www.breuls.org/rss</a> puts 2.00 in version (BR \
#0000016)<br>+ else // otherwise, we just assume \
a RSS2 compatible feed. As rss2 is \
generally<br>+ // \
backward-compatible, this should work \
<br> d->version = \
v2_0;<br> }<br></blockquote></div><br>
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic