Mercurial > prosody-modules
view mod_smacks_offline/mod_smacks_offline.lua @ 2491:5fbca7de2088
mod_smacks: Send out more ack requests where needed
Under some circumstances it was possible that more than "max_unacked_stanzas"
where left in the outgoing stanza queue without forcing an ack.
This could happen, when more stanzas entered the queue while the last ack request
was still unanswered.
Now the test "#queue > max_unacked_stanzas" is done upon receiving
an ack as well as when sending out stanzas, which fixes this bug.
author | tmolitor <thilo@eightysoft.de> |
---|---|
date | Sun, 12 Feb 2017 19:27:50 +0100 |
parents | b912cb8e0b3c |
children | a3693e0d26b8 |
line wrap: on
line source
local t_insert = table.insert; local mod_smacks = module:depends"smacks" local function store_unacked_stanzas(session) local queue = session.outgoing_stanza_queue; local replacement_queue = {}; session.outgoing_stanza_queue = replacement_queue; for _, stanza in ipairs(queue) do if stanza.name == "message" and stanza.attr.xmlns == nil and ( stanza.attr.type == "chat" or ( stanza.attr.type or "normal" ) == "normal" ) then module:fire_event("message/offline/handle", { origin = session, stanza = stanza } ) else t_insert(replacement_queue, stanza); end end end local handle_unacked_stanzas = mod_smacks.handle_unacked_stanzas; local host_sessions = prosody.hosts[module.host].sessions; mod_smacks.handle_unacked_stanzas = function (session) if session.username then local sessions = host_sessions[session.username].sessions; if next(sessions) == session.resource and next(sessions, session.resource) == nil then store_unacked_stanzas(session) end end return handle_unacked_stanzas(session); end function module.unload() mod_smacks.handle_unacked_stanzas = handle_unacked_stanzas; end