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

              Ascending order - Click to sort in descending order
              Hide
              Permalink
              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
              Permalink
              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.

                People

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

                  Dates

                  • Created:
                    Updated:
                    Resolved: