Skip to content

Conversation

@kasperg
Copy link
Contributor

@kasperg kasperg commented May 22, 2013

Use a protocol-less URL to call service which determines accept language. Thus we avoid any security warnings when this plugin is used on a page served using HTTPS.

Use a protocol-less URL to call service which determines accept language. Thus we avoid any security warnings when this plugin is used on a page served using HTTPS.
@hueitan
Copy link

hueitan commented Sep 9, 2013

ya you are right, better using // than http

@dansingerman
Copy link
Owner

Happy to merge this if it is pointed at ajaxhttpheaders.appspot.com (which acts as a loadbalancer for ajaxhttpheaders1.appspot.com - ajaxhttpheaders4.appspot.com)

@kasperg
Copy link
Contributor Author

kasperg commented Feb 8, 2016

@dansingerman: I have updated the PR to refer to ajaxhttpheaders.appspot.com as requested.

Are you sure that the load balancer is setup correctly? If I send a request to https://ajaxhttpheaders.appspot.com I get a 302 redirect to a non-HTTPS version.

5 bash 2016-02-08 17-19-46

If I send a request to https://ajaxhttpheaders2.appspot.com I get a 200 OK response

5 bash 2016-02-08 17-23-38

dansingerman added a commit that referenced this pull request Feb 9, 2016
@dansingerman dansingerman merged commit 69f0e36 into dansingerman:master Feb 9, 2016
@dansingerman
Copy link
Owner

@kasperg yes you're right, it should be fixed now :)

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.

3 participants