Permalink
Browse files

Fix option parsing in tls.connect()

  • Loading branch information...
1 parent cb06abe commit df5d5d63422b3624ae3c19c8d4424deacfcd9025 @ry ry committed Jan 28, 2011
Showing with 14 additions and 23 deletions.
  1. +14 −23 lib/tls.js
View
@@ -650,31 +650,22 @@ Server.prototype.setOptions = function(options) {
exports.connect = function(port /* host, options, cb */) {
// parse args
var host, options = {}, cb;
- switch (typeof arguments[1]) {
- case 'string':
- host = arguments[1];
- if (typeof arguments[2] == 'object') {
- options = arguments[2];
- if (typeof arguments[3] == 'function') cb = arguments[3];
- } else if (typeof arguments[2] == 'function') {
- cb = arguments[2];
- }
- break;
-
- case 'object':
- options = arguments[1];
- if (typeof arguments[2] == 'function') cb = arguments[2];
- break;
-
- case 'function':
- cb = arguments[1];
- break;
-
- default:
- break;
+ for (var i = 1; i < arguments.length; i++) {
+ switch (typeof arguments[i]) {
+ case 'string':
+ host = arguments[i];
+ break;
+
+ case 'object':
+ options = arguments[i];
+ break;
+
+ case 'function':
+ cb = arguments[i];
+ break;
+ }
}
-
var socket = new net.Stream();
var sslcontext = crypto.createCredentials(options);

0 comments on commit df5d5d6

Please sign in to comment.