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 explicit 'https' to GitHub API url #353

Closed
wants to merge 1 commit into from
Closed

Add explicit 'https' to GitHub API url #353

wants to merge 1 commit into from

Conversation

evverx
Copy link
Contributor

@evverx evverx commented Feb 15, 2015

Request to 'http' redirects to 'https' with 307 status.
Some browsers (like Safari 7.1 or Firefox 35.0.1) don't follow
redirect for cross-origin requests and print error like this:

XMLHttpRequest cannot load
http://api.github.com/repos/h5bp/html5please/commits. No
'Access-Control-Allow-Origin' header is present on the requested
resource. Origin 'http://html5please.com' is therefore not allowed
access.

Request to 'http' redirects to 'https' with 307 status.
Some browsers (like Safari 7.1 or Firefox 35.0.1) don't follow
redirect for cross-origin requests and print error like this:
```
XMLHttpRequest cannot load
http://api.github.com/repos/h5bp/html5please/commits. No
'Access-Control-Allow-Origin' header is present on the requested
resource. Origin 'http://html5please.com' is therefore not allowed
access.
```
@evverx
Copy link
Contributor Author

evverx commented Feb 16, 2015

JFYI, GitHub supports JSONP. It eliminates many problems with cross-origin requests and CORS:)

@arthurvr
Copy link
Member

@evverx Ya. Thanks for the PR, will merge this evening.

@arthurvr arthurvr closed this in f820323 Feb 16, 2015
@arthurvr
Copy link
Member

Merged! Thanks @evverx!

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.

None yet

2 participants