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.
This commit is contained in:
Kim Alvefur 2021-12-22 18:47:45 +01:00
parent b332a2aa42
commit 2f3b28380a

View file

@ -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);