diff mod_pubsub_serverinfo/mod_pubsub_serverinfo.lua @ 5815:77c5709bd57a

mod_pubsub_serverinfo: Add explicit xmlns to all pubsub tags This helps when routing between hosts on the same server, where the namespace normalization is not handled by default.
author Matthew Wild <mwild1@gmail.com>
date Mon, 08 Jan 2024 15:53:21 +0000
parents fa28fc2ee465
children 45d0802d0787
line wrap: on
line diff
--- a/mod_pubsub_serverinfo/mod_pubsub_serverinfo.lua	Mon Jan 08 15:52:22 2024 +0000
+++ b/mod_pubsub_serverinfo/mod_pubsub_serverinfo.lua	Mon Jan 08 15:53:21 2024 +0000
@@ -9,6 +9,7 @@
 local publication_interval = module:get_option(module.name .. "_publication_interval") or 300;
 local cache_ttl = module:get_option(module.name .. "_cache_ttl") or 3600;
 
+local xmlns_pubsub = "http://jabber.org/protocol/pubsub";
 
 function module.load()
 	discover_node():next(
@@ -72,9 +73,9 @@
 -- Returns a promise of a boolean
 function create_node()
 	local request = st.iq({ type = "set", to = service, from = actor, id = new_id() })
-		:tag("pubsub", { xmlns = "http://jabber.org/protocol/pubsub" })
-			:tag("create", { node = node }):up()
-			:tag("configure")
+		:tag("pubsub", { xmlns = xmlns_pubsub })
+			:tag("create", { node = node, xmlns = xmlns_pubsub }):up()
+			:tag("configure", { xmlns = xmlns_pubsub })
 				:tag("x", { xmlns = "jabber:x:data", type = "submit" })
 					:tag("field", { var = "FORM_TYPE", type = "hidden"})
 						:text_tag("value", "http://jabber.org/protocol/pubsub#node_config")
@@ -102,8 +103,8 @@
 -- Returns a promise of a boolean
 function delete_node()
 	local request = st.iq({ type = "set", to = service, from = actor, id = new_id() })
-		:tag("pubsub", { xmlns = "http://jabber.org/protocol/pubsub" })
-			:tag("delete", { node = node });
+		:tag("pubsub", { xmlns = xmlns_pubsub })
+			:tag("delete", { node = node, xmlns = xmlns_pubsub });
 
 	module:log("debug", "Sending request to delete pub/sub node '%s' at %s", node, service)
 	return module:send_iq(request):next(
@@ -173,9 +174,9 @@
 
 	-- Build the publication stanza.
 	local request = st.iq({ type = "set", to = service, from = actor, id = new_id() })
-		:tag("pubsub", { xmlns = "http://jabber.org/protocol/pubsub" })
-			:tag("publish", { node = node, xmlns = "http://jabber.org/protocol/pubsub" })
-				:tag("item", { id = "current", xmlns = "http://jabber.org/protocol/pubsub" })
+		:tag("pubsub", { xmlns = xmlns_pubsub })
+			:tag("publish", { node = node, xmlns = xmlns_pubsub })
+				:tag("item", { id = "current", xmlns = xmlns_pubsub })
 					:tag("serverinfo", { xmlns = "urn:xmpp:serverinfo:0" })
 
 	request:tag("domain", { name = local_domain })