Skip to content
This repository has been archived by the owner on Nov 27, 2020. It is now read-only.

Don't use https as remote debugger scheme. #861

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

andrewhavens
Copy link

When using https, the remote debugger page never loads. When changing the scheme to http, it loads correctly.

When using https, the remote debugger page never loads. When changing the scheme to http, it loads correctly.
@twalpole
Copy link
Contributor

twalpole commented Feb 24, 2017

The allowing of https here was done specifically to fix another issue IIRC - #579 - @route any feedback on this?

@andrewhavens
Copy link
Author

@twalpole from what I read in that issue you linked, it had more to do with prefixing the URL with http. Making https the default was not the real issue.

@route
Copy link
Contributor

route commented Apr 4, 2017

If I remember events correct we used hard coded http:// at the beginning, but then switched to just // which supposed to pick the protocol automatically and this introduced a bug.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants