From 8991b7f452766a8e5a422aed3760ac27fc56f290 Mon Sep 17 00:00:00 2001 From: Joakim Erdfelt Date: Tue, 14 Sep 2021 15:37:09 -0500 Subject: [PATCH] Issue #6752 - Extensible DefaultSessionCache map implementation Addressing changes requested from review Signed-off-by: Joakim Erdfelt --- .../jetty/server/session/DefaultSessionCache.java | 14 +++++++++++--- 1 file changed, 11 insertions(+), 3 deletions(-) diff --git a/jetty-server/src/main/java/org/eclipse/jetty/server/session/DefaultSessionCache.java b/jetty-server/src/main/java/org/eclipse/jetty/server/session/DefaultSessionCache.java index d67cca276b68..acef8b434b4d 100644 --- a/jetty-server/src/main/java/org/eclipse/jetty/server/session/DefaultSessionCache.java +++ b/jetty-server/src/main/java/org/eclipse/jetty/server/session/DefaultSessionCache.java @@ -13,6 +13,7 @@ package org.eclipse.jetty.server.session; +import java.util.Objects; import java.util.concurrent.ConcurrentHashMap; import java.util.concurrent.ConcurrentMap; import java.util.function.Function; @@ -52,12 +53,12 @@ public DefaultSessionCache(SessionHandler manager) /** * @param manager The SessionHandler related to this SessionCache - * @param sessionMap The session map implementation to use + * @param sessions The session map implementation to use */ - public DefaultSessionCache(SessionHandler manager, ConcurrentMap sessionMap) + public DefaultSessionCache(SessionHandler manager, ConcurrentMap sessions) { super(manager); - this._sessions = sessionMap; + _sessions = Objects.requireNonNull(sessions, "Session Map may not be null"); } /** @@ -194,4 +195,11 @@ public boolean doReplace(String id, Session oldValue, Session newValue) { return _sessions.replace(id, oldValue, newValue); } + + @Override + protected void doStart() throws Exception + { + Objects.requireNonNull(_sessions, "Session Map may not be null"); + super.doStart(); + } }