Merge 0.7->0.8
[prosody.git] / core / s2smanager.lua
index 88d6a796cc4c21368eb6e80e4d5de9b97e127e0a..fd9a72d06a258b54309b89e0d0946504461319d5 100644 (file)
@@ -16,10 +16,8 @@ local socket = require "socket";
 local format = string.format;
 local t_insert, t_sort = table.insert, table.sort;
 local get_traceback = debug.traceback;
-local tostring, pairs, ipairs, getmetatable, newproxy, error, tonumber,
-      setmetatable
-    = tostring, pairs, ipairs, getmetatable, newproxy, error, tonumber,
-      setmetatable;
+local tostring, pairs, ipairs, getmetatable, newproxy, error, tonumber, setmetatable
+    = tostring, pairs, ipairs, getmetatable, newproxy, error, tonumber, setmetatable;
 
 local idna_to_ascii = require "util.encodings".idna.to_ascii;
 local connlisteners_get = require "net.connlisteners".get;
@@ -47,8 +45,9 @@ local max_dns_depth = config.get("*", "core", "dns_max_depth") or 3;
 
 dns.settimeout(dns_timeout);
 
+local prosody = _G.prosody;
 incoming_s2s = {};
-_G.prosody.incoming_s2s = incoming_s2s;
+prosody.incoming_s2s = incoming_s2s;
 local incoming_s2s = incoming_s2s;
 
 module "s2smanager"
@@ -71,13 +70,13 @@ local function bounce_sendq(session, reason)
                };
                for i, data in ipairs(sendq) do
                        local reply = data[2];
-                       local xmlns = reply.attr.xmlns;
-                       if not(xmlns) and bouncy_stanzas[reply.name] then
+                       if reply and not(reply.attr.xmlns) and bouncy_stanzas[reply.name] then
                                reply.attr.type = "error";
                                reply:tag("error", {type = "cancel"})
                                        :tag("remote-server-not-found", {xmlns = "urn:ietf:params:xml:ns:xmpp-stanzas"}):up();
                                if reason then
-                                       reply:tag("text", {xmlns = "urn:ietf:params:xml:ns:xmpp-stanzas"}):text("Connection failed: "..reason):up();
+                                       reply:tag("text", {xmlns = "urn:ietf:params:xml:ns:xmpp-stanzas"})
+                                               :text("Server-to-server connection failed: "..reason):up();
                                end
                                core_process_stanza(dummy, reply);
                        end
@@ -99,13 +98,14 @@ function send_to_host(from_host, to_host, data)
                        (host.log or log)("debug", "trying to send over unauthed s2sout to "..to_host);
                        
                        -- Queue stanza until we are able to send it
-                       if host.sendq then t_insert(host.sendq, {tostring(data), st.reply(data)});
-                       else host.sendq = { {tostring(data), st.reply(data)} }; end
+                       if host.sendq then t_insert(host.sendq, {tostring(data), data.attr.type ~= "error" and data.attr.type ~= "result" and st.reply(data)});
+                       else host.sendq = { {tostring(data), data.attr.type ~= "error" and data.attr.type ~= "result" and st.reply(data)} }; end
                        host.log("debug", "stanza [%s] queued ", data.name);
                elseif host.type == "local" or host.type == "component" then
                        log("error", "Trying to send a stanza to ourselves??")
                        log("error", "Traceback: %s", get_traceback());
                        log("error", "Stanza: %s", tostring(data));
+                       return false;
                else
                        (host.log or log)("debug", "going to send stanza to "..to_host.." from "..from_host);
                        -- FIXME
@@ -121,13 +121,18 @@ function send_to_host(from_host, to_host, data)
                local host_session = new_outgoing(from_host, to_host);
 
                -- Store in buffer
-               host_session.sendq = { {tostring(data), st.reply(data)} };
+               host_session.sendq = { {tostring(data), data.attr.type ~= "error" and data.attr.type ~= "result" and st.reply(data)} };
                log("debug", "stanza [%s] queued until connection complete", tostring(data.name));
                if (not host_session.connecting) and (not host_session.conn) then
                        log("warn", "Connection to %s failed already, destroying session...", to_host);
-                       destroy_session(host_session);
+                       if not destroy_session(host_session, "Connection failed") then
+                               -- Already destroyed, we need to bounce our stanza
+                               bounce_sendq(host_session, host_session.destruction_reason);
+                       end
+                       return false;
                end
        end
+       return true;
 end
 
 local open_sessions = 0;
@@ -161,7 +166,7 @@ function new_incoming(conn)
                        return; -- Ok, we're connect[ed|ing]
                end
                -- Not connected, need to close session and clean up
-               (session.log or log)("warn", "Destroying incomplete session %s->%s due to inactivity",
+               (session.log or log)("debug", "Destroying incomplete session %s->%s due to inactivity",
                    session.from_host or "(unknown)", session.to_host or "(unknown)");
                session:close("connection-timeout");
        end);
@@ -175,6 +180,8 @@ function new_outgoing(from_host, to_host, connect)
                
                hosts[from_host].s2sout[to_host] = host_session;
                
+               host_session.close = destroy_session; -- This gets replaced by xmppserver_listener later
+               
                local log;
                do
                        local conn_name = "s2sout"..tostring(host_session):match("[a-f0-9]*$");
@@ -186,7 +193,11 @@ function new_outgoing(from_host, to_host, connect)
                
                if connect ~= false then
                        -- Kick the connection attempting machine into life
