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

Should validate the header listed as required in the hosting considerations section of the SpaceApi spec #36

Closed
eastein opened this issue Feb 1, 2013 · 5 comments
Milestone

Comments

@eastein
Copy link

eastein commented Feb 1, 2013

Access-Control-Allow-Origin: *

Perhaps the content-type should also be validated, or warned against if it's not application/json.

@eastein
Copy link
Author

eastein commented Nov 23, 2013

http://pastebin.com/raw.php?i=7DtVuFgN shows that many spaces stil ldo not have this, re-open please @slopjong

@eastein
Copy link
Author

eastein commented Nov 23, 2013

If you will not re-open @slopjong please explain why, this prevents many applications from working and also allows a violation of the spec to continue without informing the space that is in violation.

@eastein
Copy link
Author

eastein commented Nov 23, 2013

(thanks to Reventlov on irc for noticing this again)

@eastein
Copy link
Author

eastein commented Nov 23, 2013

Having talked ot @slopjong again on this, we have it warning now. I will make a new ticket for handling this in a more nuanced way.

@slopjong
Copy link

I saw too late that @eastein referenced this issue in #94.

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

No branches or pull requests

2 participants