mod_dialback: Miscellaneous logging improvements, changing levels, improving messages and using session loggers where possible

Thu, 30 Apr 2009 02:45:33 +0100

author
Matthew Wild <mwild1@gmail.com>
date
Thu, 30 Apr 2009 02:45:33 +0100
changeset 1077
d6a885cacd8c
parent 1076
ba3639692493
child 1078
24c9ee99d900

mod_dialback: Miscellaneous logging improvements, changing levels, improving messages and using session loggers where possible

plugins/mod_dialback.lua file | annotate | diff | comparison | revisions
--- a/plugins/mod_dialback.lua	Thu Apr 30 02:45:10 2009 +0100
+++ b/plugins/mod_dialback.lua	Thu Apr 30 02:45:33 2009 +0100
@@ -24,7 +24,7 @@
 module:add_handler({"s2sin_unauthed", "s2sin"}, "verify", xmlns_dialback,
 	function (origin, stanza)
 		-- We are being asked to verify the key, to ensure it was generated by us
-		log("debug", "verifying dialback key...");
+		origin.log("debug", "verifying that dialback key is ours...");
 		local attr = stanza.attr;
 		-- 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
@@ -33,9 +33,9 @@
 			type = "valid"
 		else
 			type = "invalid"
-			log("warn", "Asked to verify a dialback key that was incorrect. An imposter is claiming to be %s?", attr.to);
+			origin.log("warn", "Asked to verify a dialback key that was incorrect. An imposter is claiming to be %s?", attr.to);
 		end
-		log("debug", "verified dialback key... it is %s", type);
+		origin.log("debug", "verified dialback key... it is %s", type);
 		origin.sends2s(st.stanza("db:verify", { from = attr.to, to = attr.from, id = attr.id, type = type }):text(stanza[1]));
 	end);
 
@@ -48,7 +48,7 @@
 		
 		if not hosts[attr.to] then
 			-- Not a host that we serve
-			log("info", "%s tried to connect to %s, which we don't serve", attr.from, attr.to);
+			origin.log("info", "%s tried to connect to %s, which we don't serve", attr.from, attr.to);
 			origin:close("host-unknown");
 			return;
 		end
@@ -64,7 +64,7 @@
 			origin.to_host = attr.to;
 		end
 		
-		log("debug", "asking %s if key %s belongs to them", attr.from, stanza[1]);
+		origin.log("debug", "asking %s if key %s belongs to them", attr.from, stanza[1]);
 		send_s2s(attr.to, attr.from,
 			st.stanza("db:verify", { from = attr.to, to = attr.from, id = origin.streamid }):text(stanza[1]));
 	end);

mercurial