mod_bosh: pcall() core_process_stanza per stanza, to bring in line with other listene...
[prosody.git] / plugins / mod_bosh.lua
index 14445293341303447b3087eedb8eb7ec9a58baf5..d9c8defd90ead6d891c5948ede45d90e04ce98a0 100644 (file)
@@ -18,6 +18,10 @@ local core_process_stanza = prosody.core_process_stanza;
 local st = require "util.stanza";
 local logger = require "util.logger";
 local log = logger.init("mod_bosh");
+local initialize_filters = require "util.filters".initialize;
+local math_min = math.min;
+local xpcall, tostring, type = xpcall, tostring, type;
+local traceback = debug.traceback;
 
 local xmlns_streams = "http://etherx.jabber.org/streams";
 local xmlns_xmpp_streams = "urn:ietf:params:xml:ns:xmpp-streams";
@@ -30,6 +34,7 @@ local BOSH_DEFAULT_HOLD = module:get_option_number("bosh_default_hold", 1);
 local BOSH_DEFAULT_INACTIVITY = module:get_option_number("bosh_max_inactivity", 60);
 local BOSH_DEFAULT_POLLING = module:get_option_number("bosh_max_polling", 5);
 local BOSH_DEFAULT_REQUESTS = module:get_option_number("bosh_max_requests", 2);
+local bosh_max_wait = module:get_option_number("bosh_max_wait", 120);
 
 local consider_bosh_secure = module:get_option_boolean("consider_bosh_secure");
 
@@ -70,8 +75,8 @@ end
 local t_insert, t_remove, t_concat = table.insert, table.remove, table.concat;
 local os_time = os.time;
 
-local sessions = {};
-local inactive_sessions = {}; -- Sessions which have no open requests
+-- All sessions, and sessions that have no requests open
+local sessions, inactive_sessions = module:shared("sessions", "inactive_sessions");
 
 -- Used to respond to idle sessions (those with waiting requests)
 local waiting_requests = {};
@@ -242,18 +247,24 @@ function stream_callbacks.streamopened(context, attr)
                -- New session
                sid = new_uuid();
                local session = {
-                       type = "c2s_unauthed", conn = {}, sid = sid, rid = tonumber(attr.rid), host = attr.to,
-                       bosh_version = attr.ver, bosh_wait = attr.wait, streamid = sid,
+                       type = "c2s_unauthed", conn = {}, sid = sid, rid = tonumber(attr.rid)-1, host = attr.to,
+                       bosh_version = attr.ver, bosh_wait = math_min(attr.wait, bosh_max_wait), streamid = sid,
                        bosh_hold = BOSH_DEFAULT_HOLD, bosh_max_inactive = BOSH_DEFAULT_INACTIVITY,
                        requests = { }, send_buffer = {}, reset_stream = bosh_reset_stream,
-                       close = bosh_close_stream, dispatch_stanza = core_process_stanza,
+                       close = bosh_close_stream, dispatch_stanza = core_process_stanza, notopen = true,
                        log = logger.init("bosh"..sid), secure = consider_bosh_secure or request.secure,
                        ip = get_ip_from_request(request);
                };
                sessions[sid] = session;
                
+               local filter = initialize_filters(session);
+               
                session.log("debug", "BOSH session created for request from %s", session.ip);
                log("info", "New BOSH session, assigned it sid '%s'", sid);
+
+               -- Send creation response
+               local creating_session = true;
+
                local r = session.requests;
                function session.send(s)
                        -- We need to ensure that outgoing stanzas have the jabber:client xmlns
@@ -261,50 +272,39 @@ function stream_callbacks.streamopened(context, attr)
                                s = st.clone(s);
                                s.attr.xmlns = "jabber:client";
                        end
+                       s = filter("stanzas/out", s);
                        --log("debug", "Sending BOSH data: %s", tostring(s));
+                       t_insert(session.send_buffer, tostring(s));
+
                        local oldest_request = r[1];
-                       if oldest_request then
+                       if oldest_request and not session.bosh_processing then
                                log("debug", "We have an open request, so sending on that");
                                oldest_request.headers = default_headers;
