From 2f3b28380a3c68e48bd0e1a159f6bcdee7cad19f Mon Sep 17 00:00:00 2001 From: Kim Alvefur Date: Wed, 22 Dec 2021 18:47:45 +0100 Subject: mod_smacks: Use more compact resumption tokens UUID seems like insane overkill for something user-scoped and not security-sensitive. All that is needed is to avoid conflicts among what should be relatively long-lived sessions. --- plugins/mod_smacks.lua | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/plugins/mod_smacks.lua b/plugins/mod_smacks.lua index c52b268c..677b69fc 100644 --- a/plugins/mod_smacks.lua +++ b/plugins/mod_smacks.lua @@ -21,7 +21,7 @@ local jid = require "util.jid"; local smqueue = require "util.smqueue"; local st = require "util.stanza"; local timer = require "util.timer"; -local uuid_generate = require "util.uuid".generate; +local new_id = require "util.id".short; local watchdog = require "util.watchdog"; local sessionmanager = require "core.sessionmanager"; @@ -259,7 +259,7 @@ function handle_enable(session, stanza, xmlns_sm) local resume_token; local resume = stanza.attr.resume; if resume == "true" or resume == "1" then - resume_token = uuid_generate(); + resume_token = new_id(); session_registry[jid.join(session.username, session.host, resume_token)] = session; session.resumption_token = resume_token; resume_max = tostring(resume_timeout); -- cgit v1.2.3