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

Getting SSL error on request #64133

Closed
jookyboi opened this issue Dec 2, 2018 · 12 comments
Closed

Getting SSL error on request #64133

jookyboi opened this issue Dec 2, 2018 · 12 comments
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug verified Verification succeeded

Comments

@jookyboi
Copy link

jookyboi commented Dec 2, 2018

I AM THE CREATOR OF THE CACHER EXTENSION

  • VSCode Version: 1.30.0-Insider
  • OS Version: macOS 10.14.1

Steps to Reproduce:

  1. Install Cacher from: https://marketplace.visualstudio.com/items?itemName=Cacher.cacher-vscode
  2. Run command Cacher: Setup
  3. Enter API Key: 6ada540a7faaf0eb, API Token: 1d28a604ceaa202375251033d9725125

Expect:
Toast message to show "Successfully set up Cacher.".

Actual:
Request to https://api.cacher.io/vscode/validate never seems to complete.

When I try to log the error in Debug mode, I get the following error:

Error: write EPROTO 4557563328:error:14077438:SSL routines:SSL23_GET_SERVER_HELLO:tlsv1 alert internal error:../../vendor/node/deps/openssl/openssl/ssl/s23_clnt.c:802:
setup.controller.js:80
	at _errnoException (util.js:1024:11)
	at WriteWrap.afterWrite [as oncomplete] (net.js:867:14)

This error does not occur on VSCode v1.29.1.

Does this issue occur when all extensions are disabled?: No

@vscodebot vscodebot bot added the insiders label Dec 2, 2018
@rebornix
Copy link
Member

rebornix commented Dec 2, 2018

Please file issue against the extension you are using.

Update

This is the same issue as GitHub Pull Request login issue.

@rebornix rebornix added the *caused-by-extension Issue identified to be caused by an extension label Dec 2, 2018
@vscodebot
Copy link

vscodebot bot commented Dec 2, 2018

This issue is caused by an extension, please file it with the repository (or contact) the extension has linked in its overview in VS Code or the marketplace for VS Code. See also our issue reporting guidelines.

Happy Coding!

@vscodebot vscodebot bot closed this as completed Dec 2, 2018
@jookyboi
Copy link
Author

jookyboi commented Dec 3, 2018

Please re-open this issue. I am the creator of this extension and VSCode-Insider is causing for the request to fail for unknown reasons. @rebornix It doesn't make any sense for me to file an issue against my own extension.

If there is somewhere else I should be filing the issue, please let me know. I really think it is a VSCode issue as the extension has not changed between VSCode v1.29.1 and 1.30.0.

Also, if there is something I need to do as an extension developer to make this work, let me know.

@rebornix
Copy link
Member

rebornix commented Dec 3, 2018

@jookyboi sorry, I misread the error message you sent. This is the same as GitHub Pull Request, which fails when it tries to validate access token, due to the same SSL error. We are already looking into this issue.

Error: write EPROTO 4557563328:error:14077438:SSL routines:SSL23_GET_SERVER_HELLO:tlsv1 alert internal error:../../vendor/node/deps/openssl/openssl/ssl/s23_clnt.c:802:

It's similar to nodejs/node#16196 but we are not sure yet.

@rebornix rebornix reopened this Dec 3, 2018
@rebornix rebornix added the important Issue identified as high-priority label Dec 3, 2018
@rebornix rebornix self-assigned this Dec 3, 2018
@rebornix rebornix removed the *caused-by-extension Issue identified to be caused by an extension label Dec 3, 2018
@bpasero
Copy link
Member

bpasero commented Dec 3, 2018

We are building exploration builds that use a much newer version of Electron (our UI framework). I wonder if this issue reproduces with one of these builds, could you try? Download:

@jookyboi
Copy link
Author

jookyboi commented Dec 3, 2018

@bpasero Thanks for posting. I've confirmed that the build you sent for macOS does indeed fix the request SSL issue for me.

@rebornix
Copy link
Member

rebornix commented Dec 3, 2018

@jookyboi can you try if setting "http.systemProxy": "off" can fix the problem in Insiders?

@chrmarti
Copy link
Collaborator

chrmarti commented Dec 3, 2018

@bpasero Is the setting enabled in the Electron 3 build?

@jookyboi
Copy link
Author

jookyboi commented Dec 3, 2018

@rebornix Turning system proxy to off (instead of the default force) does fix the problem in Insiders.

@bpasero
Copy link
Member

bpasero commented Dec 4, 2018

@chrmarti I cannot answer your question because I do not know the answer

@chrmarti
Copy link
Collaborator

chrmarti commented Dec 4, 2018

Also with Electron 3.

@chrmarti chrmarti removed fixed-in-electron-3.0.x important Issue identified as high-priority labels Dec 4, 2018
@chrmarti chrmarti added this to the November 2018 milestone Dec 4, 2018
@chrmarti chrmarti added the bug Issue identified by VS Code Team member as probable bug label Dec 4, 2018
@chrmarti chrmarti modified the milestones: November 2018, December 2018 Dec 6, 2018
@chrmarti
Copy link
Collaborator

The affected servers are using SNI (server name identification), meaning: The client must send the server's name with the initial request, which the non-proxied code path was missing. Working on a fix.

chrmarti added a commit to microsoft/vscode-proxy-agent that referenced this issue Dec 19, 2018
@vscodebot vscodebot bot locked and limited conversation to collaborators Jan 26, 2019
@alexr00 alexr00 added the verified Verification succeeded label Feb 1, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Issue identified by VS Code Team member as probable bug verified Verification succeeded
Projects
None yet
Development

No branches or pull requests

5 participants