From 2f3b28380a3c68e48bd0e1a159f6bcdee7cad19f Mon Sep 17 00:00:00 2001
From: Kim Alvefur <zash@zash.se>
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(-)

(limited to 'plugins')

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