Bug 48345 - Session does time-out shorter than setting in web.xml when PersistentManager is used.
Summary: Session does time-out shorter than setting in web.xml when PersistentManager ...
Status: RESOLVED FIXED
Alias: None
Product: Tomcat 6
Classification: Unclassified
Component: Catalina (show other bugs)
Version: 6.0.20
Hardware: All All
: P2 normal (vote)
Target Milestone: default
Assignee: Tomcat Developers Mailing List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-12-07 02:09 UTC by Keiichi Fujino
Modified: 2010-03-06 17:49 UTC (History)
0 users



Attachments
patch for trunk. (1.45 KB, application/octet-stream)
2009-12-07 02:12 UTC, Keiichi Fujino
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Keiichi Fujino 2009-12-07 02:09:14 UTC
I am encountering the problem that Session does time-out shorter than setting in web.xml.

This is a simple scenario where the problem occurs. 

I am using PersistentManager by the following configuration.
<Context>
  <Manager className="org.apache.catalina.session.PersistentManager"
           maxIdleSwap="120">
    <Store className="org.apache.catalina.session.FileStore"/>
  </Manager>
</Context>

I am setting the session-timeout at five minutes.
<session-config>
    <session-timeout>5</session-timeout>
</session-config>

[scenario]
1. Create Session.
2. 120 seconds pass. 
   Session does swapOut. 
3. In addition, 120 seconds pass. 
   Get Session.
   Session does swapIn and access time of session in the memory is updated. 
4. In addition, 90 seconds pass.
   Get Session.

Because the access time of the session has been updated in (3),
The session should be able to be acquired.
However, the session can not be acquired. 

This cause is StoreBase#processExpires invoked between (3) and (4).
StoreBase#processExpire deletes the session saved in Store.
StoreBase#processExpires is as follows.

=====StoreBase#processExpires=====
public void processExpires() {
...
    for (int i = 0; i < keys.length; i++) {
        try {
            StandardSession session = (StandardSession) load(keys[i]);
            if (session == null) {
                continue;
            }
            if (session.isValid()) {
                continue;
            }
              ...
            if ( ( (PersistentManagerBase) manager).isLoaded( keys[i] )) {
                // recycle old backup session
                session.recycle();
            } else {
                // expire swapped out session
                session.expire();
            }
            remove(session.getIdInternal());
        } catch (Exception e) {
            ...
        }
    }
}
======

The session saved in Store is loaded, and StandardSession#isValid() is executed. 
StandardSession#expire(true) is invoked for the session that passes 
session time-out in "StandardSession#isValid()". 

StandardSession#expire(true) invokes Manager#remove(Session).
As a result, the session is deleted from the session map. 

In a word, session in memory is deleted by invalidating old session in Store. 
As a result, session does time-out shorter than setting in web.xml

I made a patch against trunk.

Best regards.
Comment 1 Keiichi Fujino 2009-12-07 02:12:06 UTC
Created attachment 24674 [details]
patch for trunk.

patch for trunk.


IMHO,
the following are necessary for backport to 6.0_trunk. 

Applies these revisions.
http://svn.apache.org/viewvc?view=revision&revision=711716
http://svn.apache.org/viewvc?view=revision&revision=711720

remove duplicated local variable timeNow.
Comment 2 Mark Thomas 2009-12-30 03:36:58 UTC
Thanks for the patch. I have committed it to trunk and proposed a variation for 6.0.x
Comment 3 Konstantin Kolinko 2010-03-06 17:49:24 UTC
The fix for this issue was applied to 6.0 and is in 6.0.23 onwards.