Mercurial > lxmppd-hg-xmllex / changeset

--- a/net/httpserver.lua	Sun Jun 28 08:09:57 2009 -0500
+++ b/net/httpserver.lua	Sun Jun 28 14:17:22 2009 +0100
@@ -52,7 +52,7 @@
 		end
 	else
 		-- Response we have is just a string (the body)
-		log("debug", "Sending response to %s: %s", request.id, response);
+		log("debug", "Sending response to %s: %s", request.id or "<none>", response or "<none>");
 		
 		resp = { "HTTP/1.0 200 OK\r\n" };
 		t_insert(resp, "Connection: close\r\n");

--- a/plugins/mod_console.lua	Sun Jun 28 08:09:57 2009 -0500
+++ b/plugins/mod_console.lua	Sun Jun 28 14:17:22 2009 +0100
@@ -162,7 +162,7 @@
 
 def_env.module = {};
 
-local function get_hosts_set(hosts)
+local function get_hosts_set(hosts, module)
 	if type(hosts) == "table" then
 		if hosts[1] then
 			return set.new(hosts);
@@ -172,8 +172,9 @@
 	elseif type(hosts) == "string" then
 		return set.new { hosts };
 	elseif hosts == nil then
+		local mm = require "modulemanager";
 		return set.new(array.collect(keys(prosody.hosts)))
-			/ function (host) return prosody.hosts[host].type == "local"; end;
+			/ function (host) return prosody.hosts[host].type == "local" or module and mm.is_loaded(host, module); end;
 	end
 end
 
@@ -203,7 +204,7 @@
 function def_env.module:unload(name, hosts)
 	local mm = require "modulemanager";
 
-	hosts = get_hosts_set(hosts);
+	hosts = get_hosts_set(hosts, name);
 	
 	-- Unload the module for each host
 	local ok, err, count = true, nil, 0;
@@ -225,7 +226,7 @@
 function def_env.module:reload(name, hosts)
 	local mm = require "modulemanager";
 
-	hosts = get_hosts_set(hosts);
+	hosts = get_hosts_set(hosts, name);
 	
 	-- Reload the module for each host
 	local ok, err, count = true, nil, 0;

--- a/plugins/mod_pep.lua	Sun Jun 28 08:09:57 2009 -0500
+++ b/plugins/mod_pep.lua	Sun Jun 28 14:17:22 2009 +0100
@@ -9,6 +9,7 @@
 local next = next;
 local load_roster = require "core.rostermanager".load_roster;
 
+local NULL = {};
 local data = {};
 local recipients = {};
 
@@ -35,17 +36,31 @@
 		user_data[node] = stanza;
 	end
 	
-	-- broadcast to resources
-	stanza.attr.to = bare;
-	core_route_stanza(session, stanza);
+	-- broadcast
+	for recipient in pairs(recipients[user] or NULL) do
+		stanza.attr.to = recipient;
+		core_post_stanza(session, stanza);
+	end
+end
 
-	-- broadcast to contacts
-	for jid, item in pairs(session.roster) do
-		if jid and jid ~= "pending" and (item.subscription == 'from' or item.subscription == 'both') then
-			stanza.attr.to = jid;
-			core_route_stanza(session, stanza);
+local function get_caps_hash_from_presence(stanza, current)
+	local t = stanza.attr.type;
+	if not t then
+		for _, child in pairs(stanza.tags) do
+			if child.name == "c" and child.attr.xmlns == "http://jabber.org/protocol/caps" then
+				local attr = child.attr;
+				if attr.hash then -- new caps
+					if attr.hash == 'sha-1' and attr.node and attr.ver then return attr.ver, attr.node.."#"..attr.ver; end
+				else -- legacy caps
+					if attr.node and attr.ver then return attr.node.."#"..attr.ver.."#"..(attr.ext or ""), attr.node.."#"..attr.ver; end
+				end
+				return; -- bad caps format
+			end
 		end
+	elseif t == "unavailable" or t == "error" then
+		return;
 	end
