From 568aa794417fc145a5174767b22195592f1a7fd1 Mon Sep 17 00:00:00 2001 From: Kim Alvefur Date: Fri, 27 May 2022 12:05:47 +0200 Subject: mod_smacks: Fix bounce of stanzas directed to full JID on unclean disconnect Fixes #1758 Introduced in 1ea01660c79a In e62025f949f9 to and from was inverted since it changed from acting on a reply to acting on the original stanza (or a clone thereof) Unsure of the purpose of this check, you don't usually send stanzas to your own full JID. Perhaps guarding against routing loops? The check was present in the original commit of mod_smacks, prosody-modules rev 9a7671720dec --- plugins/mod_smacks.lua | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/plugins/mod_smacks.lua b/plugins/mod_smacks.lua index 33c90ee9..3573d74a 100644 --- a/plugins/mod_smacks.lua +++ b/plugins/mod_smacks.lua @@ -418,7 +418,7 @@ local function handle_unacked_stanzas(session) session.outgoing_stanza_queue = nil; for stanza in queue._queue:consume() do if not module:fire_event("delivery/failure", { session = session, stanza = stanza }) then - if stanza.attr.type ~= "error" and stanza.attr.to ~= session.full_jid then + if stanza.attr.type ~= "error" and stanza.attr.from ~= session.full_jid then local reply = st.error_reply(stanza, "cancel", "recipient-unavailable"); core_process_stanza(session, reply); end -- cgit v1.2.3