From 4cde54475b8b9d2b6a2ccbec8ffa5b142e746e57 Mon Sep 17 00:00:00 2001 From: Kim Alvefur Date: Fri, 17 Dec 2021 16:29:17 +0100 Subject: mod_smacks: Split log messages for when an old session exists in storage Non-existent did not seem entirely accurate for this case --- plugins/mod_smacks.lua | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/plugins/mod_smacks.lua b/plugins/mod_smacks.lua index c776b35a..c750485f 100644 --- a/plugins/mod_smacks.lua +++ b/plugins/mod_smacks.lua @@ -497,14 +497,15 @@ function handle_resume(session, stanza, xmlns_sm) local id = stanza.attr.previd; local original_session = session_registry[jid.join(session.username, session.host, id)]; if not original_session then - session.log("debug", "Tried to resume non-existent session with id %s", id); local old_session = old_session_registry:get(session.username, id); if old_session then + session.log("debug", "Tried to resume old expired session with id %s", id); session.send(st.stanza("failed", { xmlns = xmlns_sm, h = format_h(old_session.h) }) :tag("item-not-found", { xmlns = xmlns_errors }) ); old_session_registry:set(session.username, id, nil); else + session.log("debug", "Tried to resume non-existent session with id %s", id); session.send(st.stanza("failed", { xmlns = xmlns_sm }) :tag("item-not-found", { xmlns = xmlns_errors }) ); -- cgit v1.2.3