From acec34136451e1a37033c2395802927528ad35cb Mon Sep 17 00:00:00 2001 From: pabloFuente Date: Fri, 23 Apr 2021 12:41:51 +0200 Subject: [PATCH] openvidu-server: fix session close up order --- .../java/io/openvidu/server/core/SessionManager.java | 11 ++++------- 1 file changed, 4 insertions(+), 7 deletions(-) diff --git a/openvidu-server/src/main/java/io/openvidu/server/core/SessionManager.java b/openvidu-server/src/main/java/io/openvidu/server/core/SessionManager.java index 0c9d23d7..3d46cfd0 100644 --- a/openvidu-server/src/main/java/io/openvidu/server/core/SessionManager.java +++ b/openvidu-server/src/main/java/io/openvidu/server/core/SessionManager.java @@ -609,21 +609,18 @@ public abstract class SessionManager { final String mediaNodeId = session.getMediaNodeId(); if (session.close(reason)) { - - this.cleanCollections(session.getSessionId()); - - log.info("Session '{}' removed and closed", session.getSessionId()); - try { sessionEventsHandler.onSessionClosed(session.getSessionId(), reason); } catch (Exception e) { log.error("Error recording 'sessionDestroyed' event for session {}: {} - {}", session.getSessionId(), e.getClass().getName(), e.getMessage()); } - } else { - this.cleanCollections(session.getSessionId()); } + this.cleanCollections(session.getSessionId()); + + log.info("Session '{}' removed and closed", session.getSessionId()); + if (mediaNodeId != null) { this.quarantineKiller.dropMediaNode(mediaNodeId); }