ICEfaces
  1. ICEfaces
  2. ICE-3007

Exception handling not working with 1.7

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.7
    • Fix Version/s: 1.7.1
    • Component/s: Framework
    • Labels:
      None
    • Environment:
      n/a

      Description

      exception handling seems broken with 1.7
      1. Test_ExceptionHandling.zip
        232 kB
        Philip Breau

        Issue Links

          Activity

          Philip Breau created issue -
          Hide
          Philip Breau added a comment -

          test case

          Show
          Philip Breau added a comment - test case
          Philip Breau made changes -
          Field Original Value New Value
          Attachment Test_ExceptionHandling.zip [ 10927 ]
          Hide
          Philip Breau added a comment -

          tried a few different scenarios:

          • defining jsp that sends a redirect to an ICEfaces page -
            result: excpetion not handled
          • defining an html page directly
            result: html page comes through the server 500 response, but is not rendered in the browser

          we need to explicitly define what should and shouldn't work with exception handling

          • can you use an ICEfaces page directly? worked in some previous version
          • can you use a JSP page to redirect? worked in 1.6.2
          • can you use an HTML page? worked in 1.6.2
          Show
          Philip Breau added a comment - tried a few different scenarios: defining jsp that sends a redirect to an ICEfaces page - result: excpetion not handled defining an html page directly result: html page comes through the server 500 response, but is not rendered in the browser we need to explicitly define what should and shouldn't work with exception handling can you use an ICEfaces page directly? worked in some previous version can you use a JSP page to redirect? worked in 1.6.2 can you use an HTML page? worked in 1.6.2
          Ken Fyten made changes -
          Fix Version/s 1.7.1 [ 10122 ]
          Assignee Mircea Toma [ mircea.toma ]
          Repository Revision Date User Message
          ICEsoft Public SVN Repository #16476 Mon Apr 21 15:43:51 MDT 2008 mircea.toma Fix how 'body' element is updated.
          ICE-3007
          Files Changed
          Commit graph MODIFY /icefaces/trunk/icefaces/bridge/lib/element.js
          Commit graph MODIFY /icefaces/trunk/icefaces/bridge/src/application.js
          Hide
          Mircea Toma added a comment -

          Fix how 'body' element is updated.

          Show
          Mircea Toma added a comment - Fix how 'body' element is updated.
          Mircea Toma made changes -
          Status Open [ 1 ] Resolved [ 5 ]
          Resolution Fixed [ 1 ]
          Ken Fyten made changes -
          Link This issue blocks ICE-2990 [ ICE-2990 ]
          Hide
          Mandeep Hayher added a comment -

          Following scenarios have been successfully verified on revision# 16533

          1) use jspx
          <error-page>
          <exception-type>java.lang.Exception</exception-type>
          <location>/error.jspx</location>
          </error-page>
          <error-page>
          <error-code>404</error-code>
          <location>/error.jspx</location>
          </error-page>

          2) use html page
          <error-page>
          <exception-type>java.lang.Exception</exception-type>
          <location>/error.html</location>
          </error-page>
          <error-page>
          <error-code>404</error-code>
          <location>/error.html</location>
          </error-page>

          Show
          Mandeep Hayher added a comment - Following scenarios have been successfully verified on revision# 16533 1) use jspx <error-page> <exception-type>java.lang.Exception</exception-type> <location>/error.jspx</location> </error-page> <error-page> <error-code>404</error-code> <location>/error.jspx</location> </error-page> 2) use html page <error-page> <exception-type>java.lang.Exception</exception-type> <location>/error.html</location> </error-page> <error-page> <error-code>404</error-code> <location>/error.html</location> </error-page>
          Hide
          Mandeep Hayher added a comment -

          Created new Jira (ICE-3028) for case where error handling jsp page is redirecting to another jspx.

          Show
          Mandeep Hayher added a comment - Created new Jira ( ICE-3028 ) for case where error handling jsp page is redirecting to another jspx.
          Mandeep Hayher made changes -
          Link This issue blocks ICE-3029 [ ICE-3029 ]
          Hide
          Mandeep Hayher added a comment -

          Created new Jira ICE-3029 for Internal Server Error when using any of the exception handling scenarios.

          Show
          Mandeep Hayher added a comment - Created new Jira ICE-3029 for Internal Server Error when using any of the exception handling scenarios.
          Repository Revision Date User Message
          ICEsoft Public SVN Repository #16631 Mon May 12 12:01:14 MDT 2008 mircea.toma Backport fixes for ICE-3007
          Files Changed
          Commit graph MODIFY /icefaces/branches/icefaces-1.7/icefaces/bridge/lib/element.js
          Hide
          Christophe Taverne added a comment -

          Redirection seems not working in the last svn 16659.
          I must do refresh on my navigator for the redirection.

          Show
          Christophe Taverne added a comment - Redirection seems not working in the last svn 16659. I must do refresh on my navigator for the redirection.
          Hide
          Christophe Taverne added a comment -

          Hi,

          Please read this topic: http://www.icefaces.org/JForum/posts/list/7986.page

          This issue seems not be entirely fixed.
          In some case redirection is working. In other case, we get tomcat error page.

          regards,

          Show
          Christophe Taverne added a comment - Hi, Please read this topic: http://www.icefaces.org/JForum/posts/list/7986.page This issue seems not be entirely fixed. In some case redirection is working. In other case, we get tomcat error page. regards,
          Hide
          Christophe Taverne added a comment -

          Perfect, it works now !

          Thanks :-D

          Show
          Christophe Taverne added a comment - Perfect, it works now ! Thanks :-D
          Ken Fyten made changes -
          Status Resolved [ 5 ] Closed [ 6 ]
          Assignee Mircea Toma [ mircea.toma ]
          Michael Thiem made changes -
          Resolution Fixed [ 1 ]
          Status Closed [ 6 ] Reopened [ 4 ]
          Michael Thiem made changes -
          Attachment Test_ErrorPage.zip [ 11125 ]
          Michael Thiem made changes -
          Ken Fyten made changes -
          Status Reopened [ 4 ] Resolved [ 5 ]
          Resolution Fixed [ 1 ]
          Ken Fyten made changes -
          Status Resolved [ 5 ] Closed [ 6 ]
          Ken Fyten made changes -
          Comment [ can be reproduced in 1.7.1 ]

            People

            • Assignee:
              Unassigned
              Reporter:
              Philip Breau
            • Votes:
              1 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: