Skip to content

Commit c893c39

Browse files
Do not wipe out current session when calling findSession with other session IDs
1 parent 14dfbc8 commit c893c39

File tree

1 file changed

+1
-13
lines changed

1 file changed

+1
-13
lines changed

src/main/java/com/orangefunction/tomcat/redissessions/RedisSessionManager.java

+1-13
Original file line numberDiff line numberDiff line change
@@ -422,26 +422,14 @@ public Session findSession(String id) throws IOException {
422422
RedisSession session = null;
423423

424424
if (null == id) {
425-
currentSessionIsPersisted.set(false);
426-
currentSession.set(null);
427-
currentSessionSerializationMetadata.set(null);
428-
currentSessionId.set(null);
425+
return null;
429426
} else if (id.equals(currentSessionId.get())) {
430427
session = currentSession.get();
431428
} else {
432429
byte[] data = loadSessionDataFromRedis(id);
433430
if (data != null) {
434431
DeserializedSessionContainer container = sessionFromSerializedData(id, data);
435432
session = container.session;
436-
currentSession.set(session);
437-
currentSessionSerializationMetadata.set(container.metadata);
438-
currentSessionIsPersisted.set(true);
439-
currentSessionId.set(id);
440-
} else {
441-
currentSessionIsPersisted.set(false);
442-
currentSession.set(null);
443-
currentSessionSerializationMetadata.set(null);
444-
currentSessionId.set(null);
445433
}
446434
}
447435

0 commit comments

Comments
 (0)