ICEfaces
  1. ICEfaces
  2. ICE-8547

BlockingServlet simplified error response

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: EE-1.8.2.GA_P03
    • Fix Version/s: EE-1.8.2.GA_P05
    • Component/s: Framework
    • Labels:
      None
    • Environment:
      ICEfaces
    • Assignee Priority:
      P2
    • Affects:
      Documentation (User Guide, Ref. Guide, etc.), Compatibility/Configuration

      Description

      BlockingServlet.java line 70 contains the following:

                  ((HttpServletResponse) servletResponse)
                          .sendError(500, e.getMessage());

      Some applications prefer not to echo the internal details of the error. Instead, the Exception should be logged and the sendError text should be the static string "Session Expired".

        Activity

        Ted Goddard created issue -
        Ted Goddard made changes -
        Field Original Value New Value
        Salesforce Case []
        Arran Mccullough made changes -
        Salesforce Case [5007000000O8ixC]
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #31368 Tue Oct 09 14:29:58 MDT 2012 jack.van.ooststroom Fixed JIRA ICE-8547 : BlockingServlet simplified error response
        Files Changed
        Commit graph MODIFY /icefaces/trunk/icefaces/core/src/com/icesoft/faces/webapp/xmlhttp/BlockingServlet.java
        Hide
        Jack Van Ooststroom added a comment - - edited

        Sending src/com/icesoft/faces/webapp/xmlhttp/BlockingServlet.java
        Transmitting file data .
        Committed revision 31368.

        Show
        Jack Van Ooststroom added a comment - - edited Sending src/com/icesoft/faces/webapp/xmlhttp/BlockingServlet.java Transmitting file data . Committed revision 31368.
        Hide
        Jack Van Ooststroom added a comment - - edited

        e.getMessage() is now removed from the 500 response and being logged at DEBUG level. Marking this one as FIXED.

        Show
        Jack Van Ooststroom added a comment - - edited e.getMessage() is now removed from the 500 response and being logged at DEBUG level. Marking this one as FIXED.
        Migration made changes -
        Assignee Jack Van Ooststroom [ jack.van.ooststroom ]
        Affects Documentation (User Guide, Ref. Guide, etc.),Compatibility/Configuration [ 10003, 10002 ]
        Assignee Priority P2 [ 10011 ]
        Migration made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Ken Fyten made changes -
        Status Resolved [ 5 ] Closed [ 6 ]

          People

          • Assignee:
            Jack Van Ooststroom
            Reporter:
            Ted Goddard
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: