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

Access to media.readthedocs.org/fonts/fontawesome-webfont.woff2 blocked by CORS policy #3586

Closed
aaksarin opened this issue Feb 8, 2018 · 11 comments
Labels
Bug A bug Good First Issue Good for new contributors

Comments

@aaksarin
Copy link

aaksarin commented Feb 8, 2018

Details

Expected Result

Page loaded without errors.

Actual Result

(index):1 Access to Font at 'https://media.readthedocs.org/fonts/fontawesome-webfont.woff2?v=4.6.3' from origin 'http://django-compressor.readthedocs.io' has been blocked by CORS policy: No 'Access-Control-Allow-Origin' header is present on the requested resource. Origin 'http://django-compressor.readthedocs.io' is therefore not allowed access. The response had HTTP status code 404.

@davidfischer
Copy link
Contributor

I think the issue is here because the URL https://media.readthedocs.org/fonts/fontawesome-webfont.woff2?v=4.6.3 404s. This shouldn't actually cause any page differences as https://media.readthedocs.org/fonts/fontawesome-webfont.woff is also loaded and loads without issue.

I believe there is a CORS issue because when a successful 200 is returned the Access-Control-Allow-Origin header is added. However, because there's a 404, that header isn't there. I think simply uploading the woff2 file to the media server will resolve this.

@aaksarin
Copy link
Author

aaksarin commented Feb 9, 2018

@davidfischer I agree. Thanks.

Now someone need to put woff2 file from this release https://github.com/FortAwesome/Font-Awesome/releases/tag/v4.6.3 to this path https://media.readthedocs.org/fonts/.

@RichardLitt RichardLitt added Bug A bug Good First Issue Good for new contributors labels Feb 9, 2018
@Ayukha
Copy link

Ayukha commented Feb 12, 2018

@aaksarin Can i try?

@aaksarin
Copy link
Author

@Ayukha, yeah, fill free to fix it.

@fiennyangeln
Copy link

@aaksarin Hi, a newcomer here. I have downloaded the fontawesome file and put the woff2 file. Please check if my way of doing things are correct. Thanks a lot

@aaksarin
Copy link
Author

@fiennyangeln, Hi. I don't know if this is correct, i'm just posted issue and supposed fix. @RichardLitt, can you look please?

@RichardLitt
Copy link
Member

@fiennyangeln Looks like a good start! Comments will happen on that PR; when it is merged, this will be fixed. :)

@fenilgandhi
Copy link
Contributor

fenilgandhi commented Feb 18, 2018

Is this issue still open ? Or can i fix it ?

@stsewd
Copy link
Member

stsewd commented Feb 18, 2018

@fenilgandhi there is a PR #3602 already.

@fenilgandhi
Copy link
Contributor

Ok Thanks

@agjohnson
Copy link
Contributor

I believe we fixed this with a previous release. I don't see the behavior on the reported doc site anymore, so safe to close I think.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug A bug Good First Issue Good for new contributors
Projects
None yet
Development

Successfully merging a pull request may close this issue.

8 participants