Comparison

mod_watchregistrations: Convert JID list to a set, and prep before use to fix traceback on invalid JIDs (thanks sMi)

comparison

equal deleted inserted replaced

-:8c5b5ebaacb0
+:01bfb9a76660
 -- COPYING file in the source package for more information.
 --
 local host = module:get_host();
+local jid_prep = require "util.jid".prep;
-local registration_watchers = module:get_option("registration_watchers", module:get_option("admins", {}));
+local registration_watchers = module:get_option_set("registration_watchers", module:get_option("admins", {})) / jid_prep;
 local registration_notification = module:get_option("registration_notification", "User $username just registered on $host from $ip");
 local st = require "util.stanza";
 module:hook("user-registered", function (user)
 	local message = st.message{ type = "chat", from = host }
 		:tag("body")
 			:text(registration_notification:gsub("%$(%w+)", function (v)
 				return user[v] or user.session and user.session[v] or nil;
 			end));
-	for _, jid in ipairs(registration_watchers) do
+	for jid in registration_watchers do
 		module:log("debug", "Notifying %s", jid);
 		message.attr.to = jid;
 		core_route_stanza(hosts[host], message);
 	end
 end);