You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by ma...@apache.org on 2006/12/14 03:07:32 UTC
svn commit: r486929 - in /tomcat/container/tc5.5.x:
catalina/src/share/org/apache/catalina/connector/Request.java
catalina/src/share/org/apache/catalina/session/PersistentManagerBase.java
webapps/docs/changelog.xml
Author: markt
Date: Wed Dec 13 18:07:32 2006
New Revision: 486929
URL: http://svn.apache.org/viewvc?view=rev&rev=486929
Log:
Revert fix for 40551 as per discussion on dev list.
Modified:
tomcat/container/tc5.5.x/catalina/src/share/org/apache/catalina/connector/Request.java
tomcat/container/tc5.5.x/catalina/src/share/org/apache/catalina/session/PersistentManagerBase.java
tomcat/container/tc5.5.x/webapps/docs/changelog.xml
Modified: tomcat/container/tc5.5.x/catalina/src/share/org/apache/catalina/connector/Request.java
URL: http://svn.apache.org/viewvc/tomcat/container/tc5.5.x/catalina/src/share/org/apache/catalina/connector/Request.java?view=diff&rev=486929&r1=486928&r2=486929
==============================================================================
--- tomcat/container/tc5.5.x/catalina/src/share/org/apache/catalina/connector/Request.java (original)
+++ tomcat/container/tc5.5.x/catalina/src/share/org/apache/catalina/connector/Request.java Wed Dec 13 18:07:32 2006
@@ -2207,12 +2207,6 @@
if ((session != null) && !session.isValid())
session = null;
if (session != null) {
- if(!requestedSessionId.equals(session.getId())) {
- Cookie cookie = new Cookie(Globals.SESSION_COOKIE_NAME,
- session.getIdInternal());
- configureSessionCookie(cookie);
- response.addCookie(cookie);
- }
session.access();
return (session);
}
Modified: tomcat/container/tc5.5.x/catalina/src/share/org/apache/catalina/session/PersistentManagerBase.java
URL: http://svn.apache.org/viewvc/tomcat/container/tc5.5.x/catalina/src/share/org/apache/catalina/session/PersistentManagerBase.java?view=diff&rev=486929&r1=486928&r2=486929
==============================================================================
--- tomcat/container/tc5.5.x/catalina/src/share/org/apache/catalina/session/PersistentManagerBase.java (original)
+++ tomcat/container/tc5.5.x/catalina/src/share/org/apache/catalina/session/PersistentManagerBase.java Wed Dec 13 18:07:32 2006
@@ -772,20 +772,6 @@
}
} else {
session = store.load(id);
-
- String jvmRoute = getJvmRoute();
- if (jvmRoute != null && session != null) {
- String requestJvmRoute = null;
- int index = id.indexOf(".");
- if (index > 0) {
- requestJvmRoute = id.substring(index + 1, id.length());
- }
- if (requestJvmRoute != null &&
- !requestJvmRoute.equals(jvmRoute)) {
- id = id.substring(0, index) + "." + jvmRoute;
- session.setId(id);
- }
- }
}
} catch (ClassNotFoundException e) {
log.error(sm.getString("persistentManager.deserializeError", id, e));
Modified: tomcat/container/tc5.5.x/webapps/docs/changelog.xml
URL: http://svn.apache.org/viewvc/tomcat/container/tc5.5.x/webapps/docs/changelog.xml?view=diff&rev=486929&r1=486928&r2=486929
==============================================================================
--- tomcat/container/tc5.5.x/webapps/docs/changelog.xml (original)
+++ tomcat/container/tc5.5.x/webapps/docs/changelog.xml Wed Dec 13 18:07:32 2006
@@ -86,11 +86,6 @@
Ben Clifford. (markt)
</fix>
<fix>
- <bug>40551</bug>: When restoring sessions from a PersistentManager to
- a new node ensure that the session ID is updated to reflect the new
- jvmRoute. Based on a patch by Chris Chandler. (markt)
- </fix>
- <fix>
<bug>40585</bug>: Fix parameterised constructor for o.a.juli.FileHandler
so parameters have an effect. (markt)
</fix>
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org