Opened 12 years ago

Closed 11 years ago

#18623 closed Bug (invalid)

RSS feeds specify Atom namespace

Reported by: djangoproject@… 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 Aymeric Augustin, 12 years ago

Triage Stage: UnreviewedAccepted

The offending line was added in c4c27d8a04c9125cfbc5c3611557d8e5d3845b0d.

It was a large commit; this small regression probably went unnoticed.

comment:2 by anonymous, 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 namn, 12 years ago

Keywords: rss atom added
Triage Stage: AcceptedDesign 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 Ben Firshman, 11 years ago

(Author of that commit here!) Yep - this should probably be marked as invalid. It is intentional for the reason namn pointed out.

comment:5 by Aymeric Augustin, 11 years ago

Resolution: invalid
Status: newclosed

Thanks for your feedback!

Note: See TracTickets for help on using tickets.
Back to Top