remm 2004/01/28 10:17:48
Modified: catalina/src/share/org/apache/coyote/tomcat5
CoyoteAdapter.java
Log:
- Consider all session ids sent as cookies until one is valid.
- Fix bug 26492 (which likely has duplicates in 4.1.x land).
- Controvertial change: don't create cookie objects from sessionId cookies.
Since we're hiding the sessionId from the URL, it IMO should be the same with
cookies. This is faster and IMO cleaner.
Revision Changes Path
1.17 +11 -4
jakarta-tomcat-catalina/catalina/src/share/org/apache/coyote/tomcat5/CoyoteAdapter.java
Index: CoyoteAdapter.java
===================================================================
RCS file:
/home/cvs/jakarta-tomcat-catalina/catalina/src/share/org/apache/coyote/tomcat5/CoyoteAdapter.java,v
retrieving revision 1.16
retrieving revision 1.17
diff -u -r1.16 -r1.17
--- CoyoteAdapter.java 10 Jan 2004 20:39:50 -0000 1.16
+++ CoyoteAdapter.java 28 Jan 2004 18:17:48 -0000 1.17
@@ -418,7 +418,14 @@
log.debug(" Requested cookie session id is " +
((HttpServletRequest) request.getRequest())
.getRequestedSessionId());
+ } else {
+ if (!request.isRequestedSessionIdValid()) {
+ // Replace the session id until one is valid
+ request.setRequestedSessionId
+ (scookie.getValue().toString());
+ }
}
+ continue;
}
try {
Cookie cookie = new Cookie(scookie.getName().toString(),
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
