sessionmanager: Return 'not-allowed' error instead of the non-existent 'already-bound...
[prosody.git] / core / stanza_router.lua
index a28110d5c30ce4af7848c84415d8dab13e1d2f95..a2c7b3963abfba49273e12de367aebb864e28ade 100644 (file)
@@ -1,59 +1,81 @@
--- Prosody IM v0.4
--- Copyright (C) 2008-2009 Matthew Wild
--- Copyright (C) 2008-2009 Waqas Hussain
+-- Prosody IM
+-- Copyright (C) 2008-2010 Matthew Wild
+-- Copyright (C) 2008-2010 Waqas Hussain
 -- 
 -- This project is MIT/X11 licensed. Please see the
 -- COPYING file in the source package for more information.
 --
 
-
-
 local log = require "util.logger".init("stanzarouter")
 
-local hosts = _G.hosts;
-
-local st = require "util.stanza";
-local send_s2s = require "core.s2smanager".send_to_host;
-local user_exists = require "core.usermanager".user_exists;
-
-local rostermanager = require "core.rostermanager";
-local offlinemanager = require "core.offlinemanager";
-
-local modules_handle_stanza = require "core.modulemanager".handle_stanza;
-local component_handle_stanza = require "core.componentmanager".handle_stanza;
-
+local hosts = _G.prosody.hosts;
 local tostring = tostring;
-local t_insert = table.insert;
-local pairs = pairs;
-local ipairs = ipairs;
-
+local st = require "util.stanza";
 local jid_split = require "util.jid".split;
 local jid_prepped_split = require "util.jid".prepped_split;
-local fire_event = prosody.events.fire_event;
 
-local select_best_resources;
+local full_sessions = _G.prosody.full_sessions;
+local bare_sessions = _G.prosody.bare_sessions;
 
-function core_process_stanza(origin, stanza)
-       (origin.log or log)("debug", "Received[%s]: %s", origin.type, stanza:top_tag())
+local core_post_stanza, core_process_stanza, core_route_stanza;
 
-       -- Currently we guarantee every stanza to have an xmlns, should we keep this rule?
-       if not stanza.attr.xmlns then stanza.attr.xmlns = "jabber:client"; end
-       
-       -- TODO verify validity of stanza (as well as JID validity)
-       if stanza.attr.type == "error" and #stanza.tags == 0 then return; end -- TODO invalid stanza, log
-       if stanza.name == "iq" then
-               if (stanza.attr.type == "set" or stanza.attr.type == "get") and #stanza.tags ~= 1 then
-                       origin.send(st.error_reply(stanza, "modify", "bad-request"));
+function deprecated_warning(f)
+       _G[f] = function(...)
+               log("warn", "Using the global %s() is deprecated, use module:send() or prosody.%s(). %s", f, f, debug.traceback());
+               return prosody[f](...);
+       end
+end
+deprecated_warning"core_post_stanza";
+deprecated_warning"core_process_stanza";
+deprecated_warning"core_route_stanza";
+
+local valid_stanzas = { message = true, presence = true, iq = true };
+local function handle_unhandled_stanza(host, origin, stanza)
+       local name, xmlns, origin_type = stanza.name, stanza.attr.xmlns or "jabber:client", origin.type;
+       if xmlns == "jabber:client" and valid_stanzas[name] then
+               -- A normal stanza
+               local st_type = stanza.attr.type;
+               if st_type == "error" or (name == "iq" and st_type == "result") then
+                       log("debug", "Discarding %s from %s of type: %s", name, origin_type, st_type or '<nil>');
                        return;
                end
+               if name == "iq" and (st_type == "get" or st_type == "set") and stanza.tags[1] then
+                       xmlns = stanza.tags[1].attr.xmlns or "jabber:client";
+               end
+               log("debug", "Unhandled %s stanza: %s; xmlns=%s", origin_type, name, xmlns);
+               if origin.send then
+                       origin.send(st.error_reply(stanza, "cancel", "service-unavailable"));
+               end
+       elseif not((name == "features" or name == "error") and xmlns == "http://etherx.jabber.org/streams") then -- FIXME remove check once we handle S2S features
+               log("warn", "Unhandled %s stream element or stanza: %s; xmlns=%s: %s", origin_type, name, xmlns, tostring(stanza)); -- we didn't handle it
+               origin:close("unsupported-stanza-type");
        end
+end
+
+local iq_types = { set=true, get=true, result=true, error=true };
+function core_process_stanza(origin, stanza)
+       (origin.log or log)("debug", "Received[%s]: %s", origin.type, stanza:top_tag())
+
+       if origin.type == "c2s" and not stanza.attr.xmlns then
+               local name, st_type = stanza.name, stanza.attr.type;
+               if st_type == "error" and #stanza.tags == 0 then
+                       return handle_unhandled_stanza(origin.host, origin, stanza);
+               end
+               if name == "iq" then
+                       if not stanza.attr.id then stanza.attr.id = ""; end -- COMPAT Jabiru doesn't send the id attribute on roster requests
+                       if not iq_types[st_type] or ((st_type == "set" or st_type == "get") and (#stanza.tags ~= 1)) then
+                               origin.send(st.error_reply(stanza, "modify", "bad-request", "Invalid IQ type or incorrect number of children"));
+                               return;
+                       end
+               end
 
-       if origin.type == "c2s" then
                if not origin.full_jid
-                       and not(stanza.name == "iq" and stanza.attr.type == "set" and stanza.tags[1] and stanza.tags[1].name == "bind"
+                       and not(name == "iq" and st_type == "set" and stanza.tags[1] and stanza.tags[1].name == "bind"
                                        and stanza.tags[1].attr.xmlns == "urn:ietf:params:xml:ns:xmpp-bind") then
                        -- authenticated client isn't bound and current stanza is not a bind request
-                       origin.send(st.error_reply(stanza, "auth", "not-authorized")); -- FIXME maybe allow stanzas to account or server
+                       if stanza.attr.type ~= "result" and stanza.attr.type ~= "error" then
+                               origin.send(st.error_reply(stanza, "auth", "not-authorized")); -- FIXME maybe allow stanzas to account or server
+                       end
                        return;
                end
 
@@ -66,19 +88,23 @@ function core_process_stanza(origin, stanza)
        local from_node, from_host, from_resource;
        local to_bare, from_bare;
        if to then
-               node, host, resource = jid_prepped_split(to);
-               if not host then
-                       log("warn", "Received stanza with invalid destination JID: %s", to);
-                       if stanza.attr.type ~= "error" and stanza.attr.type ~= "result" then
-                               origin.send(st.error_reply(stanza, "modify", "jid-malformed", "The destination address is invalid: "..to));
+               if full_sessions[to] or bare_sessions[to] or hosts[to] then
+                       node, host = jid_split(to); -- TODO only the host is needed, optimize
+               else
+                       node, host, resource = jid_prepped_split(to);
+                       if not host then
+                               log("warn", "Received stanza with invalid destination JID: %s", to);
+                               if stanza.attr.type ~= "error" and stanza.attr.type ~= "result" then
+                                       origin.send(st.error_reply(stanza, "modify", "jid-malformed", "The destination address is invalid: "..to));
+                               end
+                               return;
                        end
-                       return;
+                       to_bare = node and (node.."@"..host) or host; -- bare JID
+                       if resource then to = to_bare.."/"..resource; else to = to_bare; end
+                       stanza.attr.to = to;
                end
-               to_bare = node and (node.."@"..host) or host; -- bare JID
-               if resource then to = to_bare.."/"..resource; else to = to_bare; end
-               stanza.attr.to = to;
        end
-       if from then
+       if from and not origin.full_jid then
                -- We only stamp the 'from' on c2s stanzas, so we still need to check validity
                from_node, from_host, from_resource = jid_prepped_split(from);
                if not from_host then
@@ -93,46 +119,46 @@ function core_process_stanza(origin, stanza)
                stanza.attr.from = from;
        end
 
-       --[[if to and not(hosts[to]) and not(hosts[to_bare]) and (hosts[host] and hosts[host].type ~= "local") then -- not for us?
-               log("warn", "stanza recieved for a non-local server");
-               return; -- FIXME what should we do here?
-       end]] -- FIXME
-
-       if (origin.type == "s2sin" or origin.type == "c2s" or origin.type == "component") and xmlns == "jabber:client" then
+       if (origin.type == "s2sin" or origin.type == "c2s" or origin.type == "component") and xmlns == nil then
                if origin.type == "s2sin" and not origin.dummy then
                        local host_status = origin.hosts[from_host];
                        if not host_status or not host_status.authed then -- remote server trying to impersonate some other server?
                                log("warn", "Received a stanza claiming to be from %s, over a stream authed for %s!", from_host, origin.from_host);
-                               return; -- FIXME what should we do here? does this work with subdomains?
+                               origin:close("not-authorized");
+                               return;
+                       elseif not hosts[host] then
+                               log("warn", "Remote server %s sent us a stanza for %s, closing stream", origin.from_host, host);
+                               origin:close("host-unknown");
+                               return;
                        end
                end
-               core_post_stanza(origin, stanza);
+               core_post_stanza(origin, stanza, origin.full_jid);
        else
                local h = hosts[stanza.attr.to or origin.host or origin.to_host];
                if h then
                        local event;
-                       if stanza.attr.xmlns == "jabber:client" then
+                       if xmlns == nil then
                                if stanza.name == "iq" and (stanza.attr.type == "set" or stanza.attr.type == "get") then
                                        event = "stanza/iq/"..stanza.tags[1].attr.xmlns..":"..stanza.tags[1].name;
                                else
                                        event = "stanza/"..stanza.name;
                                end
                        else
-                               event = "stanza/"..stanza.attr.xmlns..":"..stanza.name;
+                               event = "stanza/"..xmlns..":"..stanza.name;
                        end
                        if h.events.fire_event(event, {origin = origin, stanza = stanza}) then return; end
                end
-               if host and not hosts[host] then host = nil; end -- workaround for a Pidgin bug which sets 'to' to the SRV result
-               modules_handle_stanza(host or origin.host or origin.to_host, origin, stanza);
+               if host and not hosts[host] then host = nil; end -- COMPAT: workaround for a Pidgin bug which sets 'to' to the SRV result
+               handle_unhandled_stanza(host or origin.host or origin.to_host, origin, stanza);
        end
 end
 
-function core_post_stanza(origin, stanza)
+function core_post_stanza(origin, stanza, preevents)
        local to = stanza.attr.to;
        local node, host, resource = jid_split(to);
        local to_bare = node and (node.."@"..host) or host; -- bare JID
 
-       local to_type;
+       local to_type, to_self;
        if node then
                if resource then
                        to_type = '/full';
@@ -140,6 +166,7 @@ function core_post_stanza(origin, stanza)
                        to_type = '/bare';
                        if node == origin.username and host == origin.host then
                                stanza.attr.to = nil;
+                               to_self = true;
                        end
                end
        else
@@ -147,181 +174,53 @@ function core_post_stanza(origin, stanza)
                        to_type = '/host';
                else
                        to_type = '/bare';
+                       to_self = true;
                end
        end
 
        local event_data = {origin=origin, stanza=stanza};
-       if origin.full_jid then -- c2s connection
+       if preevents then -- c2s connection
                if hosts[origin.host].events.fire_event('pre-'..stanza.name..to_type, event_data) then return; end -- do preprocessing
        end
        local h = hosts[to_bare] or hosts[host or origin.host];
        if h then
-               if h.type == "component" then
-                       component_handle_stanza(origin, stanza);
-                       return;
-               else
-                       if h.events.fire_event(stanza.name..to_type, event_data) then return; end -- do processing
-               end
-       end
-
-       if host and fire_event(host.."/"..stanza.name, event_data) then
-               -- event handled
-       elseif stanza.name == "presence" and origin.host and fire_event(origin.host.."/"..stanza.name, event_data) then
-               -- event handled
-       elseif not to then
-               modules_handle_stanza(host or origin.host or origin.to_host, origin, stanza);
-       elseif hosts[to] and hosts[to].type == "local" then -- directed at a local server
-               modules_handle_stanza(host or origin.host or origin.to_host, origin, stanza);
-       elseif hosts[to_bare] and hosts[to_bare].type == "component" then -- hack to allow components to handle node@server
-               component_handle_stanza(origin, stanza);
-       elseif hosts[host] and hosts[host].type == "component" then -- directed at a component
-               component_handle_stanza(origin, stanza);
-       elseif hosts[host] and hosts[host].type == "local" and stanza.name == "iq" and not resource then -- directed at bare JID
-               modules_handle_stanza(host or origin.host or origin.to_host, origin, stanza);
+               if h.events.fire_event(stanza.name..to_type, event_data) then return; end -- do processing
+               if to_self and h.events.fire_event(stanza.name..'/self', event_data) then return; end -- do processing
+               handle_unhandled_stanza(h.host, origin, stanza);
        else
                core_route_stanza(origin, stanza);
        end
 end
 
 function core_route_stanza(origin, stanza)
-       -- Hooks
-       --- ...later
-
-       -- Deliver
-       local to = stanza.attr.to;
-       local node, host, resource = jid_split(to);
-       local to_bare = node and (node.."@"..host) or host; -- bare JID
-       local from = stanza.attr.from;
-       local from_node, from_host, from_resource = jid_split(from);
-       local from_bare = from_node and (from_node.."@"..from_host) or from_host; -- bare JID
+       local node, host, resource = jid_split(stanza.attr.to);
+       local from_node, from_host, from_resource = jid_split(stanza.attr.from);
 
        -- Auto-detect origin if not specified
        origin = origin or hosts[from_host];
        if not origin then return false; end
        
-       if hosts[to_bare] and hosts[to_bare].type == "component" then -- hack to allow components to handle node@server
-               return component_handle_stanza(origin, stanza);
-       elseif hosts[host] and hosts[host].type == "component" then -- directed at a component
-               return component_handle_stanza(origin, stanza);
-       end
-
-       if stanza.name == "presence" and (stanza.attr.type ~= nil and stanza.attr.type ~= "unavailable" and stanza.attr.type ~= "error") then resource = nil; end
-
-       local host_session = hosts[host]
-       if host_session and host_session.type == "local" then
-               -- Local host
-               local user = host_session.sessions[node];
-               if user then
-                       local res = user.sessions[resource];
-                       if res then -- resource is online...
-                               res.send(stanza); -- Yay \o/
-                       else
-                               -- if we get here, resource was not specified or was unavailable
-                               if stanza.name == "presence" then
-                                       if stanza.attr.type ~= nil and stanza.attr.type ~= "unavailable" and stanza.attr.type ~= "error" then
-                                               -- inbound presence subscriptions and probes, already handled, so should never get here
-                                       elseif not resource then -- sender is available or unavailable or error
-                                               for _, session in pairs(user.sessions) do -- presence broadcast to all user resources.
-                                                       if session.full_jid then -- FIXME should this be just for available resources? Do we need to check subscription?
-                                                               stanza.attr.to = session.full_jid; -- reset at the end of function
-                                                               session.send(stanza);
-                                                       end
-                                               end
-                                       end
-                               elseif stanza.name == "message" then -- select a resource to recieve message
-                                       stanza.attr.to = to_bare;
-                                       if stanza.attr.type == 'headline' then
-                                               for _, session in pairs(user.sessions) do -- find resource with greatest priority
-                                                       if session.presence and session.priority >= 0 then
-                                                               session.send(stanza);
-                                                       end
-                                               end
-                                       elseif stanza.attr.type == 'groupchat' then
-                                               -- Groupchat message sent to offline resource
-                                               origin.send(st.error_reply(stanza, "cancel", "service-unavailable"));
-                                       else
-                                               local count = 0;
-                                               for _, session in ipairs(select_best_resources(user)) do
-                                                       session.send(stanza);
-                                                       count = count + 1;
-                                               end
-                                               if count == 0 and (stanza.attr.type == "chat" or stanza.attr.type == "normal" or not stanza.attr.type) then
-                                                       offlinemanager.store(node, host, stanza);
-                                                       -- TODO deal with storage errors
-                                               end
-                                       end
-                               elseif stanza.attr.type == "get" or stanza.attr.type == "set" then
-                                       origin.send(st.error_reply(stanza, "cancel", "service-unavailable"));
-                               end
-                       end
-               else
-                       -- user not online
-                       if user_exists(node, host) then
-                               if stanza.name == "presence" then
-                                       if stanza.attr.type ~= nil and stanza.attr.type ~= "unavailable" and stanza.attr.type ~= "error" then
-                                               -- inbound presence subscriptions and probes, already handled, so should never get here
-                                       else
-                                               -- TODO send unavailable presence or unsubscribed
-                                       end
-                               elseif stanza.name == "message" then -- FIXME if full jid, then send out to resources with highest priority
-                                       stanza.attr.to = to_bare; -- TODO not in RFC, but seems obvious. Should discuss on the mailing list.
-                                       if stanza.attr.type == "chat" or stanza.attr.type == "normal" or not stanza.attr.type then
-                                               offlinemanager.store(node, host, stanza);
-                                               -- FIXME don't store messages with only chat state notifications
-                                       elseif stanza.attr.type == "groupchat" then
-                                               local reply = st.error_reply(stanza, "cancel", "service-unavailable");
-                                               reply.attr.from = to;
-                                               origin.send(reply);
-                                       end
-                                       -- TODO allow configuration of offline storage
-                                       -- TODO send error if not storing offline
-                               elseif stanza.name == "iq" and (stanza.attr.type == "get" or stanza.attr.type == "set") then
-                                       origin.send(st.error_reply(stanza, "cancel", "service-unavailable"));
-                               end
-                       else -- user does not exist
-                               -- TODO we would get here for nodeless JIDs too. Do something fun maybe? Echo service? Let plugins use xmpp:server/resource addresses?
-                               if stanza.name == "presence" then
-                                       local t = stanza.attr.type;
-                                       if t == "subscribe" or t == "probe" then
-                                               origin.send(st.presence({from = to_bare, to = from_bare, type = "unsubscribed"}));
-                                       end
-                                       -- else ignore
-                               elseif stanza.attr.type ~= "error" and (stanza.name ~= "iq" or stanza.attr.type ~= "result") then
-                                       origin.send(st.error_reply(stanza, "cancel", "service-unavailable"));
-                               end
-                       end
-               end
-       elseif origin.type == "c2s" then
-               -- Remote host
-               local xmlns = stanza.attr.xmlns;
-               --stanza.attr.xmlns = "jabber:server";
-               stanza.attr.xmlns = nil;
-               log("debug", "sending s2s stanza: %s", tostring(stanza));
-               send_s2s(origin.host, host, stanza); -- TODO handle remote routing errors
-               stanza.attr.xmlns = xmlns; -- reset
-       elseif origin.type == "component" or origin.type == "local" then
-               -- Route via s2s for components and modules
-               log("debug", "Routing outgoing stanza for %s to %s", from_host, host);
-               send_s2s(from_host, host, stanza);
+       if hosts[host] then
+               -- old stanza routing code removed
+               core_post_stanza(origin, stanza);
        else
-               log("warn", "received stanza from unhandled connection type: %s", origin.type);
-       end
-       stanza.attr.to = to; -- reset
-end
-
-function select_best_resources(user)
-       local priority = 0;
-       local recipients = {};
-       for _, session in pairs(user.sessions) do -- find resource with greatest priority
-               if session.presence then
-                       local p = session.priority;
-                       if p > priority then
-                               priority = p;
-                               recipients = {session};
-                       elseif p == priority then
-                               t_insert(recipients, session);
+               log("debug", "Routing to remote...");
+               local host_session = hosts[from_host];
+               if not host_session then
+                       log("error", "No hosts[from_host] (please report): %s", tostring(stanza));
+               else
+                       local xmlns = stanza.attr.xmlns;
+                       stanza.attr.xmlns = nil;
+                       local routed = host_session.events.fire_event("route/remote", { origin = origin, stanza = stanza, from_host = from_host, to_host = host });
+                       stanza.attr.xmlns = xmlns; -- reset
+                       if not routed then
+                               log("debug", "... no, just kidding.");
+                               if stanza.attr.type == "error" or (stanza.name == "iq" and stanza.attr.type == "result") then return; end
+                               core_route_stanza(host_session, st.error_reply(stanza, "cancel", "not-allowed", "Communication with remote domains is not enabled"));
                        end
                end
        end
-       return recipients;
 end
+prosody.core_process_stanza = core_process_stanza;
+prosody.core_post_stanza = core_post_stanza;
+prosody.core_route_stanza = core_route_stanza;