[jboss-cvs] JBossAS SVN: r66405 - trunk/tomcat/src/main/org/jboss/web/tomcat/service/session.

[jboss-cvs-commits at lists.jboss.org]

Author: remy.maucherat at jboss.com
Date: 2007-10-24 19:29:12 -0400 (Wed, 24 Oct 2007)
New Revision: 66405

Modified:
   trunk/tomcat/src/main/org/jboss/web/tomcat/service/session/JBossManager.java
Log:
- Not setting setMaxAge(-1) seems like a mistake (it will not be a
  session cookie).

Modified: trunk/tomcat/src/main/org/jboss/web/tomcat/service/session/JBossManager.java
===================================================================
--- trunk/tomcat/src/main/org/jboss/web/tomcat/service/session/JBossManager.java	2007-10-24 23:23:10 UTC (rev 66404)
+++ trunk/tomcat/src/main/org/jboss/web/tomcat/service/session/JBossManager.java	2007-10-24 23:29:12 UTC (rev 66405)
@@ -320,9 +320,8 @@
             {
                log_.trace("Setting cookie with session id:" + sessionId + " & name:" + Globals.SESSION_COOKIE_NAME);
             }
-
             newCookie.setPath("/");
-
+            newCookie.setMaxAge(-1);
             if (connector.getSecure()) {
                 newCookie.setSecure(true);
             }