+	return current; -- no caps, could mean caps optimization, so return current
 end
 
 module:hook("presence/bare", function(event)
@@ -56,18 +71,18 @@
 	local bare = jid_bare(stanza.attr.from);
 	local item = load_roster(jid_split(user))[bare];
 	if not stanza.attr.to or (item and (item.subscription == 'from' or item.subscription == 'both')) then
-		local t = stanza.attr.type;
 		local recipient = stanza.attr.from;
-		if t == "unavailable" or t == "error" then
+		local current = recipients[user] and recipients[user][recipient];
+		local hash = get_caps_hash_from_presence(stanza, current);
+		if current == hash then return; end
+		if not hash then
 			if recipients[user] then recipients[user][recipient] = nil; end
-		elseif not t then
+		else
 			recipients[user] = recipients[user] or {};
-			if not recipients[user][recipient] then
-				recipients[user][recipient] = true;
-				for node, message in pairs(data[user] or {}) do
-					message.attr.to = stanza.attr.from;
-					origin.send(message);
-				end
+			recipients[user][recipient] = hash;
+			for node, message in pairs(data[user] or NULL) do
+				message.attr.to = stanza.attr.from;
+				origin.send(message);
 			end
 		end
 	end
@@ -85,9 +100,22 @@
 				if payload then -- <item>
 					publish(session, node, payload);
 					return true;
-				end -- TODO else error
-			end -- TODO else error
+				end
+			end
 		end
 	end
 end);
 
+module:hook("iq/bare/disco", function(event)
+	local session, stanza = event.origin, event.stanza;
+	if stanza.attr.type == "result" then
+		local disco = stanza.tags[1];
+		if disco and disco.name == "query" and disco.attr.xmlns == "http://jabber.org/protocol/disco#info" then
+			-- Process disco response
+			-- TODO check if waiting for recipient's response
+			local hash; -- TODO calculate hash
+			-- TODO update hash map
+			-- TODO set recipient's data to calculated data
+		end
+	end
+end);

--- a/util/muc.lua	Sun Jun 28 08:09:57 2009 -0500
+++ b/util/muc.lua	Sun Jun 28 14:17:22 2009 +0100
@@ -165,7 +165,7 @@
 
 local function room_get_disco_info(self, stanza) end
 local function room_get_disco_items(self, stanza) end
-local function room_set_subject(room, current_nick, room, subject)
+local function room_set_subject(room, current_nick, subject)
 	-- TODO check nick's authority
 	if subject == "" then subject = nil; end
 	room._data['subject'] = subject;
@@ -315,7 +315,7 @@
 		local current_nick = self._jid_nick[stanza.attr.from];
 		if current_nick then
 			stanza.attr.to = current_nick;
-			room_handle_to_occupant(origin, stanza);
+			room_handle_to_occupant(self, origin, stanza);
 			stanza.attr.to = to;
 		elseif type ~= "error" and type ~= "result" then
 			origin.send(st.error_reply(stanza, "cancel", "service-unavailable"));

--- a/util/stanza.lua	Sun Jun 28 08:09:57 2009 -0500
+++ b/util/stanza.lua	Sun Jun 28 14:17:22 2009 +0100
@@ -24,6 +24,7 @@
 local unpack        =        unpack;
 local s_gsub        =   string.gsub;
 local s_char        =   string.char;
+local s_find        =   string.find;
 local os            =            os;
 
 local do_pretty_printing = not os.getenv("WINDIR");
@@ -122,27 +123,33 @@
 	return function(str) return (s_gsub(str, "['&<>\"]", escape_table)); end
 end)();
 local function _dostring(t, buf, self, xml_escape)
-	local nsid, ns, attrk = 0;
-	t_insert(buf, "<"..t.name);
+	local nsid = 0;
+	local name = t.name
+	t_insert(buf, "<"..name);
 	for k, v in pairs(t.attr) do
-		ns, attrk = s_match(k, "^([^|]+)|(.+)$");
-		if ns then
+		if s_find(k, "|", 1, true) then
+			local ns, attrk = s_match(k, "^([^|]+)|(.+)$");
 			nsid = nsid + 1;
 			t_insert(buf, " xmlns:ns"..nsid.."='"..xml_escape(ns).."' ".."ns"..nsid..":"..attrk.."='"..xml_escape(v).."'");
 		else
 			t_insert(buf, " "..k.."='"..xml_escape(v).."'");
 		end
 	end
-	t_insert(buf, ">");
-	for n=1,#t do
-		local child = t[n];
-		if child.name then
-			self(child, buf, self, xml_escape);
-		else
-			t_insert(buf, xml_escape(child));
+	local len = #t;
+	if len == 0 then
+		t_insert(buf, "/>");
+	else
+		t_insert(buf, ">");
+		for n=1,len do
+			local child = t[n];
+			if child.name then
+				self(child, buf, self, xml_escape);
+			else
+				t_insert(buf, xml_escape(child));
+			end
 		end
+		t_insert(buf, "</"..name..">");
 	end
-	t_insert(buf, "</"..t.name..">");
 end
 function stanza_mt.__tostring(t)
 	local buf = {};