ICEfaces
  1. ICEfaces
  2. ICE-6064

Bridge handling of session expiry notifications

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.0-Beta1
    • Fix Version/s: 2.0-Beta2, 2.0.0
    • Component/s: Bridge
    • Labels:
      None
    • Environment:
      ICEfaces 2

      Description

      A number of cases have been resolved related to handling of session expiration (see ICE-5980, ICE-5997, ICE-6004, ICE-6005). Now, under most proper conditions, a custom SessionExpiredException will be thrown and displayed on the client. This works when ICEpush is present and will display in all views related to the expired session.

      There are a still some outstanding behaviours related to the bridge related to proper handling of the exception.

      - the Session Expired dialog appears in compat mode even if a connection status component is present (this blocks ICE-5779)
      - it appears that, if push is running, it continues to run even after receiving the session expired response

        Activity

        Deryk Sinotte created issue -
        Deryk Sinotte made changes -
        Field Original Value New Value
        Salesforce Case []
        Fix Version/s 2.0-Beta2 [ 10242 ]
        Assignee Priority P2
        Assignee Mircea Toma [ mircea.toma ]
        Hide
        Deryk Sinotte added a comment -

        The test case appears to work properly now. The only issue that I see is that the Session Expired popup gets displayed even though the connection status component is present.

        Show
        Deryk Sinotte added a comment - The test case appears to work properly now. The only issue that I see is that the Session Expired popup gets displayed even though the connection status component is present.
        Deryk Sinotte made changes -
        Link This issue blocks ICE-5779 [ ICE-5779 ]
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #22418 Fri Sep 24 09:50:17 MDT 2010 mircea.toma ICE-6064 Trap SessionExpiredException server error to invoke callback registered with onSessionExpired. Removed old session expiration code that used a special pushId to get the session expiry notification.
        Files Changed
        Commit graph MODIFY /icefaces2/trunk/icefaces/core/src/main/javascript/application.js
        Commit graph MODIFY /icefaces2/trunk/icefaces/core/src/main/java/org/icefaces/impl/event/BridgeSetup.java
        Commit graph MODIFY /icefaces2/trunk/icefaces/core/src/main/javascript/blockui.js
        Hide
        Mircea Toma added a comment -

        Changed server error handling to trap SessionExpiredException server error and invoke callbacks registered with ice.onSessionExpired method. Removed old session expiration code that used a special pushId to get the session expiry notification.

        Show
        Mircea Toma added a comment - Changed server error handling to trap SessionExpiredException server error and invoke callbacks registered with ice.onSessionExpired method. Removed old session expiration code that used a special pushId to get the session expiry notification.
        Mircea Toma made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #22479 Thu Sep 30 13:59:40 MDT 2010 mircea.toma ICE-6064 Remove ICEpush code used for sending session expiry notification.
        Files Changed
        Commit graph MODIFY /icefaces2/trunk/icefaces/core/src/main/java/org/icefaces/impl/application/SessionTimeoutMonitor.java
        Commit graph MODIFY /icefaces2/trunk/icefaces/core/src/main/java/org/icefaces/impl/application/WindowScopeManager.java
        Hide
        Mircea Toma added a comment -

        Removed ICEpush code in WindowScopeManager used for sending session expiry notification to all the windows.

        Show
        Mircea Toma added a comment - Removed ICEpush code in WindowScopeManager used for sending session expiry notification to all the windows.
        Ken Fyten made changes -
        Fix Version/s 2.0.0 [ 10230 ]
        Ken Fyten made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Assignee Priority P2

          People

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

            Dates

            • Created:
              Updated:
              Resolved: