# HG changeset patch # User Matthew Wild # Date 1254515056 -3600 # Node ID 388b125b784a5485effcb49d3e17fed2004eb2b7 # Parent 115f274dd17fb953ff415792d782cf61c4d741fd# Parent b9389286eecee81a79ca0e9a139fc97f8d923738 Merge with 0.5 diff -r 115f274dd17f -r 388b125b784a net/server.lua --- a/net/server.lua Fri Oct 02 17:56:24 2009 +0500 +++ b/net/server.lua Fri Oct 02 21:24:16 2009 +0100 @@ -720,7 +720,7 @@ removeserver = function( port ) local handler = _server[ port ] if not handler then - return nil, "no server found on port '" .. tostring( port ) "'" + return nil, "no server found on port '" .. tostring( port ) .. "'" end handler.close( ) _server[ port ] = nil diff -r 115f274dd17f -r 388b125b784a plugins/mod_bosh.lua --- a/plugins/mod_bosh.lua Fri Oct 02 17:56:24 2009 +0500 +++ b/plugins/mod_bosh.lua Fri Oct 02 21:24:16 2009 +0100 @@ -258,6 +258,7 @@ end end +local dead_sessions = {}; function on_timer() -- log("debug", "Checking for requests soon to timeout..."); -- Identify requests timing out within the next few seconds @@ -274,18 +275,26 @@ end now = now - 3; + local n_dead_sessions = 0; for session, inactive_since in pairs(inactive_sessions) do if session.bosh_max_inactive then if now - inactive_since > session.bosh_max_inactive then (session.log or log)("debug", "BOSH client inactive too long, destroying session at %d", now); sessions[session.sid] = nil; inactive_sessions[session] = nil; - sm_destroy_session(session, "BOSH client silent for over "..session.bosh_max_inactive.." seconds"); + n_dead_sessions = n_dead_sessions + 1; + dead_sessions[n_dead_sessions] = session; end else inactive_sessions[session] = nil; end end + + for i=1,n_dead_sessions do + local session = dead_sessions[i]; + dead_sessions[i] = nil; + sm_destroy_session(session, "BOSH client silent for over "..session.bosh_max_inactive.." seconds"); + end end local ports = module:get_option("bosh_ports") or { 5280 };