Non-Ajax requests badly handled if ViewExpiredExceptions are managed on client
------------------------------------------------------------------------------

                 Key: RF-5456
                 URL: https://jira.jboss.org/jira/browse/RF-5456
             Project: RichFaces
          Issue Type: Bug
    Affects Versions: 3.2.2
         Environment: Tomcat 6 + Facelets + JSF 1.2_09 RI
            Reporter: Pierre Wargnier


If you activate org.ajax4jsf.handleViewExpiredOnClient in web.xml, the 
ViewExpiredExceptions are catched by the Ajax filter and a Ajax response will 
create a javascript "confim" window to ask the user to reload the page.

If the request that cause the ViewExpiredExceptions is not an AJAX one: only a 
blank page with the AJAX XML behind.

Proposed solution: Handle ViewExpired on client _only_ if request is AJAX. 
Throw the exception the normal way otherwise.

In org.ajax4jsf.webapp.BaseXMLFilter.doXmlFilter(FilterChain, 
HttpServletRequest, HttpServletResponse)
instead of

log.error("Exception in the filter chain", e);
if(handleViewExpiredOnClient && isViewExpired(e)){

do this:

if(handleViewExpiredOnClient && isViewExpired(e) && 
request.getParameterMap().containsKey("AJAXREQUEST")){
   log.info("View expired handling sent to client"); //View expired exceptions 
are quite commons, don't clog the logs tracing them


-- 
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

        
_______________________________________________
richfaces-issues mailing list
[email protected]
https://lists.jboss.org/mailman/listinfo/richfaces-issues

Reply via email to