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

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