Bug 97180 - Konqueror shows link to RSS feed with wrong mime type
Summary: Konqueror shows link to RSS feed with wrong mime type
Status: RESOLVED FIXED
Alias: None
Product: akregator
Classification: Applications
Component: akregator konqueror plugin (show other bugs)
Version: unspecified
Platform: openSUSE Linux
: NOR normal
Target Milestone: ---
Assignee: kdepim bugs
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2005-01-16 21:21 UTC by Candid Dauth
Modified: 2005-06-18 14:19 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:


Attachments
Test case which shows incorrect behavior (467 bytes, text/html)
2005-04-25 00:22 UTC, Eckhart Wörner
Details
Proposed patch (1.54 KB, patch)
2005-06-18 00:44 UTC, Eckhart Wörner
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Candid Dauth 2005-01-16 21:21:24 UTC
Version:           3.3.91 (using KDE KDE 3.3.90)
Installed from:    SuSE RPMs

Konqueror shows a link to an RSS feed if there is a <link> of the following syntax in the document:
<link rel/rev="alternate" ...>

But, this can also be a link to another document, for example to the same website in another language. And that would produce an error in aKregator. I'd propose to check if the attribute
type="application/rss+xml"
is available.

Additionally, those alternative documents can be defined by
<link rel="alternate" ...
as well as by
<link rev="alternate" ...

The «RSS feed detector» detects an RSS feed in case of both versions, but the Document Relation Toolbar doesn't. It only shows links on alternative versions with the syntax
<link rel="alternate" ...
Comment 1 Frank Osterfeld 2005-01-17 08:40:23 UTC
Actually, the code should check if the "type" attribute is either "application/rss+xml", "application/rdf+xml" or "application/atom+xml".
Could you give an example of a site where this happens?
Comment 2 Candid Dauth 2005-01-17 15:15:17 UTC
For example
http://www.phoenix-world.com/Candid/3.0.0_alpha/

I don't know if you already fixed this bug in a newer version -- my aKregator version is 1.0-beta8 under KDE 3.3.91.
Comment 3 Eckhart Wörner 2005-04-25 00:22:01 UTC
Created attachment 10783 [details]
Test case which shows incorrect behavior
Comment 4 Eckhart Wörner 2005-06-18 00:44:16 UTC
Created attachment 11496 [details]
Proposed patch

First patch ever, so be careful. ;-)
Comment 5 Frank Osterfeld 2005-06-18 14:19:18 UTC
Closed in CVS