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

How to use" secureProtocol: 'SSLv3_method'" #530

Closed
umutm opened this issue Apr 24, 2013 · 7 comments
Closed

How to use" secureProtocol: 'SSLv3_method'" #530

umutm opened this issue Apr 24, 2013 · 7 comments

Comments

@umutm
Copy link

umutm commented Apr 24, 2013

For some HTTPS requests, I'm getting an error:


7068:error:140773F2:SSL routines:SSL23_GET_SERVER_HELLO:sslv3 alert unexpected message:openssl\ssl\s23_clnt.c:658:


which is told to be fixed by using "secureProtocol: 'SSLv3_method'" option in the request (for node https) (via: http://stackoverflow.com/questions/11091974/ssl-error-in-nodejs).

Is it possible to use that option in node-request or customize which SSL connection version to use?

@smtlaissezfaire
Copy link

+1

Also reported in the node bug tracker here:

nodejs/node-v0.x-archive#5360

Shouldn't there be a good way to generically fix this (allowing a set of override options into request)?

@smtlaissezfaire
Copy link

Actually, nm. I guess that stack overflow ticket points out that adding this will work:

var https = require('https');
https.globalAgent.options.secureProtocol = 'SSLv3_method';

I think you can close this one out.

@umutm
Copy link
Author

umutm commented May 8, 2013

@smtlaissezfaire,

It somehow didn't work that way for me but adding that to the node-request's options worked fine. Thanks.

@richmarr
Copy link

Given the recent SSLv3 vulnerability (CVE-2014-3566) it looks like a bad idea to set a global secureProtocol of SSLv3_method for non-trivial apps.

@calidion
Copy link

what is the replacement for SSLv3_method now?

@umutm
Copy link
Author

umutm commented Dec 23, 2015

Per request, you can define:
secureProtocol: "TLSv1_method"

or just not define it and, if I'm not wrong, it is again TLS.

@calidion
Copy link

just tried 'TLSv1_method', and it works.
thanks.

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

No branches or pull requests

4 participants