Garbage collection

[3rd-Eden]

Garbage collection function to kill unused handshakes.

[3rd-Eden]

https://gist.github.com/1100205

Benchmark test, I didn't think something like would be suitable for our unit tests, so we might needs need have some sort of performance benchmark suite where these kind of tests can run.

[rauchg]

Not merging this one as I still want to make some changes to the handshake store logic to make garbage collection more flexible.

[3rd-Eden]

Sure, but this is a major memory leak and can leak hunderds om kb in a production env.

So i suggesting fixen it now instead of after another release as ALLOT of users are suffering from this bug.

~ Arnout Kazemier

On Jul 31, 2011, at 3:51 AM, [email protected] wrote:

Not merging this one as I still want to make some changes to the handshake store logic to make garbage collection more flexible.

Reply to this email directly or view it on GitHub:
#408 (comment)

[scanales]

Hi, I'm getting this error

TypeError: Cannot use 'in' operator to search for 'issued' in null
at String.IN (native)
at Manager.garbageCollection (/node_modules/socket.io/lib/manager.js:1021:21)
at Timer. (native)
at Timer.ontimeout (timers.js:223:14)

Any ideas why this is happening?

Thanks,

[vikram-chaitanya]

I am also getting this error. I temporarily put a fix by forking and checking for null value of handshake. Any ideas on what would have caused this error?

Thanks,
Vikram

[gkorland]

@scanales / @vikramnet2003 - did you find a solution?

[adrian-ludwig]

In my system i have about 6 different system that have connected the same Redis Store for Socket.io.
This error happens only when we restart one of the systems that have connected clients.

[nguyenvansutt]

I am also getting this error when using socket.io-clusterhub. did anyone find out a solutions?