Changeset

4660:ce826ac8496e

mod_spam_reporting: Refactor to deduplicate code Same event firing code regardless of protocol now.
author Kim Alvefur <zash@zash.se>
date Wed, 25 Aug 2021 19:04:40 +0200
parents 4659:cc8b221f137c
children 4661:e40ed7d93d68
files mod_spam_reporting/mod_spam_reporting.lua
diffstat 1 files changed, 11 insertions(+), 14 deletions(-) [+]
line wrap: on
line diff
--- a/mod_spam_reporting/mod_spam_reporting.lua	Wed Aug 25 18:56:45 2021 +0200
+++ b/mod_spam_reporting/mod_spam_reporting.lua	Wed Aug 25 19:04:40 2021 +0200
@@ -14,28 +14,25 @@
 
 module:hook("iq-set/self/urn:xmpp:blocking:block", function (event)
 	for item in event.stanza.tags[1]:childtags("item") do
-		local report = item:get_child("report", "urn:xmpp:reporting:0");
+		local report = item:get_child("report", "urn:xmpp:reporting:0") or item:get_child("report", "urn:xmpp:reporting:1");
 		local jid = jid_prep(item.attr.jid);
 		if report and jid then
-			local report_type = report:get_child("spam") and "spam" or
-				report:get_child("abuse") and "abuse" or
-				"unknown";
-			local reason = report:get_child_text("text");
-			module:log("warn", "Received report of %s from JID '%s', %s", type, jid, reason);
-			module:fire_event(module.name.."/"..report_type.."-report", {
-				origin = event.origin, stanza = event.stanza, jid = jid,
-				item = item, report = report, reason = reason, });
-		else
-			report = item:get_child("report", "urn:xmpp:reporting:1");
-			if report and jid then
-				local report_type = "unknown";
+			local report_type, reason;
+			if report.attr.xmlns == "urn:xmpp:reporting:0" then
+				report_type = report:get_child("spam") and "spam" or report:get_child("abuse") and "abuse" or "unknown";
+				reason = report:get_child_text("text");
+			elseif report.attr.xmlns == "urn:xmpp:reporting:1" then
+				report_type = "unknown";
 				if report.attr.reason == "urn:xmpp:reporting:abuse" then
 					report_type = "abuse";
 				end
 				if report.attr.reason == "urn:xmpp:reporting:spam" then
 					report_type = "spam";
 				end
-				local reason = report:get_child_text("text");
+				reason = report:get_child_text("text");
+			end
+
+			if report_type then
 				module:log("warn", "Received report of %s from JID '%s', %s", type, jid, reason);
 				module:fire_event(module.name.."/"..report_type.."-report", {
 					origin = event.origin, stanza = event.stanza, jid = jid,