handshakeData.address is undefined, "version": "0.9.16"

[gruntfuddler]

In the callback for io.set("authorization", fn(handshakeData, callback){...});
The handshakeData.address is undefined in some logged connection requests.

In Manager.prototype.handshakeData, should there be a check for data.request.socket.address?

var connection = data.request.connection,
socket = data.request.socket

...
else if (socket && socket.remoteAddress) {
connectionAddress = {
address: socket.remoteAddress
, port: socket.remotePort
};
}

[3rd-Eden]

In Primus, which wraps socket.io i use: https://github.com/primus/primus/blob/master/forwarded.js to extract IP address information. Might be useful for you as well.

[gruntfuddler]

What the current handshakeData prototype is doing, what I am asking is whether there is another source of the address, as per my previous post, it seems that yours is doing what I suggest.

Manager.prototype.handshakeData = function (data) {
var connection = data.request.connection
, connectionAddress
, date = new Date;

if (connection.remoteAddress) {
connectionAddress = {
address: connection.remoteAddress
, port: connection.remotePort
};
} else if (connection.socket && connection.socket.remoteAddress) {
connectionAddress = {
address: connection.socket.remoteAddress
, port: connection.socket.remotePort
};
}
return {
headers: data.headers
, address: connectionAddress
, time: date.toString()
, query: data.query
, url: data.request.url
, xdomain: !!data.request.headers.origin
, secure: data.request.connection.secure
, issued: +date
};
};