瀏覽代碼

Move log statement

Moved after removeSessionInformation() is called for comprehensible output to log file. The log statements are now written in correct order. Before the change, this could be confusing when debugging an application, since it seemed that the registered session was immediately removed when only looking at the logs.
joos-edia 6 年之前
父節點
當前提交
0e7462272f

+ 4 - 4
core/src/main/java/org/springframework/security/core/session/SessionRegistryImpl.java

@@ -120,15 +120,15 @@ public class SessionRegistryImpl implements SessionRegistry,
 		Assert.hasText(sessionId, "SessionId required as per interface contract");
 		Assert.hasText(sessionId, "SessionId required as per interface contract");
 		Assert.notNull(principal, "Principal required as per interface contract");
 		Assert.notNull(principal, "Principal required as per interface contract");
 
 
+		if (getSessionInformation(sessionId) != null) {
+			removeSessionInformation(sessionId);
+		}
+
 		if (logger.isDebugEnabled()) {
 		if (logger.isDebugEnabled()) {
 			logger.debug("Registering session " + sessionId + ", for principal "
 			logger.debug("Registering session " + sessionId + ", for principal "
 					+ principal);
 					+ principal);
 		}
 		}
 
 
-		if (getSessionInformation(sessionId) != null) {
-			removeSessionInformation(sessionId);
-		}
-
 		sessionIds.put(sessionId,
 		sessionIds.put(sessionId,
 				new SessionInformation(principal, sessionId, new Date()));
 				new SessionInformation(principal, sessionId, new Date()));