[ http://jira.magnolia.info/browse/MAGNOLIA-1196?page=all ]
Fabrizio Giustina resolved MAGNOLIA-1196.
-----------------------------------------
Resolution: Fixed
Assignee: Fabrizio Giustina (was: Sameer Charles)
Fixed by MAGNOLIA-1698
> Using MagnoliaFilterChain as <dispatcher>FORWARD</dispatcher> could crash the
> system
> ------------------------------------------------------------------------------------
>
> Key: MAGNOLIA-1196
> URL: http://jira.magnolia.info/browse/MAGNOLIA-1196
> Project: Magnolia
> Issue Type: Bug
> Components: core
> Affects Versions: 3.0 RC3
> Reporter: Filippo Fratoni
> Assigned To: Fabrizio Giustina
> Fix For: 3.1
>
> Attachments: FilterStubCms.java, log.txt
>
>
> /DocSpesaFrom.html is my page
> /templates/jsp/docSpesaForm.jsp is the template associated with my page
> When I call my page what happens is>
> 1)cms filter handle requestURI /DocSpesaFrom.html and forward to
> /templates/jsp/docSpesaForm.jsp
> 2)again in the cms filter. It should handle(skip in my case)
> /templates/jsp/docSpesaForm.jsp but Path.getURI(request) return
> "/DocSpesaFrom.html", we are back to step 1) end here loop starts.
> I resolved this problem introducing this few trivial lines in
> MgnlCmsFilter.doFilter():
> // Introduced to allow request forward when dispatching filters
> // If Path class find "MGNL_REQUEST_URI_DECODED" attribute, cms
> filter loop
> // If Path class miss it, the attribute gets aligned with current
> requestURI
> if (req.getAttribute("CHECK_FWD") != null){
> log.debug("Already been in Filter Cms");
> req.removeAttribute(MGNL_REQUEST_URI_DECODED);}
> else
> {
> log.debug("First time in Filter Cms");
> req.setAttribute("CHECK_FWD", new Integer(0));
> }
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://jira.magnolia.info/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
----------------------------------------------------------------
for list details see
http://documentation.magnolia.info/docs/en/editor/stayupdated.html
----------------------------------------------------------------
