-
-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
Blocked by CORS policy #3273
Comments
Hi, thanks for the report. I don’t see any reason we couldn’t add CORS headers. |
When will that be fixed? |
The maintainers and contributors all work on this project in their spare time, so there’s no timeline I’m afraid! This is a community project; anyone who is interested in trying to solve the problem is welcome to. The maintainers are happy to provide guidance and review code. (If you don’t want to or aren’t able to, no worries! Maybe someone else will come along and work on it.) In terms of technical advice, I’ve used the cors package before, which is the standard way to go about this. Though I’ve never done it with Scoutcamp (that’s the web framework Shields uses) and I’m not sure if there is a CORS handler built in. The relevant bits of code would probably belong in |
It may also be worth researching whether anything has to be configured in Cloudflare for CORS. |
I think all the other headers we set are passed through by CloudFlare without having to whitelist them. The same-origin policy is implemented in the browser sandbox, so we should only need to add the header. |
We should just be able to add CloudFlare, no matter what settings you set, will not interfere with CORS, and will pass it through unaltered. |
Fixed in #4171 🎉 |
Are you experiencing an issue with...
🪲 Description
When I try using the badges in a Steam addon description, I get this error and the badge doesnt show up:
🔗 Link to the badge
https://img.shields.io/github/issues-raw/Aperture-Development/MSync-2.png
Steps for reproduction:
[img]{badge}[/img]
to your addon descriptionThe text was updated successfully, but these errors were encountered: