ICEpush
  1. ICEpush
  2. PUSH-402

Remove the minCloudPushInterval and its usage

          Details

          • Type: Improvement Improvement
          • Status: Resolved
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: EE-4.0.0.GA, 4.1
          • Fix Version/s: EE-4.2.0.GA
          • Component/s: Push Library
          • Labels:
            None
          • Environment:
            ICEpush
          • Assignee Priority:
            P1

            Description

            The minCloudPushInterval and its usage should be removed from ICEpush. As a result the application developer is responsible for ensuring a Cloud Push storm does not occur. This is to avoid the dropping of Cloud Push Notifications if by chance they are within this interval.

              Activity

              Ascending order - Click to sort in descending order
              Hide
              Permalink
              Jack Van Ooststroom added a comment -

              Sending icepush/core/src/main/java/org/icepush/Browser.java
              Sending icepush/core/src/main/java/org/icepush/ConfirmationTimeout.java
              Sending icepush/core/src/main/java/org/icepush/LocalPushGroupManager.java
              Sending icepush/core/src/main/java/org/icepush/servlet/BrowserDispatcher.java
              Transmitting file data ....
              Committed revision 48639.

              Show
              Jack Van Ooststroom added a comment - Sending icepush/core/src/main/java/org/icepush/Browser.java Sending icepush/core/src/main/java/org/icepush/ConfirmationTimeout.java Sending icepush/core/src/main/java/org/icepush/LocalPushGroupManager.java Sending icepush/core/src/main/java/org/icepush/servlet/BrowserDispatcher.java Transmitting file data .... Committed revision 48639.
              Hide
              Permalink
              Jack Van Ooststroom added a comment -

              Sending icepush-ee/eps/src/main/java/com/icesoft/push/Browser.java
              Sending icepush-ee/eps/src/main/java/com/icesoft/push/ConfirmationTimeout.java
              Sending icepush-ee/eps/src/main/java/com/icesoft/push/LocalPushGroupManager.java
              Sending icepush-ee/eps/src/main/java/com/icesoft/push/servlet/BrowserDispatcher.java
              Transmitting file data ....
              Committed revision 44968.

              Show
              Jack Van Ooststroom added a comment - Sending icepush-ee/eps/src/main/java/com/icesoft/push/Browser.java Sending icepush-ee/eps/src/main/java/com/icesoft/push/ConfirmationTimeout.java Sending icepush-ee/eps/src/main/java/com/icesoft/push/LocalPushGroupManager.java Sending icepush-ee/eps/src/main/java/com/icesoft/push/servlet/BrowserDispatcher.java Transmitting file data .... Committed revision 44968.
              Hide
              Permalink
              Jack Van Ooststroom added a comment -

              The minCloudPushInterval and its usage has been removed at both the ICEpush and ICEpush-EE levels. It is now the responsibility of the application developer to avoid unintended Cloud Push Notification "storms". Marking this one as FIXED.

              Show
              Jack Van Ooststroom added a comment - The minCloudPushInterval and its usage has been removed at both the ICEpush and ICEpush-EE levels. It is now the responsibility of the application developer to avoid unintended Cloud Push Notification "storms". Marking this one as FIXED.
              Hide
              Permalink
              Ken Fyten added a comment - - edited

              Re-opened due to testing failures:

              Tested with ICEfaces EE-4.1. r48645,
              ipad ios 9, android nexus 5 version 6.0.1

              cloudPush scenarios
              1.) Go to registration page, enable cloudPush.
              Go to redPage, send message.
              Go to yellowPage, send message.
              Minimize browser.
              Fail android- Only the red notification comes through.
              Pass ios - Both notifications come through. (Liana reports only receiving one notification on iPad sometimes).

              Show
              Ken Fyten added a comment - - edited Re-opened due to testing failures: Tested with ICEfaces EE-4.1. r48645, ipad ios 9, android nexus 5 version 6.0.1 cloudPush scenarios 1.) Go to registration page, enable cloudPush. Go to redPage, send message. Go to yellowPage, send message. Minimize browser. Fail android- Only the red notification comes through. Pass ios - Both notifications come through. (Liana reports only receiving one notification on iPad sometimes).
              Hide
              Permalink
              Liana Munroe added a comment -

              Tested with Labs 1 server, ios 9.3.1 and android 6.0.1 devices.
              Multiple messages are now arriving on devices as expected.

              Show
              Liana Munroe added a comment - Tested with Labs 1 server, ios 9.3.1 and android 6.0.1 devices. Multiple messages are now arriving on devices as expected.
              Hide
              Permalink
              Jack Van Ooststroom added a comment -

              Sending icepush-ee/core/src/main/java/org/icepush/LocalPushGroupManager.java
              Transmitting file data .
              Committed revision 48698.

              Show
              Jack Van Ooststroom added a comment - Sending icepush-ee/core/src/main/java/org/icepush/LocalPushGroupManager.java Transmitting file data . Committed revision 48698.
              Hide
              Permalink
              Jack Van Ooststroom added a comment -

              The scenario mentioned by Ken has been addressed in PUSH-410.

              Show
              Jack Van Ooststroom added a comment - The scenario mentioned by Ken has been addressed in PUSH-410 .
              Hide
              Permalink
              Jack Van Ooststroom added a comment -

              The old minCloudPushInterval logic has been removed. Marking this one as FIXED again.

              Show
              Jack Van Ooststroom added a comment - The old minCloudPushInterval logic has been removed. Marking this one as FIXED again.

                People

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

                  Dates

                  • Created:
                    Updated:
                    Resolved: