# HG changeset patch # User Matthew Wild # Date 1224814015 -3600 # Node ID fbb3a4ff9cf1c262063ab47a3cfb8709874b07e8 # Parent ed78c1a0401ec8cc49aac100318588f5e900fd22 dialback keys now verified diff -r ed78c1a0401e -r fbb3a4ff9cf1 core/sessionmanager.lua --- a/core/sessionmanager.lua Thu Oct 23 19:10:16 2008 +0100 +++ b/core/sessionmanager.lua Fri Oct 24 03:06:55 2008 +0100 @@ -11,7 +11,7 @@ local modulemanager = require "core.modulemanager"; local log = require "util.logger".init("sessionmanager"); local error = error; -local uuid_generate = require "util.uuid".uuid_generate; +local uuid_generate = require "util.uuid".generate; local rm_load_roster = require "core.rostermanager".load_roster; local newproxy = newproxy; diff -r ed78c1a0401e -r fbb3a4ff9cf1 core/stanza_router.lua --- a/core/stanza_router.lua Thu Oct 23 19:10:16 2008 +0100 +++ b/core/stanza_router.lua Fri Oct 24 03:06:55 2008 +0100 @@ -12,6 +12,10 @@ local send_s2s = require "core.s2smanager".send_to_host; local user_exists = require "core.usermanager".user_exists; +local s2s_verify_dialback = require "core.s2smanager".verify_dialback; +local format = string.format; +local tostring = tostring; + local jid_split = require "util.jid".split; local print = print; @@ -33,10 +37,11 @@ end local to = stanza.attr.to; - stanza.attr.from = origin.full_jid; -- quick fix to prevent impersonation (FIXME this would be incorrect when the origin is not c2s) -- TODO also, stazas should be returned to their original state before the function ends + if origin.type == "c2s" then + stanza.attr.from = origin.full_jid; -- quick fix to prevent impersonation (FIXME this would be incorrect when the origin is not c2s) + end - -- TODO presence subscriptions if not to then core_handle_stanza(origin, stanza); elseif hosts[to] and hosts[to].type == "local" then @@ -90,6 +95,22 @@ log("debug", "Routing stanza to local"); handle_stanza(session, stanza); end + elseif origin.type == "s2sin_unauthed" then + if stanza.name == "verify" and stanza.attr.xmlns == "jabber:server:dialback" then + log("debug", "verifying dialback key..."); + local attr = stanza.attr; + print(tostring(attr.to), tostring(attr.from)) + print(tostring(origin.to_host), tostring(origin.from_host)) + -- FIXME: Grr, ejabberd breaks this one too?? it is black and white in XEP-220 example 34 + --if attr.from ~= origin.to_host then error("invalid-from"); end + local type = "invalid"; + if s2s_verify_dialback(attr.id, attr.from, attr.to, stanza[1]) then + type = "valid" + end + origin.send(format("%s", attr.to, attr.from, attr.id, type, stanza[1])); + end + else + log("warn", "Unhandled origin: %s", origin.type); end end diff -r ed78c1a0401e -r fbb3a4ff9cf1 core/xmlhandlers.lua --- a/core/xmlhandlers.lua Thu Oct 23 19:10:16 2008 +0100 +++ b/core/xmlhandlers.lua Fri Oct 24 03:06:55 2008 +0100 @@ -11,6 +11,8 @@ local t_concatall = function (t, sep) local tt = {}; for _, s in ipairs(t) do t_insert(tt, tostring(s)); end return t_concat(tt, sep); end local sm_destroy_session = import("core.sessionmanager", "destroy_session"); +local default_log = require "util.logger".init("xmlhandlers"); + local error = error; module "xmlhandlers" @@ -21,7 +23,7 @@ local curr_tag; local chardata = {}; local xml_handlers = {}; - local log = session.log; + local log = session.log or default_log; local print = function (...) log("info", "xmlhandlers", t_concatall({...}, "\t")); end local send = session.send; @@ -33,8 +35,11 @@ stanza:text(t_concat(chardata)); chardata = {}; end - curr_ns,name = name:match("^(.+):(%w+)$"); - if not stanza then + log("debug", "Start element: %s", tostring(name)); + curr_ns,name = name:match("^(.+):([%w%-]+)$"); + attr.xmlns = curr_ns; + + if not stanza then --if we are not currently inside a stanza if session.notopen then if name == "stream" then streamopened(session, attr); @@ -45,10 +50,10 @@ if curr_ns == "jabber:client" and name ~= "iq" and name ~= "presence" and name ~= "message" then error("Client sent invalid top-level stanza"); end - attr.xmlns = curr_ns; + stanza = st.stanza(name, attr); --{ to = attr.to, type = attr.type, id = attr.id, xmlns = curr_ns }); curr_tag = stanza; - else + else -- we are inside a stanza, so add a tag attr.xmlns = curr_ns; stanza:tag(name, attr); end @@ -59,12 +64,14 @@ end end function xml_handlers:EndElement(name) - curr_ns,name = name:match("^(.+):(%w+)$"); + curr_ns,name = name:match("^(.+):([%w%-]+)$"); if (not stanza) or #stanza.last_add < 0 or (#stanza.last_add > 0 and name ~= stanza.last_add[#stanza.last_add].name) then if name == "stream" then log("debug", "Stream closed"); sm_destroy_session(session); return; + elseif name == "error" then + error("Stream error: "..tostring(name)..": "..tostring(stanza)); else error("XML parse error in client stream"); end diff -r ed78c1a0401e -r fbb3a4ff9cf1 main.lua --- a/main.lua Thu Oct 23 19:10:16 2008 +0100 +++ b/main.lua Fri Oct 24 03:06:55 2008 +0100 @@ -54,5 +54,6 @@ local protected_disconnect = function (conn, err) local success, ret = pcall(disconnect, conn, err); if not success then print("ERROR on "..tostring(conn).." disconnect: "..ret); conn:close(); end end; start("xmppclient", { ssl = ssl_ctx }) +start("xmppserver", { ssl = ssl_ctx }) server.loop(); diff -r ed78c1a0401e -r fbb3a4ff9cf1 net/connlisteners.lua --- a/net/connlisteners.lua Thu Oct 23 19:10:16 2008 +0100 +++ b/net/connlisteners.lua Fri Oct 24 03:06:55 2008 +0100 @@ -28,7 +28,6 @@ if not h then pcall(dofile, "net/"..name:gsub("[^%w%-]", "_").."_listener.lua"); h = listeners[name]; - end return h; end diff -r ed78c1a0401e -r fbb3a4ff9cf1 util/stanza.lua --- a/util/stanza.lua Thu Oct 23 19:10:16 2008 +0100 +++ b/util/stanza.lua Fri Oct 24 03:06:55 2008 +0100 @@ -6,8 +6,14 @@ local pairs = pairs; local ipairs = ipairs; local type = type; +local next = next; +local print = print; local unpack = unpack; local s_gsub = string.gsub; + +local debug = debug; +local log = require "util.logger".init("stanza"); + module "stanza" stanza_mt = {}; @@ -91,7 +97,6 @@ if t.attr then for k, v in pairs(t.attr) do if type(k) == "string" then attr_string = attr_string .. s_format(" %s='%s'", k, tostring(v)); end end end - return s_format("<%s%s>%s", t.name, attr_string, children_text, t.name); end diff -r ed78c1a0401e -r fbb3a4ff9cf1 util/uuid.lua --- a/util/uuid.lua Thu Oct 23 19:10:16 2008 +0100 +++ b/util/uuid.lua Fri Oct 24 03:06:55 2008 +0100 @@ -2,7 +2,7 @@ local m_random = math.random; module "uuid" -function uuid_generate() +function generate() return m_random(0, 99999999); end