Index: lams_common/src/java/org/lamsfoundation/lams/web/session/SessionManager.java =================================================================== diff -u -rf053892fcb0fb6e1163690ee40a518d2e8201ee2 -r213c2e221c577adb9986040687a5119ecadaa6a6 --- lams_common/src/java/org/lamsfoundation/lams/web/session/SessionManager.java (.../SessionManager.java) (revision f053892fcb0fb6e1163690ee40a518d2e8201ee2) +++ lams_common/src/java/org/lamsfoundation/lams/web/session/SessionManager.java (.../SessionManager.java) (revision 213c2e221c577adb9986040687a5119ecadaa6a6) @@ -56,7 +56,7 @@ private static final Logger log = Logger.getLogger(SessionManager.class); - //signlton + //singleton private static SessionManager sessionMgr; //KEY: sessionId, each session will have an identified id. VALUE: SessionImpl instance, which contains //true session key/value pairs. @@ -95,7 +95,7 @@ */ public static HttpSession getSession(String sessionId){ if(sessionId == null){ - log.error("Failed on finding current system session with null sessionId"); + log.debug("Failed on finding current system session with null sessionId"); return null; } return (HttpSession) getInstance().sessionContainer.get(sessionId); @@ -106,20 +106,18 @@ //initialize a new one HttpSession session = getInstance().new SessionImpl(sessionId); getInstance().sessionContainer.put(sessionId,session); - // TODO remove this debugging - only put in to diagnose Ozgur's session problem - log.debug("SessionManager: creating new session "+sessionId); } /** * Return SessionVisitor of currentSessionId. - * An internal method, only avaliable in package. + * An internal method, only available in package. * @return */ static SessionVisitor getSessionVisitor() { return (SessionVisitor)getSession(); } /** - * An internal method, only avaliable in package. + * An internal method, only available in package. * @param currentSessionId */ static void setCurrentSessionId(String currentSessionId) { @@ -177,11 +175,6 @@ if(cookie != null){ currentSessionId = cookie.getValue(); Object obj = getSession(currentSessionId); - // TODO remove this debugging - only put in to diagnose Ozgur's session problem - if ( log.isDebugEnabled() ) { - log.debug("SessionManager: cookie is not null, currentSessionId="+currentSessionId - +" session="+obj); - } //if cookie exist, but session does not. This usually menas seesion expired. //then delete the cookie first and set it null in order to create a new one if(obj == null){