billbarker    2002/12/31 22:24:26

  Modified:    catalina/src/share/org/apache/catalina/session
                        ManagerBase.java
  Log:
  Set the sessionId property only once.
  
  It seems that this is an issue for the community, so let's get it right.
  
  Submitted by: Phil Steitz [EMAIL PROTECTED]
  
  Revision  Changes    Path
  1.17      +4 -6      
jakarta-tomcat-4.0/catalina/src/share/org/apache/catalina/session/ManagerBase.java
  
  Index: ManagerBase.java
  ===================================================================
  RCS file: 
/home/cvs/jakarta-tomcat-4.0/catalina/src/share/org/apache/catalina/session/ManagerBase.java,v
  retrieving revision 1.16
  retrieving revision 1.17
  diff -u -r1.16 -r1.17
  --- ManagerBase.java  31 Dec 2002 03:45:48 -0000      1.16
  +++ ManagerBase.java  1 Jan 2003 06:24:26 -0000       1.17
  @@ -584,7 +584,6 @@
           // @todo Move appending of jvmRoute generateSessionId()???
           if (jvmRoute != null) {
               sessionId += '.' + jvmRoute;
  -            session.setId(sessionId);
           }
           synchronized (sessions) {
               while (sessions.get(sessionId) != null){ // Guarantee uniqueness
  @@ -592,7 +591,6 @@
                   // @todo Move appending of jvmRoute generateSessionId()???
                   if (jvmRoute != null) {
                       sessionId += '.' + jvmRoute;
  -                    session.setId(sessionId);
                   }
               }
           }
  
  
  

--
To unsubscribe, e-mail:   <mailto:[EMAIL PROTECTED]>
For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>

Reply via email to