ICEpush
  1. ICEpush
  2. PUSH-149

Cloud Push is delivered even when the client is in the foreground.

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 3.0
    • Fix Version/s: 3.0
    • Component/s: Push Library
    • Labels:
      None
    • Environment:
      ICEfaces 2.1, ICEmobile, ICEpush

      Description

      The changes for ICE-7242 have caused a regression in Cloud Push notification -- Cloud Push is now delivered even when the client is in the foreground.

      Created this JIRA to find common solution that will work for both MyFaces and Could Push for 2.1 final.

        Activity

        Ken Fyten created issue -
        Ken Fyten made changes -
        Field Original Value New Value
        Project ICEfaces [ 10021 ] ICEpush [ 10050 ]
        Key ICE-7421 PUSH-149
        Fix Version/s 2.1 [ 10259 ]
        Fix Version/s 2.1 [ 10241 ]
        Affects Version/s 2.1 [ 10259 ]
        Affects Version/s 2.1-Beta2 [ 10294 ]
        Ken Fyten made changes -
        Salesforce Case []
        Component/s Push Library [ 10044 ]
        Assignee Priority P2
        Assignee Mircea Toma [ mircea.toma ]
        Hide
        Ted Goddard added a comment -

        Some aspects of this may be resolved by Cloud Push notification queuing: if an ICEpush notification is issued but the browser does not respond in a timely fashion with a "listen" request, then a Cloud Push notification should be issued. However, to detect "in a timely fashion" it is necessary to schedule the task to check for the subsequent listen (since the Cloud Push is sent only if the "listen" is not received).

        Show
        Ted Goddard added a comment - Some aspects of this may be resolved by Cloud Push notification queuing: if an ICEpush notification is issued but the browser does not respond in a timely fashion with a "listen" request, then a Cloud Push notification should be issued. However, to detect "in a timely fashion" it is necessary to schedule the task to check for the subsequent listen (since the Cloud Push is sent only if the "listen" is not received).
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #26545 Wed Nov 23 15:19:02 MST 2011 mircea.toma PUSH-149 Separated the clearing of the pending notification from the incoming pushId notification sent to the PushGroupManager since they should occur at different times during blocking connection's lifecycle. Introduced PushGroupManager.clearPendingNotifications(String[] ids) method.
        Files Changed
        Commit graph MODIFY /icepush/trunk/icepush/core/src/main/java/org/icepush/PushGroupManager.java
        Commit graph MODIFY /icepush/trunk/icepush/core/src/main/java/org/icepush/LocalPushGroupManager.java
        Commit graph MODIFY /icepush/trunk/icepush/core/src/main/java/org/icepush/NoopPushGroupManager.java
        Commit graph MODIFY /icepush/trunk/icepush/core/src/main/java/org/icepush/BlockingConnectionServer.java
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #26546 Wed Nov 23 15:41:44 MST 2011 mircea.toma PUSH-149 Moved ice.push.parkInactivePushIds function into ICEpush core.
        Files Changed
        Commit graph MODIFY /icepush/trunk/icepush/core/src/main/javascript/application.js
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #26547 Wed Nov 23 15:44:18 MST 2011 mircea.toma PUSH-149 Updated icepush.jar library.
        Files Changed
        Commit graph MODIFY /icemobile/trunk/icemobile/lib/icepush.jar
        Hide
        Mircea Toma added a comment -

        Separated the clearing of the pending notification from the incoming pushId notification sent to the PushGroupManager since they occur at different times during blocking connection's lifecycle. Introduced PushGroupManager.clearPendingNotifications(String[] ids) method.

        Show
        Mircea Toma added a comment - Separated the clearing of the pending notification from the incoming pushId notification sent to the PushGroupManager since they occur at different times during blocking connection's lifecycle. Introduced PushGroupManager.clearPendingNotifications(String[] ids) method.
        Mircea Toma made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #26596 Fri Nov 25 14:22:37 MST 2011 mircea.toma PUSH-149 Move pushID parking feature into core ICEpush to fully support the email notification provider.
        Files Changed
        Commit graph MODIFY /icepush/trunk/icepush/core/src/main/java/org/icepush/servlet/MainServlet.java
        Commit graph MODIFY /icepush/trunk/icepush/core/src/main/java/org/icepush/servlet/BrowserBoundServlet.java
        Commit graph ADD /icepush/trunk/icepush/core/src/main/java/org/icepush/ParkIDsServer.java
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #26610 Mon Nov 28 10:43:15 MST 2011 ted.goddard recover when no extensions are available (PUSH-149)
        Files Changed
        Commit graph MODIFY /icepush/trunk/icepush/core/src/main/java/org/icepush/servlet/MainServlet.java
        Commit graph MODIFY /icepush/trunk/icepush/core/src/main/java/org/icepush/util/ExtensionRegistry.java
        Ken Fyten made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Assignee Priority P2

          People

          • Assignee:
            Mircea Toma
            Reporter:
            Ken Fyten
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: