Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

latest socket.io update throws node error #88

Open
inspire22 opened this issue Jul 22, 2011 · 2 comments
Open

latest socket.io update throws node error #88

inspire22 opened this issue Jul 22, 2011 · 2 comments

Comments

@inspire22
Copy link

debug - set close timeout for client 8742131551014762006
warn - client not handshaken client should reconnect
info - transport end
debug - cleared close timeout for client 8742131551014762006
debug - discarding transport
debug - setting request GET /socket.io/1/xhr-polling/472935549180658144?t1311297173968
debug - setting poll timeout
debug - clearing poll timeout
debug - xhr-polling writing 7:::1+0
debug - set close timeout for client 472935549180658144
warn - client not handshaken client should reconnect
info - transport end
debug - cleared close timeout for client 472935549180658144
debug - discarding transport
debug - xhr-polling received data packet �69�3:::{"type":"subscribe","channel":"ce750cb608619b9d14d50"}�3�0::
21 Jul 20:12:56 - Received: {"type":"subscribe","channel":"ce750cb608619b9d14d50"}
21 Jul 20:12:56 - Client subscribing to: ce750cb608619b9d14d50
debug - xhr-polling received data packet �69�3:::{"type":"subscribe","channel":"ce750cb608619b9d14d50"}�3�0::
debug - got disconnection packet
info - transport end by forced client disconnection
info - transport end
debug - cleared close timeout for client 17337047622115684384
debug - discarding transport

/data/u/apps/newap/shared/node_modules/juggernaut/node_modules/socket.io/lib/manager.js:466
this.rooms[room].splice(this.rooms[room].indexOf(id), 1);
^
TypeError: Cannot call method 'indexOf' of undefined
at Manager.onDisconnect (/data/u/apps/newap/shared/node_modules/juggernaut/node_modules/socket.io/lib/manager.js:466:48)
at Manager.onClientDisconnect (/data/u/apps/newap/shared/node_modules/juggernaut/node_modules/socket.io/lib/manager.js:428:8)
at XHRPolling. (/data/u/apps/newap/shared/node_modules/juggernaut/node_modules/socket.io/lib/transport.js:467:20)
at XHRPolling.end (/data/u/apps/newap/shared/node_modules/juggernaut/node_modules/socket.io/lib/transports/http-polling.js:133:38)
at XHRPolling.onForcedDisconnect (/data/u/apps/newap/shared/node_modules/juggernaut/node_modules/socket.io/lib/transport.js:208:10)
at XHRPolling.onMessage (/data/u/apps/newap/shared/node_modules/juggernaut/node_modules/socket.io/lib/transport.js:360:17)
at XHRPolling.onData (/data/u/apps/newap/shared/node_modules/juggernaut/node_modules/socket.io/lib/transports/http.js:89:10)
at IncomingMessage. (/data/u/apps/newap/shared/node_modules/juggernaut/node_modules/socket.io/lib/transports/http.js:57:12)
at IncomingMessage.emit (events.js:61:17)
at HTTPParser.onMessageComplete (http.js:133:23)

@mreinsch
Copy link

mreinsch commented Aug 6, 2011

getting the same error when a client disconnects. Seems pretty reproducible.

@mreinsch
Copy link

mreinsch commented Aug 6, 2011

updating to socket.io master fixes this issue, so it should be fixed once 0.7.8 is out.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants