mirrors/prosody
1
0
Fork 0
mirror of https://github.com/bjc/prosody.git synced 2025-04-04 05:37:39 +03:00
Code Issues Projects Releases Packages Wiki Activity Actions

mod_mam: Avoid storing bounces for messages from the bare account (thanks Ge0rG)

Browse source 
This should rule out error replies to PEP notifications, which come from
the account bare JID, and would be reflected back to it if they can't be
delivered for some reason, e.g. s2s problems.
This commit is contained in:
Kim Alvefur 2021-12-10 22:25:34 +01:00
parent 449311b2d1
commit ab4991e829
1 changed files with 12 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

14
plugins/mod_mam/mod_mam.lua
View file

@ -24,6 +24,7 @@ local prefs_to_stanza = module:require"mamprefsxml".tostanza;
local prefs_from_stanza = module:require"mamprefsxml".fromstanza;
local jid_bare = require "util.jid".bare;
local jid_split = require "util.jid".split;
local jid_resource = require "util.jid".resource;
local jid_prepped_split = require "util.jid".prepped_split;
local dataform = require "util.dataforms".new;
local get_form_type = require "util.dataforms".get_type;
@ -320,8 +321,17 @@ local function should_store(stanza, c2s) --> boolean, reason: string
return false, "headline";
end
if st_type == "error" and not c2s then
-- Store delivery failure notifications so you know if your own messages were not delivered
return true, "bounce";
-- Errors not sent sent from a local client
-- Why would a client send an error anyway?
if jid_resource(stanza.attr.to) then
-- Store delivery failure notifications so you know if your own messages
-- were not delivered.
return true, "bounce";
else
-- Skip errors for messages that come from your account, such as PEP
-- notifications.
return false, "bounce";
end
end
if st_type == "groupchat" then
-- MUC messages always go to the full JID, usually archived by the MUC