-                               oldest_request:send(t_concat({
-                                       "<body xmlns='http://jabber.org/protocol/httpbind' ",
-                                       session.bosh_terminate and "type='terminate' " or "",
-                                       "sid='", sid, "' xmlns:stream = 'http://etherx.jabber.org/streams'>",
-                                       tostring(s),
-                                       "</body>"
-                               }));
-                       elseif s ~= "" then
-                               log("debug", "Saved to send buffer because there are %d open requests", #r);
-                               -- Hmm, no requests are open :(
-                               t_insert(session.send_buffer, tostring(s));
-                               log("debug", "There are now %d things in the send_buffer", #session.send_buffer);
+                               local body_attr = { xmlns = "http://jabber.org/protocol/httpbind",
+                                       ["xmlns:stream"] = "http://etherx.jabber.org/streams";
+                                       type = session.bosh_terminate and "terminate" or nil;
+                                       sid = sid;
+                               };
+                               if creating_session then
+                                       creating_session = nil;
+                                       body_attr.inactivity = tostring(BOSH_DEFAULT_INACTIVITY);
+                                       body_attr.polling = tostring(BOSH_DEFAULT_POLLING);
+                                       body_attr.requests = tostring(BOSH_DEFAULT_REQUESTS);
+                                       body_attr.wait = tostring(session.bosh_wait);
+                                       body_attr.hold = tostring(session.bosh_hold);
+                                       body_attr.authid = sid;
+                                       body_attr.secure = "true";
+                                       body_attr.ver  = '1.6';
+                                       body_attr.from = session.host;
+                                       body_attr["xmlns:xmpp"] = "urn:xmpp:xbosh";
+                                       body_attr["xmpp:version"] = "1.0";
+                               end
+                               oldest_request:send(st.stanza("body", body_attr):top_tag()..t_concat(session.send_buffer).."</body>");
+                               session.send_buffer = {};
                        end
                        return true;
                end
-               
-               -- Send creation response
-               
-               local features = st.stanza("stream:features");
-               hosts[session.host].events.fire_event("stream-features", { origin = session, features = features });
-               fire_event("stream-features", session, features);
-               --xmpp:version='1.0' xmlns:xmpp='urn:xmpp:xbosh'
-               local body = st.stanza("body", { xmlns = xmlns_bosh,
-                       wait = attr.wait,
-                       inactivity = tostring(BOSH_DEFAULT_INACTIVITY),
-                       polling = tostring(BOSH_DEFAULT_POLLING),
-                       requests = tostring(BOSH_DEFAULT_REQUESTS),
-                       hold = tostring(session.bosh_hold),
-                       sid = sid, authid = sid,
-                       ver  = '1.6', from = session.host,
-                       secure = 'true', ["xmpp:version"] = "1.0",
-                       ["xmlns:xmpp"] = "urn:xmpp:xbosh",
-                       ["xmlns:stream"] = "http://etherx.jabber.org/streams"
-               }):add_child(features);
-               response.headers = default_headers;
-               response:send(tostring(body));
-               
                request.sid = sid;
-               return;
        end
        
        local session = sessions[sid];
@@ -324,7 +324,7 @@ function stream_callbacks.streamopened(context, attr)
                        session.log("warn", "rid too large (means a request was lost). Last rid: %d New rid: %s", session.rid, attr.rid);
                elseif diff <= 0 then
                        -- Repeated, ignore
-                       session.log("debug", "rid repeated (on request %s), ignoring: %s (diff %d)", request.id, session.rid, diff);
+                       session.log("debug", "rid repeated, ignoring: %s (diff %d)", session.rid, diff);
                        context.notopen = nil;
                        context.ignore = true;
                        context.sid = sid;
@@ -343,16 +343,18 @@ function stream_callbacks.streamopened(context, attr)
        context.notopen = nil; -- Signals that we accept this opening tag
        t_insert(session.requests, response);
        context.sid = sid;
+       session.bosh_processing = true; -- Used to suppress replies until processing of this request is done
 
        if session.notopen then
                local features = st.stanza("stream:features");
                hosts[session.host].events.fire_event("stream-features", { origin = session, features = features });
                fire_event("stream-features", session, features);
-               session.send(features);
+               session.send(tostring(features));
                session.notopen = nil;
        end
 end
 
+local function handleerr(err) log("error", "Traceback[bosh]: %s", traceback(tostring(err), 2)); end
 function stream_callbacks.handlestanza(context, stanza)
        if context.ignore then return; end
        log("debug", "BOSH stanza received: %s\n", stanza:top_tag());
@@ -361,7 +363,20 @@ function stream_callbacks.handlestanza(context, stanza)
                if stanza.attr.xmlns == xmlns_bosh then
                        stanza.attr.xmlns = nil;
                end
-               core_process_stanza(session, stanza);
+               stanza = session.filter("stanzas/in", stanza);
+               if stanza then
+                       return xpcall(function () return core_process_stanza(session, stanza) end, handleerr);
+               end
+       end
+end
+
+function stream_callbacks.streamclosed(request)
+       local session = sessions[request.sid];
+       if session then
+               session.bosh_processing = false;
+               if #session.send_buffer > 0 then
+                       session.send("");
+               end
        end
 end