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

https.get, TLS, *node*, *async*, and perhaps others and EPROTO type 40 error #18688

Closed
Martii opened this issue Feb 9, 2018 · 5 comments
Closed
Labels
https Issues or PRs related to the https subsystem. invalid Issues and PRs that are invalid. question Issues that look for answers.

Comments

@Martii
Copy link

Martii commented Feb 9, 2018

Trying the direct approach since I got maintainer having a bad day over at nodejs/help#1098

  • 8.9.4 and 9.5.0:
  • Linux (multiple distros):
  • Scope (install, code, runtime, meta, other?):
  • Module https://www.npmjs.com/package/request (and version of 2.83.0) (relevant when converted from node internal https.get and produces similar output ):

Basically the .on('error', ... isn't happening when using https.request, http.get, and request from the url at https://confluence.bredex.de/images/icons/profilepics/default.png

Clearly the server is misconfigured but it trips our production and development environment.

What am I doing incorrectly? This routine works for properly configured servers just not that server.

The simple request at https://github.com/OpenUserJS/OpenUserJS.org/pull/1324/files#diff-b755ddbc6b2edf964c5586d742e3afdaL1459
The data stream at https://github.com/OpenUserJS/OpenUserJS.org/pull/1324/files#diff-b755ddbc6b2edf964c5586d742e3afdaL1461
The end stream at https://github.com/OpenUserJS/OpenUserJS.org/pull/1324/files#diff-b755ddbc6b2edf964c5586d742e3afdaL1468
The error trap at https://github.com/OpenUserJS/OpenUserJS.org/pull/1324/files#diff-b755ddbc6b2edf964c5586d742e3afdaL1488

All of this is wrapped in the async package.

Thanks so much.

@Martii
Copy link
Author

Martii commented Feb 9, 2018

I've been given some advice on rechecking something... please hold while I retest.

@bnoordhuis
Copy link
Member

And also given advice on posting a minimal test case, remember? The contributing guide has more hints if you need them.

I'm closing this out because we already established the issue wasn't with node.js itself.

@bnoordhuis bnoordhuis added invalid Issues and PRs that are invalid. https Issues or PRs related to the https subsystem. labels Feb 9, 2018
@Martii
Copy link
Author

Martii commented Feb 10, 2018

I am a contributor already @bnoordhuis (see here) . Please reopen this and refrain from being snarky. You have managed to upset me by your negative reactions now. You need to let someone else handle this and not you. Thank you for your cooperation.

@ChALkeR ChALkeR added the question Issues that look for answers. label Feb 10, 2018
@jasnell
Copy link
Member

jasnell commented Feb 10, 2018

@Martii ... I've reopened the original issue in the help repo since your issue is still unresolved. If this does turn out to be some kind of bug in core, please come back and open an issue here. Definitely hope you manage to get the issue resolved :-)

@Martii
Copy link
Author

Martii commented Feb 10, 2018

@jasnell
Thank you... I'm really confused why those two test cases work but buried in async it doesn't appear to trap it. Will continue over there.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
https Issues or PRs related to the https subsystem. invalid Issues and PRs that are invalid. question Issues that look for answers.
Projects
None yet
Development

No branches or pull requests

4 participants