Merge pull request #17 from azuqua/fix/argsWithProtocol

fix for duplicate protocol when provided a host arg with protocol

clients/server/token-sockjs-ws.js

@@ -214,6 +214,9 @@ var TokenSocket = function(options, actions){
 
   var parsed = url.parse(options.host);
   options.protocol = parsed.protocol === "https:" ? "https:" : "http:";
+  if (parsed.protocol) {
+    options.host = parsed.host;
+  }
 
   if(!options.port){
     var parts = options.host.split(":");

test/server/unit.js

@@ -46,7 +46,15 @@ module.exports = function(TokenSocket, mocks){
 				assert.property(socketReq, "token", "Socket auth request has token property");
 				assert.equal(socketReq.token, token, "Socket auth request has correct token");
 			});
-
+
+			it("Should create a valid socket URL when passed a host argument that includes protocol", function(){
+				assert.doesNotThrow(function(){
+					var socket = new TokenSocket({ host: 'http://foo.com'	});
+					var req = socket._rest._requests.shift();
+					assert.isOk(req.options.host.indexOf('http') === -1, 'parsed host does not include protocol');
+				}, "Constructor does not create an invalid url when passed a host argument that includes protocol");
+			});
+
 			it("Should not throw an error when created with all possible arguments", function(done){
 				assert.doesNotThrow(function(){
 					var socket = new TokenSocket({
@@ -128,4 +136,4 @@ module.exports = function(TokenSocket, mocks){
 
 	});
 
-};
+};