Opened 3 years ago

Closed 3 years ago

Last modified 3 years ago

#19909 closed Bug (fixed)

Django RSS feed not validate on

Reported by: julyzergcn1@… Owned by: aaugustin
Component: * Version: master
Severity: Normal Keywords: RSS invalid format
Cc: Triage Stage: Accepted
Has patch: no Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no


Django RSS feed not validate on
For example, I test this feed
The result is at

It says:
This feed does not validate.

Change History (5)

comment:1 Changed 3 years ago by carljm

  • Needs documentation unset
  • Needs tests unset
  • Patch needs improvement unset
  • Resolution set to invalid
  • Status changed from new to closed

There are only two actual validation errors there: one is because the feed self-references at an "http" URL and you are validating it at an "https" URL - if you read the help for that issue, that's not actually a problem as long as the two URLs reference the same document, which they do.

The second is an issue with a "guid" tag whose contents are not a URL, that does not have the isPermaLink="false" attribute. I'm not sure, but I think since this feed is an aggregator we are taking that guid straight from the aggregated blog. It will be hard to avoid issues like this with an aggregator.

The rest are warnings that mostly seem to do with the validator not liking certain tags used in entity-escaped HTML in some entries. Again, this is not something we'll be able to control in an aggregated feed.

This bug was tagged as "contrib.syndication", but none of the above issues reflect an actual problem with code in "contrib.syndication"

comment:2 Changed 3 years ago by aaugustin

  • Component changed from contrib.syndication to *
  • Resolution invalid deleted
  • Status changed from closed to new
  • Triage Stage changed from Unreviewed to Accepted
  • Version changed from 1.4 to master

I'm going to have a look at the http link, everything should be https.

Indeed the other warnings don't appear to be under our control.

comment:3 Changed 3 years ago by aaugustin

  • Owner changed from nobody to aaugustin
  • Status changed from new to assigned

comment:4 Changed 3 years ago by Aymeric Augustin <aymeric.augustin@…>

  • Resolution set to fixed
  • Status changed from assigned to closed

comment:5 Changed 3 years ago by aaugustin

The feed now validates.

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