Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add RSS feeds #990

Merged
merged 1 commit into from
Feb 29, 2016
Merged

Add RSS feeds #990

merged 1 commit into from
Feb 29, 2016

Conversation

di
Copy link
Member

@di di commented Feb 29, 2016

This fixes #794 and fixes #390 by adding duplicates of PyPI's two RSS feeds:

It also adds redirects from the old feed URLs to the new ones.

Note: If you look at these feeds in a browser such as Chrome, the XML will probably look a little more like garbage than you're used to. This is because our Content Security Policy is not allowing the browser to inline JS or CSS, but it still rewrites the HTML expecting the styles and scripts to be there. You'll see an error in the console such as:

Refused to apply inline style because it violates the following Content Security
Policy directive: "style-src 'self' fonts.googleapis.com". Either the
'unsafe-inline' keyword, a hash
('sha256-nx7T2vM3FG2GqDCK5NErQR6tT6iOS/rBjaBaUvmACC0='), or a nonce
('nonce-...') is required to enable inline execution.

@dstufft
Copy link
Member

dstufft commented Feb 29, 2016

Ok, that's kind of hilarious the the browser's own UI is being blocked by CSP.

dstufft added a commit that referenced this pull request Feb 29, 2016
@dstufft dstufft merged commit 30139ae into pypi:master Feb 29, 2016
@di di deleted the fix-794 branch February 29, 2016 22:38
@dstufft
Copy link
Member

dstufft commented Feb 29, 2016

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Work out what to do with RSS Implement Legacy RSS Feeds
2 participants