net.server_select: Restore real sendbuffer() before calling onconnect handler, in...

[prosody.git] / net / xmppcomponent_listener.lua

diff --git a/net/xmppcomponent_listener.lua b/net/xmppcomponent_listener.lua
index de1dca4580133671e5e8d61e408cf7d8408a19b9..a96a1cffd57eac0d948bb4f596d64a8635ebfc23 100644 (file)
--- a/net/xmppcomponent_listener.lua
+++ b/net/xmppcomponent_listener.lua
@@ -67,19 +67,16 @@ end
 
 function stream_callbacks.streamopened(session, attr)
        if config.get(attr.to, "core", "component_module") ~= "component" then
-               -- Trying to act as a component domain which 
+               -- Trying to act as a component domain which
                -- hasn't been configured
                session:close{ condition = "host-unknown", text = tostring(attr.to).." does not match any configured external components" };
                return;
        end
        
-       -- Store the original host (this is used for config, etc.)
-       session.user = attr.to;
-       -- Set the host for future reference
-       session.host = config.get(attr.to, "core", "component_address") or attr.to;
-       -- Note that we don't create the internal component 
+       -- Note that we don't create the internal component
        -- until after the external component auths successfully
 
+       session.host = attr.to;
        session.streamid = uuid_gen();
        session.notopen = nil;
        
@@ -89,7 +86,7 @@ function stream_callbacks.streamopened(session, attr)
 end
 
 function stream_callbacks.streamclosed(session)
-       session.log("Received </stream:stream>");
+       session.log("debug", "Received </stream:stream>");
        session:close();
 end
 
@@ -107,6 +104,7 @@ function stream_callbacks.handlestanza(session, stanza)
                                local _, domain = jid_split(stanza.attr.from);
                                if domain ~= session.host then
                                        -- Return error
+                                       session.log("warn", "Component sent stanza with missing or invalid 'from' address");
                                        session:close{
                                                condition = "invalid-from";
                                                text = "Component tried to send from address <"..tostring(from)
@@ -119,6 +117,7 @@ function stream_callbacks.handlestanza(session, stanza)
                        stanza.attr.from = session.host;
                end
                if not stanza.attr.to then
+                       session.log("warn", "Rejecting stanza with no 'to' address");
                        session.send(st.error_reply(stanza, "modify", "bad-request", "Components MUST specify a 'to' address on stanzas"));
                        return;
                end
@@ -204,7 +203,7 @@ function component_listener.ondisconnect(conn, err)
        local session = sessions[conn];
        if session then
                (session.log or log)("info", "component disconnected: %s (%s)", tostring(session.host), tostring(err));
-               if session.host then
+               if hosts[session.host] then
                        log("debug", "Deregistering component");
                        cm_deregister_component(session.host);
                        hosts[session.host].connected = nil;