From a9bbcf8700bf203e0cacd432ccae2ed734f99b77 Mon Sep 17 00:00:00 2001 From: dahn Date: Tue, 26 Apr 2022 02:58:41 +0200 Subject: [PATCH] remove superfluent counter and fix log message (#6313) * remove superfluent counter and fix log message * Apply suggestions from code review Co-authored-by: Pearl Dsilva Co-authored-by: Pearl Dsilva --- .../main/java/com/cloud/api/ApiSessionListener.java | 13 +++---------- 1 file changed, 3 insertions(+), 10 deletions(-) diff --git a/server/src/main/java/com/cloud/api/ApiSessionListener.java b/server/src/main/java/com/cloud/api/ApiSessionListener.java index 6bb853d76c1..7f99bf596d6 100644 --- a/server/src/main/java/com/cloud/api/ApiSessionListener.java +++ b/server/src/main/java/com/cloud/api/ApiSessionListener.java @@ -33,17 +33,12 @@ public class ApiSessionListener implements HttpSessionListener, ServletRequestLi public static final Logger LOGGER = Logger.getLogger(ApiSessionListener.class.getName()); private static final String ATTRIBUTE_NAME = "SessionCounter"; private static Map sessions = new ConcurrentHashMap<>(); - private static long sessionCount; - - public ApiSessionListener() { - sessionCount = 0; - } /** * @return the internal adminstered session count */ public static long getSessionCount() { - return sessionCount; + return sessions.size(); } /** @@ -57,17 +52,15 @@ public class ApiSessionListener implements HttpSessionListener, ServletRequestLi synchronized (this) { HttpSession session = event.getSession(); sessions.put(session, event.getSource()); - sessionCount++; } - LOGGER.debug("Sessions count: " + sessions); + LOGGER.debug("Sessions count: " + getSessionCount()); } public void sessionDestroyed(HttpSessionEvent event) { LOGGER.debug("Session destroyed by Id : " + event.getSession().getId() + " , session: " + event.getSession().toString() + " , source: " + event.getSource().toString() + " , event: " + event.toString()); synchronized (this) { - sessionCount--; sessions.remove(event.getSession()); } - LOGGER.debug("Sessions count: " + sessions); + LOGGER.debug("Sessions count: " + getSessionCount()); } @Override