inconsistency between ws.WebSocket.readyState and websocket.io/hybi/WebSocket.readyState about websocket.io

[ruxkor]

tl;dr
I have not quite wrapped my head around this one, but can it be possible that the hybi Socket object emits a readyState open, although the connection is actually already closed again?

some details

In order to visualize the whole procedure, I drew a callgraph, (handleUpgrade.dot.png)[https://github.com/ruxkor/engine.io/blob/master/docs/handleUpgrade.dot.png].

A small legend:

• blue, dashed: synchronos process flow
• blue, dashed: async process flow
• green, solid: place where a function is constructed, and also called, if there is no other green, dashed arrow
• green, dashed: where the function is called
• red, dashed: assumed order if the asynchronous functions

combining the blue, solid arrow with the red, dashed arrows should present the actual flow of engine.handleUpgrade.

Back to the problem:
In ws.Websocket.establishConnection the object sets its own readyState on OPEN, after attaching the first and dataHandler to be called on process.nextTick and socket.on('data') respectively. It also attaches cleanupWebSocketResorces to its socket object. Only then it emits 'open',

Only then the w.io.hybi.WebSocket.onOpen (fn 1) will be called, which in turn attaches a function to process.nextTick, that will set the Sockets readyState on open and emit 'open'.

If a socket gets cleaned up, because of a sudden disconnect, I think the w.io.hybi.WebSocket (fn 1.1) will still be called, since it was already attached to process.nextTick.

possible solution
One possibility would be to check the readyState of the ws object inside (fn 1.1), and only set its own readyState and emit if it equals WebSocket.OPEN, similiar how the firstHandler is doing it in establishConnection.

An imo even better solution would be to implement getters and setters for the websocket.io.WebSocket readyState. In this way, it would be trivial to return th actual readyState, i.e. the one of the ws object, but still using an own attribute for the drafts.js WebSocket (since there is no ws attribute to use here).