MyFaces Core
  1. MyFaces Core
  2. MYFACES-3587

Not existing viewId will not be handled

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.1.8
    • Fix Version/s: 2.1.13
    • Component/s: General
    • Labels:
      None
    • Environment:
      Jetty/Tomcat, JUEL, CODI, ExtVal

      Description

      If i call a page, which does not exist, following exceptions occurs: Cannot reset buffer after response has been committed.

      After digging deeper into this problem, i found out that getViewHandlerSupport()#calculateViewId returns null and the JspViewDeclarationLanguageStrategy will be used ->

      Cannot reset buffer after response has been committed.

      occurs.

      I added a null check for the logicalViewId in RestoreViewExecutor#execute to call HttpServletResponse#sendError.

      It does not work as expected because it just renders the errorPage and no redirect will be done.

      Why there is not such null check?

      Is it possible to add this check and redirect to the web.xml defined 404 or common error page? Or should it use the ErrorHandler?

        Issue Links

          Activity

          Thomas Andraschko created issue -
          Leonardo Uribe made changes -
          Field Original Value New Value
          Status Open [ 1 ] Closed [ 6 ]
          Assignee Leonardo Uribe [ lu4242 ]
          Resolution Won't Fix [ 2 ]
          Gerhard Petracek made changes -
          Resolution Won't Fix [ 2 ]
          Status Closed [ 6 ] Reopened [ 4 ]
          Martin Kočí made changes -
          Link This issue is related to MYFACES-3223 [ MYFACES-3223 ]
          Leonardo Uribe made changes -
          Status Reopened [ 4 ] Resolved [ 5 ]
          Fix Version/s 2.1.13 [ 12324569 ]
          Resolution Fixed [ 1 ]
          Leonardo Uribe made changes -
          Status Resolved [ 5 ] Closed [ 6 ]

            People

            • Assignee:
              Leonardo Uribe
              Reporter:
              Thomas Andraschko
            • Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development