[
https://issues.apache.org/jira/browse/JSEC-34?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Les Hazlewood resolved JSEC-34.
-------------------------------
Resolution: Fixed
Assignee: Les Hazlewood
Fixed by making the path "/" if the cookieAttribute.path is null and
request.getContextPath() returns null or the empty string. 2 test cases added
for verification.
http://svn.apache.org/viewvc?view=rev&revision=736234
> Remember-me cookie path wrong if servlet context path is root ("/")
> -------------------------------------------------------------------
>
> Key: JSEC-34
> URL: https://issues.apache.org/jira/browse/JSEC-34
> Project: JSecurity
> Issue Type: Bug
> Components: Web
> Affects Versions: 0.9-RC1
> Reporter: Peter Ledbrook
> Assignee: Les Hazlewood
> Fix For: 1.0
>
>
> If a web application is installed in the root context of a servlet container,
> the remember-me cookie defaults to a value of "" (because that's what
> {{request.getContextPath()}} returns). Unfortunately, this appears to attach
> the cookie to the current request's URL rather than the required root, "/".
> The offending code is in {{org.jsecurity.web.attr.CookieAttribute}} where the
> {{onStoreValue()}} method does this:
> {code}
> String path = getPath() != null ? getPath() : request.getContextPath()
> {code}
> We should either always append '/' to the context path or set the cookie path
> to "/" if the request context path is an empty string.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
