Do not wipe out current session when calling findSession with other s…

…ession IDs
jcoleman · Feb 18, 2015 · c893c39 · c893c39
1 parent 14dfbc8
commit c893c39
Showing 1 changed file with 1 addition and 13 deletions.
diff --git a/src/main/java/com/orangefunction/tomcat/redissessions/RedisSessionManager.java b/src/main/java/com/orangefunction/tomcat/redissessions/RedisSessionManager.java
@@ -422,26 +422,14 @@ public Session findSession(String id) throws IOException {
     RedisSession session = null;
 
     if (null == id) {
-      currentSessionIsPersisted.set(false);
-      currentSession.set(null);
-      currentSessionSerializationMetadata.set(null);
-      currentSessionId.set(null);
+      return null;
     } else if (id.equals(currentSessionId.get())) {
       session = currentSession.get();
     } else {
       byte[] data = loadSessionDataFromRedis(id);
       if (data != null) {
         DeserializedSessionContainer container = sessionFromSerializedData(id, data);
         session = container.session;
-        currentSession.set(session);
-        currentSessionSerializationMetadata.set(container.metadata);
-        currentSessionIsPersisted.set(true);
-        currentSessionId.set(id);
-      } else {
-        currentSessionIsPersisted.set(false);
-        currentSession.set(null);
-        currentSessionSerializationMetadata.set(null);
-        currentSessionId.set(null);
       }
     }