.address() returned server ip >_<, solved with this patch and adjusted the test

3rd-Eden · 3rd-Eden · commit 65d7229079f7 · 2011-07-02T17:49:32.000+02:00
suite
diff --git a/lib/manager.js b/lib/manager.js
@@ -820,10 +820,16 @@ Manager.prototype.handshakeData = function (data) {
   var connection = data.request.connection
     , connectionAddress;
 
-  if (connection.address) {
-    connectionAddress = connection.address();
-  } else if (connection.socket && connection.socket.address) {
-    connectionAddress = connection.socket.address()
+  if (connection.remoteAddress) {
+    connectionAddress = {
+        remoteAddress: connection.remoteAddress
+      , remotePort: connection.remotePort
+    }; 
+  } else if (connection.socket && connection.socket.remoteAddress) {
+    connectionAddress = {
+        remoteAddress: connection.socket.remoteAddress
+      , remotePort: connection.socket.remotePort
+    }; 
   }
 
   return {
diff --git a/test/namespace.test.js b/test/namespace.test.js
@@ -80,8 +80,8 @@ module.exports = {
         fn(null, true);
       })
       .on('connection', function (socket) {
-        socket.handshake.address.address.should.equal('127.0.0.1');
-        socket.handshake.address.port.should.equal(ports);
+        (!!socket.handshake.address.remoteAddress).should.be.true;
+        (!!socket.handshake.address.remotePort).should.be.true;
         socket.handshake.headers.host.should.equal('localhost');
         socket.handshake.headers.connection.should.equal('keep-alive');
         socket.handshake.time.should.match(/GMT/);
diff --git a/test/transports.websocket.test.js b/test/transports.websocket.test.js
@@ -1493,8 +1493,8 @@ module.exports = {
       , ws;
 
     io.sockets.on('connection', function (socket) {
-      socket.handshake.address.address.should.equal('127.0.0.1');
-      socket.handshake.address.port.should.equal(ports);
+      (!!socket.handshake.address.remoteAddress).should.be.true;
+      (!!socket.handshake.address.remotePort).should.be.true;
       socket.handshake.headers.host.should.equal('localhost');
       socket.handshake.headers.connection.should.equal('keep-alive');
       socket.handshake.time.should.match(/GMT/);
