# HG changeset patch # User John Livingston # Date 1722252415 -7200 # Node ID bfb8d7b53954d978ff37221e91ab2dbf71b0798c # Parent a316fee71bed1086728b2b98f32dc2b7bf278eef mod_muc_anonymize_moderation_actions: fix XEP-0425 v0.2.1 compliance. diff -r a316fee71bed -r bfb8d7b53954 mod_muc_anonymize_moderation_actions/mod_muc_anonymize_moderation_actions.lua --- 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