Opened 12 years ago

Closed 12 years ago

Last modified 12 years ago

#19909 closed Bug (fixed)

Django RSS feed not validate on http://validator.w3.org

Reported by: julyzergcn1@… Owned by: Aymeric Augustin
Component: *.djangoproject.com Version: dev
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

Description

Django RSS feed not validate on http://validator.w3.org.
For example, I test this feed https://www.djangoproject.com/rss/community/blogs/
The result is at http://validator.w3.org/feed/check.cgi?url=https%3A%2F%2Fwww.djangoproject.com%2Frss%2Fcommunity%2Fblogs%2F

It says:
Sorry
This feed does not validate.

Change History (5)

comment:1 by Carl Meyer, 12 years ago

Resolution: invalid
Status: newclosed

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 by Aymeric Augustin, 12 years ago

Component: contrib.syndication*.djangoproject.com
Resolution: invalid
Status: closednew
Triage Stage: UnreviewedAccepted
Version: 1.4master

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 by Aymeric Augustin, 12 years ago

Owner: changed from nobody to Aymeric Augustin
Status: newassigned

comment:4 by Aymeric Augustin <aymeric.augustin@…>, 12 years ago

Resolution: fixed
Status: assignedclosed

comment:5 by Aymeric Augustin, 12 years ago

The feed now validates.

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