-                       attempt_connection(host_session);
+                       if not attempt_connection(host_session) then
+                               -- Intentionally not returning here, the
+                               -- session is needed, connected or not
+                               destroy_session(host_session);
+                       end
                end
                
                if not host_session.sends2s then
@@ -276,7 +287,7 @@ end
 function try_connect(host_session, connect_host, connect_port)
        host_session.connecting = true;
        local handle;
-       handle = adns.lookup(function (reply)
+       handle = adns.lookup(function (reply, err)
                handle = nil;
                host_session.connecting = nil;
                
@@ -294,12 +305,19 @@ function try_connect(host_session, connect_host, connect_port)
                
                if reply and reply[#reply] and reply[#reply].a then
                        log("debug", "DNS reply for %s gives us %s", connect_host, reply[#reply].a);
-                       return make_connect(host_session, reply[#reply].a, connect_port);
+                       local ok, err = make_connect(host_session, reply[#reply].a, connect_port);
+                       if not ok then
+                               if not attempt_connection(host_session, err or "closed") then
+                                       err = err and (": "..err) or "";
+                                       destroy_session(host_session, "Connection failed"..err);
+                               end
+                       end
                else
                        log("debug", "DNS lookup failed to get a response for %s", connect_host);
                        if not attempt_connection(host_session, "name resolution failed") then -- Retry if we can
                                log("debug", "No other records to try for %s - destroying", host_session.to_host);
-                               destroy_session(host_session, "DNS resolution failed"); -- End of the line, we can't
+                               err = err and (": "..err) or "";
+                               destroy_session(host_session, "DNS resolution failed"..err); -- End of the line, we can't
                        end
                end
        end, connect_host, "A", "IN");
@@ -313,7 +331,7 @@ function make_connect(host_session, connect_host, connect_port)
        
        local from_host, to_host = host_session.from_host, host_session.to_host;
        
-       local conn, handler = socket.tcp()
+       local conn, handler = socket.tcp();
        
        if not conn then
                log("warn", "Failed to create outgoing connection, system error: %s", handler);
@@ -491,9 +509,11 @@ function make_authenticated(session, host)
        elseif session.type == "s2sin_unauthed" then
                session.type = "s2sin";
                if host then
+                       if not session.hosts[host] then session.hosts[host] = {}; end
                        session.hosts[host].authed = true;
                end
        elseif session.type == "s2sin" and host then
+               if not session.hosts[host] then session.hosts[host] = {}; end
                session.hosts[host].authed = true;
        else
                return false;
@@ -514,8 +534,16 @@ function mark_connected(session)
        session.log("info", session.direction.." s2s connection "..from.."->"..to.." complete");
        
        local send_to_host = send_to_host;
-       function session.send(data) send_to_host(to, from, data); end
+       function session.send(data) return send_to_host(to, from, data); end
        
+       local event_data = { session = session };
+       if session.type == "s2sout" then
+               prosody.events.fire_event("s2sout-established", event_data);
+               hosts[session.from_host].events.fire_event("s2sout-established", event_data);
+       else
+               prosody.events.fire_event("s2sin-established", event_data);
+               hosts[session.to_host].events.fire_event("s2sin-established", event_data);
+       end
        
        if session.direction == "outgoing" then
                if sendq then
@@ -543,7 +571,7 @@ local resting_session = { -- Resting, not dead
                filter = function (type, data) return data; end;
        }; resting_session.__index = resting_session;
 
-function retire_session(session)
+function retire_session(session, reason)
        local log = session.log or log;
        for k in pairs(session) do
                if k ~= "trace" and k ~= "log" and k ~= "id" then
@@ -551,6 +579,8 @@ function retire_session(session)
                end
        end
 
+       session.destruction_reason = reason;
+
        function session.send(data) log("debug", "Discarding data sent to resting session: %s", tostring(data)); end
        function session.data(data) log("debug", "Discarding data received from resting session: %s", tostring(data)); end
        return setmetatable(session, resting_session);
@@ -558,7 +588,7 @@ end
 
 function destroy_session(session, reason)
        if session.destroyed then return; end
-       (session.log or log)("info", "Destroying "..tostring(session.direction).." session "..tostring(session.from_host).."->"..tostring(session.to_host));
+       (session.log or log)("debug", "Destroying "..tostring(session.direction).." session "..tostring(session.from_host).."->"..tostring(session.to_host));
        
        if session.direction == "outgoing" then
                hosts[session.from_host].s2sout[session.to_host] = nil;
@@ -567,7 +597,21 @@ function destroy_session(session, reason)
                incoming_s2s[session] = nil;
        end
        
-       retire_session(session); -- Clean session until it is GC'd
+       local event_data = { session = session, reason = reason };
+       if session.type == "s2sout" then
+               prosody.events.fire_event("s2sout-destroyed", event_data);
+               if hosts[session.from_host] then
+                       hosts[session.from_host].events.fire_event("s2sout-destroyed", event_data);
+               end
+       elseif session.type == "s2sin" then
+               prosody.events.fire_event("s2sin-destroyed", event_data);
+               if hosts[session.to_host] then
+                       hosts[session.to_host].events.fire_event("s2sin-destroyed", event_data);
+               end
+       end
+       
+       retire_session(session, reason); -- Clean session until it is GC'd
+       return true;
 end
 
 return _M;