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){