Opened 4 years ago

Closed 4 years ago

Last modified 4 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: 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

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 Changed 4 years ago by Carl Meyer

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 Changed 4 years ago by Aymeric Augustin

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 Changed 4 years ago by Aymeric Augustin

Owner: changed from nobody to Aymeric Augustin
Status: newassigned

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

Resolution: fixed
Status: assignedclosed

comment:5 Changed 4 years ago by Aymeric Augustin

The feed now validates.

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