socketio / socket.io

Realtime application framework (Node.JS server)
https://socket.io
MIT License
61.09k stars 10.11k forks source link

Client transport getter #177

Closed ioRekz closed 13 years ago

ioRekz commented 13 years ago

Would be nice to have some sort of client.transport to know which transport this client is currently using !

dcaravana commented 13 years ago

Definitely! Currently using "instanceof" of client.

3rd-Eden commented 13 years ago

@guille this is probably an easy fix with dvv's patch as it exposes the data object from the managers checkRequest method. Which contains the protocol version, transport, and session id + more

3rd-Eden commented 13 years ago

Fixed with pull req: https://github.com/LearnBoost/Socket.IO-node/pull/286

which exposes socket.data that contains a transport property