From 47c5a6d43f498805655a614ab861a0ff9d89c474 Mon Sep 17 00:00:00 2001 From: Luke Taylor Date: Tue, 28 Aug 2007 22:43:13 +0000 Subject: [PATCH] SEC-539: Renamed extractSecurityContextFromSession to readSecurityContextFromSession to emphasize that it doesn't actually modify anything (the context is still stored in the session). --- .../context/HttpSessionContextIntegrationFilter.java | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/core/src/main/java/org/acegisecurity/context/HttpSessionContextIntegrationFilter.java b/core/src/main/java/org/acegisecurity/context/HttpSessionContextIntegrationFilter.java index c3697386ae..def5211f1a 100644 --- a/core/src/main/java/org/acegisecurity/context/HttpSessionContextIntegrationFilter.java +++ b/core/src/main/java/org/acegisecurity/context/HttpSessionContextIntegrationFilter.java @@ -209,7 +209,7 @@ public class HttpSessionContextIntegrationFilter implements InitializingBean, Fi boolean httpSessionExistedAtStartOfRequest = httpSession != null; - SecurityContext contextBeforeChainExecution = extractSecurityContextFromSession(httpSession); + SecurityContext contextBeforeChainExecution = readSecurityContextFromSession(httpSession); // Make the HttpSession null, as we don't want to keep a reference to it lying // around in case chain.doFilter() invalidates it. @@ -259,7 +259,7 @@ public class HttpSessionContextIntegrationFilter implements InitializingBean, Fi } /** - * Extracts the security context from the session (if available) and returns it. + * Gets the security context from the session (if available) and returns it. *

* If the session is null, the context object is null or the context object stored in the session * is not an instance of SecurityContext it will return null. @@ -269,7 +269,7 @@ public class HttpSessionContextIntegrationFilter implements InitializingBean, Fi * * @param httpSession the session obtained from the request. */ - private SecurityContext extractSecurityContextFromSession(HttpSession httpSession) { + private SecurityContext readSecurityContextFromSession(HttpSession httpSession) { if (httpSession == null) { if (logger.isDebugEnabled()) { logger.debug("No HttpSession currently exists");