ICEfaces
  1. ICEfaces
  2. ICE-7344

Session Expired responses received but not displayed.

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.1-Beta
    • Fix Version/s: 2.1-Beta2, 3.0
    • Component/s: Bridge
    • Labels:
      None
    • Environment:
      ICEfaces 2 Auction AuctionMonitor MyFaces Mojarra
    • Assignee Priority:
      P2

      Description

      In testing the Auction example with MyFaces, I noticed that when the session expired, I was receiving the proper response from the push requests:

      <?xml version='1.0' encoding='UTF-8'?>
          <partial-response>
              <error>
                  <error-name>class org.icefaces.application.SessionExpiredException</error-name>
                  <error-message><![CDATA[Session has expired]]></error-message>
              </error>
          </partial-response>

      but it was not popping up a notification on the client nor was it shutting down push. Push would keep running and returning the SessionExpiredException over and over.

      I set some breakpoints at various points in the bridge.js code to see where it might be breaking down but it doesn't appear that the following function is being called at all:

              function filterICEfacesEvents(f) {...

      which suggests that the callback is not being called for some reason:

              jsf.ajax.addOnError(filterICEfacesEvents(submitErrorBroadcaster(networkErrorListeners, serverErrorListeners)));

        Issue Links

          Activity

          Repository Revision Date User Message
          ICEsoft Public SVN Repository #25920 Fri Oct 14 09:11:45 MDT 2011 mircea.toma ICE-7344 Changed filterICEfacesEvents() function to detect ICEfaces enabled forms also when server error partial updates are received.
          Files Changed
          Commit graph MODIFY /icefaces2/trunk/icefaces/core/src/main/javascript/application.js

            People

            • Assignee:
              Mircea Toma
              Reporter:
              Deryk Sinotte
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: