Merge 0.8->trunk
[prosody.git] / core / stanza_router.lua
index 72ddebd1d45f72809c1a3270ff3b17cacebbb201..97d328a1afbe3b2c51b2bff12a31fa139581bbda 100644 (file)
@@ -1,6 +1,6 @@
 -- Prosody IM
--- Copyright (C) 2008-2009 Matthew Wild
--- Copyright (C) 2008-2009 Waqas Hussain
+-- 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.
@@ -12,20 +12,37 @@ local hosts = _G.prosody.hosts;
 local tostring = tostring;
 local st = require "util.stanza";
 local send_s2s = require "core.s2smanager".send_to_host;
-local modules_handle_stanza = require "core.modulemanager".handle_stanza;
-local component_handle_stanza = require "core.componentmanager".handle_stanza;
 local jid_split = require "util.jid".split;
 local jid_prepped_split = require "util.jid".prepped_split;
 
 local full_sessions = _G.prosody.full_sessions;
 local bare_sessions = _G.prosody.bare_sessions;
 
+local function handle_unhandled_stanza(host, origin, stanza)
+       local name, xmlns, origin_type = stanza.name, stanza.attr.xmlns or "jabber:client", origin.type;
+       if name == "iq" and xmlns == "jabber:client" then
+               if stanza.attr.type == "get" or stanza.attr.type == "set" then
+                       xmlns = stanza.tags[1].attr.xmlns or "jabber:client";
+                       log("debug", "Stanza of type %s from %s has xmlns: %s", name, origin_type, xmlns);
+               else
+                       log("debug", "Discarding %s from %s of type: %s", name, origin_type, stanza.attr.type);
+                       return true;
+               end
+       end
+       if stanza.attr.xmlns == nil then
+               log("debug", "Unhandled %s stanza: %s; xmlns=%s", origin.type, stanza.name, xmlns); -- we didn't handle it
+               if stanza.attr.type ~= "error" and stanza.attr.type ~= "result" 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: %s; xmlns=%s: %s", origin.type, stanza.name, xmlns, tostring(stanza)); -- we didn't handle it
+               origin:close("unsupported-stanza-type");
+       end
+end
+
 function core_process_stanza(origin, stanza)
        (origin.log or log)("debug", "Received[%s]: %s", origin.type, stanza:top_tag())
 
-       -- 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
@@ -36,12 +53,14 @@ function core_process_stanza(origin, stanza)
                end
        end
 
-       if origin.type == "c2s" then
+       if origin.type == "c2s" and not stanza.attr.xmlns 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 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
 
@@ -90,7 +109,7 @@ function core_process_stanza(origin, stanza)
                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?
@@ -103,19 +122,19 @@ function core_process_stanza(origin, stanza)
                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 -- COMPAT: 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);
+               handle_unhandled_stanza(host or origin.host or origin.to_host, origin, stanza);
        end
 end
 
@@ -152,12 +171,7 @@ function core_post_stanza(origin, stanza, preevents)
        if h then
                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
-
-               if h.type == "component" then
-                       component_handle_stanza(origin, stanza);
-                       return;
-               end
-               modules_handle_stanza(h.host, origin, stanza);
+               handle_unhandled_stanza(h.host, origin, stanza);
        else
                core_route_stanza(origin, stanza);
        end