You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@shiro.apache.org by lh...@apache.org on 2010/05/13 09:41:32 UTC

svn commit: r943834 - in /incubator/shiro/trunk/web/src/main/java/org/apache/shiro/web: mgt/CookieRememberMeManager.java session/DefaultWebSessionManager.java

Author: lhazlewood
Date: Thu May 13 07:41:32 2010
New Revision: 943834

URL: http://svn.apache.org/viewvc?rev=943834&view=rev
Log:
Fixed bug introduced by forcing cookies to default to the root path ('/').  Cookie path defaults to null now, which will use the request's context path.  User-specified explicit paths will of course override this default.

Modified:
    incubator/shiro/trunk/web/src/main/java/org/apache/shiro/web/mgt/CookieRememberMeManager.java
    incubator/shiro/trunk/web/src/main/java/org/apache/shiro/web/session/DefaultWebSessionManager.java

Modified: incubator/shiro/trunk/web/src/main/java/org/apache/shiro/web/mgt/CookieRememberMeManager.java
URL: http://svn.apache.org/viewvc/incubator/shiro/trunk/web/src/main/java/org/apache/shiro/web/mgt/CookieRememberMeManager.java?rev=943834&r1=943833&r2=943834&view=diff
==============================================================================
--- incubator/shiro/trunk/web/src/main/java/org/apache/shiro/web/mgt/CookieRememberMeManager.java (original)
+++ incubator/shiro/trunk/web/src/main/java/org/apache/shiro/web/mgt/CookieRememberMeManager.java Thu May 13 07:41:32 2010
@@ -87,7 +87,6 @@ public class CookieRememberMeManager ext
      */
     public CookieRememberMeManager() {
         Cookie cookie = new SimpleCookie(DEFAULT_REMEMBER_ME_COOKIE_NAME);
-        cookie.setPath(Cookie.ROOT_PATH);
         //One year should be long enough - most sites won't object to requiring a user to log in if they haven't visited
         //in a year:
         cookie.setMaxAge(Cookie.ONE_YEAR);

Modified: incubator/shiro/trunk/web/src/main/java/org/apache/shiro/web/session/DefaultWebSessionManager.java
URL: http://svn.apache.org/viewvc/incubator/shiro/trunk/web/src/main/java/org/apache/shiro/web/session/DefaultWebSessionManager.java?rev=943834&r1=943833&r2=943834&view=diff
==============================================================================
--- incubator/shiro/trunk/web/src/main/java/org/apache/shiro/web/session/DefaultWebSessionManager.java (original)
+++ incubator/shiro/trunk/web/src/main/java/org/apache/shiro/web/session/DefaultWebSessionManager.java Thu May 13 07:41:32 2010
@@ -51,7 +51,6 @@ public class DefaultWebSessionManager ex
 
     public DefaultWebSessionManager() {
         this.sessionIdCookie = new SimpleCookie(ShiroHttpSession.DEFAULT_SESSION_ID_NAME);
-        this.sessionIdCookie.setPath(Cookie.ROOT_PATH);
         this.sessionIdCookieEnabled = true;
     }