Opened 12 years ago
Closed 12 years ago
#18623 closed Bug (invalid)
RSS feeds specify Atom namespace
Reported by: | Owned by: | nobody | |
---|---|---|---|
Component: | contrib.syndication | Version: | 1.4 |
Severity: | Normal | Keywords: | rss atom |
Cc: | Triage Stage: | Design decision needed | |
Has patch: | no | Needs documentation: | no |
Needs tests: | no | Patch needs improvement: | no |
Easy pickings: | yes | UI/UX: | no |
Description
All feeds generated using django.util.feedgenerator specify the Atom namespace in the root node, including Rss201rev2Feed and RssUserland091Feed. For example, when feed_type is specified as Rss201rev2Feed, django outputs as its root node: <rss xmlns:atom="http://www.w3.org/2005/Atom" version="2.0">
Change History (5)
comment:1 by , 12 years ago
Triage Stage: | Unreviewed → Accepted |
---|
comment:2 by , 12 years ago
Line 229 also uses atom namespace for item links in the base RssFeed class: https://github.com/django/django/blob/master/django/utils/feedgenerator.py#L229
comment:3 by , 12 years ago
Keywords: | rss atom added |
---|---|
Triage Stage: | Accepted → Design decision needed |
There is a note in 1.2 release notes (in that same commit) that says atom:link
is now a best practice in RSS.
In accordance with `RSS best practices`_, RSS feeds will now include an ``atom:link`` element. You may need to update your tests to take this into account.
The original document at http://www.rssboard.org/rss-profile says:
The Atom syndication format, which serves a similar purpose to RSS, offers some elements closely comparable to RSS elements and others that provide new capabilities. Any of these elements can be used in RSS by employing Atom as a namespace.
So, I guess having atom
namespace in RSS feeds is okay because (quoted from the same document):
The RSS specification encourages the extension of the format through the use of namespaces.
Should this ticket be marked invalid?
comment:4 by , 12 years ago
(Author of that commit here!) Yep - this should probably be marked as invalid. It is intentional for the reason namn pointed out.
The offending line was added in c4c27d8a04c9125cfbc5c3611557d8e5d3845b0d.
It was a large commit; this small regression probably went unnoticed.