[jbossseam-issues] [JBoss JIRA] Commented: (JBSEAM-3875) Seam's Rewrite filter breaks URL-session-id-rewriting
Stephen Friedrich (JIRA)
jira-events at lists.jboss.org
Sun Jan 4 18:59:54 EST 2009
[ https://jira.jboss.org/jira/browse/JBSEAM-3875?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12444450#action_12444450 ]
Stephen Friedrich commented on JBSEAM-3875:
-------------------------------------------
Hm, the RewritingResponse extends HttpServletResponseWrapper.
I don't know how (or even if) url rewriting is working in a portlet context.
RewritingResponse gets passed HttpServletRequest and HttpServletResponse in its constructor and is already storing the request in a field anyway, so the same should probably be done for the response (avoiding the cast and call to super).
> Seam's Rewrite filter breaks URL-session-id-rewriting
> -----------------------------------------------------
>
> Key: JBSEAM-3875
> URL: https://jira.jboss.org/jira/browse/JBSEAM-3875
> Project: Seam
> Issue Type: Bug
> Components: Core
> Affects Versions: 2.1.1.GA
> Reporter: Stephen Friedrich
>
> If Seam's rewrite filter is enabled, the session id is never added as a parameter to URLs.
> That should happen when one of the HttpServletResponse.encode...() methods is called and no session cookie is found.
> However the rewrite filter adds a response wrapper that delegates all encode methods to a method that only (possibly) rewrites the URL according to defined rewrite-rules, but never actually let's the servlet container encode the url.
> Fix: In org.jboss.seam.web.RewritingResponse.encodeURL(String) add this line:
> result = ((HttpServletResponse) getResponse()).encodeURL(result);
> before returning the result.
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: https://jira.jboss.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
More information about the seam-issues
mailing list