changeset 5946:bfb8d7b53954

mod_muc_anonymize_moderation_actions: fix XEP-0425 v0.2.1 compliance.
author John Livingston <git@john-livingston.fr>
date Mon, 29 Jul 2024 13:26:55 +0200
parents a316fee71bed
children 22effe87fed2
files mod_muc_anonymize_moderation_actions/mod_muc_anonymize_moderation_actions.lua
diffstat 1 files changed, 10 insertions(+), 2 deletions(-) [+]
line wrap: on
line diff
--- a/mod_muc_anonymize_moderation_actions/mod_muc_anonymize_moderation_actions.lua	Mon Jul 29 13:11:43 2024 +0200
+++ b/mod_muc_anonymize_moderation_actions/mod_muc_anonymize_moderation_actions.lua	Mon Jul 29 13:26:55 2024 +0200
@@ -48,7 +48,11 @@
   local moderated = announcement:find("{urn:xmpp:fasten:0}apply-to/{urn:xmpp:message-moderate:0}moderated");
   if moderated then
     module:log("debug", "We must anonymize the moderation announcement for stanza %s", event.stanza_id);
-    moderated.attr.by = nil;
+    -- FIXME: XEP-0245 has changed.
+    -- urn:xmpp:message-moderate:0 requires a "by" attribute
+    -- urn:xmpp:message-moderate:1 do not require the "by" attribute
+    -- So, for now, settings the room jid, as we only implement urn:xmpp:message-moderate:0.
+    moderated.attr.by = room.jid;
     moderated:remove_children("occupant-id", "urn:xmpp:occupant-id:0");
   end
 
@@ -56,7 +60,11 @@
     local moderated = tombstone:get_child("moderated", "urn:xmpp:message-moderate:0");
     if moderated then
       module:log("debug", "We must anonymize the moderation tombstone for stanza %s", event.stanza_id);
-      moderated.attr.by = nil;
+      -- FIXME: XEP-0245 has changed.
+      -- urn:xmpp:message-moderate:0 requires a "by" attribute
+      -- urn:xmpp:message-moderate:1 do not require the "by" attribute
+      -- So, for now, settings the room jid, as we only implement urn:xmpp:message-moderate:0.
+      moderated.attr.by = room.jid;
       moderated:remove_children("occupant-id", "urn:xmpp:occupant-id:0");
     end
   end