Permalink
Browse files

SSL_OP_ALL causes incorrect parameter error due to Node bug

  • Loading branch information...
smfreegard committed Dec 5, 2012
1 parent 19f5bd6 commit 082be646ebf65f53946908080596e4cf0f05f0b7
Showing with 4 additions and 2 deletions.
  1. +4 −2 tls_socket.js
View
@@ -172,7 +172,8 @@ function createServer(cb) {
// Set SSL_OP_ALL for maximum compatibility with broken clients
// See http://www.openssl.org/docs/ssl/SSL_CTX_set_options.html
if (!options) options = {};
- options.secureOptions = SSL_OP_ALL;
+ // TODO: bug in Node means we can't do this until it's fixed
+ // options.secureOptions = SSL_OP_ALL;
var sslcontext = crypto.createCredentials(options);
@@ -229,7 +230,8 @@ function connect(port, host, cb) {
// Set SSL_OP_ALL for maximum compatibility with broken servers
// See http://www.openssl.org/docs/ssl/SSL_CTX_set_options.html
if (!options) options = {};
- options.secureOptions = SSL_OP_ALL;
+ // TODO: bug in Node means we can't do this until it's fixed
+ // options.secureOptions = SSL_OP_ALL;
var sslcontext = crypto.createCredentials(options);

0 comments on commit 082be64

Please sign in to comment.