# HG changeset patch
# User Matthew Wild <mwild1@gmail.com>
# Date 1672331100 0
# Node ID a4ac16e5b6558996d2e44ec0ed21b40932a5513e
# Parent  b9577117143973f0a5c617871431bc40d8886884
sessionmanager: Fire event before retiring old session

This allows for modules to update fields, which is generally better than
maintaining this hard-coded list of transferable properties here.

diff -r b95771171439 -r a4ac16e5b655 core/sessionmanager.lua
--- a/core/sessionmanager.lua	Thu Dec 29 16:23:08 2022 +0000
+++ b/core/sessionmanager.lua	Thu Dec 29 16:25:00 2022 +0000
@@ -126,14 +126,15 @@
 	-- Inform xmppstream of the new session (passed to its callbacks)
 	to_session.stream:set_session(to_session);
 
-	-- Retire the session we've pulled from, to avoid two sessions on the same connection
-	retire_session(from_session);
-
+	-- Notify modules, allowing them to copy further fields or update state
 	prosody.events.fire_event("c2s-session-updated", {
 		session = to_session;
 		from_session = from_session;
 		replaced_conn = replaced_conn;
 	});
+
+	-- Retire the session we've pulled from, to avoid two sessions on the same connection
+	retire_session(from_session);
 end
 
 local function destroy_session(session, err)