ICEpush
  1. ICEpush
  2. PUSH-403

Add ability to force a Cloud Push Notification

          Details

          • Type: New Feature New Feature
          • 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, ICEnotify

            Description

            Add the ability to force a Cloud Push Notification regardless of the client being present/active.

              Activity

              Ascending order - Click to sort in descending order
              Jack Van Ooststroom created issue -
              Jack Van Ooststroom made changes -
              Field Original Value New Value
              Assignee Jack Van Ooststroom [ jack.van.ooststroom ]
              Ken Fyten made changes -
              Fix Version/s EE-4.1.0.GA [ 12172 ]
              Hide
              Permalink
              Jack Van Ooststroom added a comment -

              Sending core/src/main/java/org/icefaces/application/PushMessage.java
              Sending icepush/core/src/main/java/org/icepush/BlockingConnectionServer.java
              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/InternalPushGroupManager.java
              Sending icepush/core/src/main/java/org/icepush/LocalPushGroupManager.java
              Sending icepush/core/src/main/java/org/icepush/NotificationEntry.java
              Sending icepush/core/src/main/java/org/icepush/OutOfBandNotification.java
              Sending icepush/core/src/main/java/org/icepush/PushConfiguration.java
              Sending icepush/core/src/main/java/org/icepush/PushNotification.java
              Sending icepush/core/src/main/javascript/application.js
              Transmitting file data ...........
              Committed revision 48664.

              Show
              Jack Van Ooststroom added a comment - Sending core/src/main/java/org/icefaces/application/PushMessage.java Sending icepush/core/src/main/java/org/icepush/BlockingConnectionServer.java 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/InternalPushGroupManager.java Sending icepush/core/src/main/java/org/icepush/LocalPushGroupManager.java Sending icepush/core/src/main/java/org/icepush/NotificationEntry.java Sending icepush/core/src/main/java/org/icepush/OutOfBandNotification.java Sending icepush/core/src/main/java/org/icepush/PushConfiguration.java Sending icepush/core/src/main/java/org/icepush/PushNotification.java Sending icepush/core/src/main/javascript/application.js Transmitting file data ........... Committed revision 48664.
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #48664 Wed Apr 27 17:11:40 MDT 2016 jack.van.ooststroom Fixed JIRA PUSH-403 : Add ability to force a Cloud Push Notification
              Files Changed
              Commit graph MODIFY /icepush/trunk/icepush/core/src/main/java/org/icepush/NotificationEntry.java
              Commit graph MODIFY /icepush/trunk/icepush/core/src/main/java/org/icepush/OutOfBandNotification.java
              Commit graph MODIFY /icepush/trunk/icepush/core/src/main/java/org/icepush/InternalPushGroupManager.java
              Commit graph MODIFY /icepush/trunk/icepush/core/src/main/java/org/icepush/Browser.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/PushConfiguration.java
              Commit graph MODIFY /icefaces4/trunk/icefaces/core/src/main/java/org/icefaces/application/PushMessage.java
              Commit graph MODIFY /icepush/trunk/icepush/core/src/main/javascript/application.js
              Commit graph MODIFY /icepush/trunk/icepush/core/src/main/java/org/icepush/BlockingConnectionServer.java
              Commit graph MODIFY /icepush/trunk/icepush/core/src/main/java/org/icepush/ConfirmationTimeout.java
              Commit graph MODIFY /icepush/trunk/icepush/core/src/main/java/org/icepush/PushNotification.java
              Hide
              Permalink
              Jack Van Ooststroom added a comment -

              Sending eps/src/main/java/com/icesoft/push/Browser.java
              Sending eps/src/main/java/com/icesoft/push/ConfirmationTimeout.java
              Sending eps/src/main/java/com/icesoft/push/DynamicPushGroupManager.java
              Sending eps/src/main/java/com/icesoft/push/InternalPushGroupManager.java
              Sending eps/src/main/java/com/icesoft/push/LocalPushGroupManager.java
              Sending eps/src/main/java/com/icesoft/push/RemotePushGroupManager.java
              Sending eps/src/main/java/com/icesoft/push/StatusManager.java
              Transmitting file data .......
              Committed revision 45000.

              Show
              Jack Van Ooststroom added a comment - Sending eps/src/main/java/com/icesoft/push/Browser.java Sending eps/src/main/java/com/icesoft/push/ConfirmationTimeout.java Sending eps/src/main/java/com/icesoft/push/DynamicPushGroupManager.java Sending eps/src/main/java/com/icesoft/push/InternalPushGroupManager.java Sending eps/src/main/java/com/icesoft/push/LocalPushGroupManager.java Sending eps/src/main/java/com/icesoft/push/RemotePushGroupManager.java Sending eps/src/main/java/com/icesoft/push/StatusManager.java Transmitting file data ....... Committed revision 45000.
              Hide
              Permalink
              Jack Van Ooststroom added a comment -

              The ability to force a Cloud Push Notification to be sent regardless of the client being active or not has been added at three levels:

              1. At the server-side ICEfaces-level using the PushMessage class. The existing two constructors have been overloaded to include a new forced boolean parameter.
              2. At the server-side ICEpush-level using the PushNotification class. The existing two constructors have been overloaded to include a new forced boolean parameter.
              3. At the client-side ICEpush-level using the notify function. It now parses out a new forced option.

              The server-side logic has been augmented to handle this new forced Cloud Push Notification feature. The ConfirmationTimeout ends up with the forced boolean. Previously, upon receiving a new listen.icepush request any outstanding ConfirmationTimeout for the matching Browser-ID would get cancelled. Now it only gets cancelled if the ConfirmationTimeout's isForced() method returns false. Once any ConfirmationTimeout is finished executing it will still cancel itself regardless of what isForced() returns.

              Marking this one as FIXED.

              Show
              Jack Van Ooststroom added a comment - The ability to force a Cloud Push Notification to be sent regardless of the client being active or not has been added at three levels: At the server-side ICEfaces-level using the PushMessage class. The existing two constructors have been overloaded to include a new forced boolean parameter. At the server-side ICEpush-level using the PushNotification class. The existing two constructors have been overloaded to include a new forced boolean parameter. At the client-side ICEpush-level using the notify function. It now parses out a new forced option. The server-side logic has been augmented to handle this new forced Cloud Push Notification feature. The ConfirmationTimeout ends up with the forced boolean. Previously, upon receiving a new listen.icepush request any outstanding ConfirmationTimeout for the matching Browser-ID would get cancelled. Now it only gets cancelled if the ConfirmationTimeout 's isForced() method returns false . Once any ConfirmationTimeout is finished executing it will still cancel itself regardless of what isForced() returns. Marking this one as FIXED.
              Jack Van Ooststroom made changes -
              Status Open [ 1 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Ken Fyten made changes -
              Fix Version/s EE-4.1.0.BETA [ 13073 ]
              Hide
              Permalink
              Liana Munroe added a comment -

              Verified ICEfaces EE-4.1.0 r48686. Tomcat 8, android 6.0.1, ios 9.3.1.
              The QA test application using the PushMessage class was updated and checked in.
              The application has been updated on the Labs1 server.

              Show
              Liana Munroe added a comment - Verified ICEfaces EE-4.1.0 r48686. Tomcat 8, android 6.0.1, ios 9.3.1. The QA test application using the PushMessage class was updated and checked in. The application has been updated on the Labs1 server.
              Ken Fyten made changes -
              Fix Version/s EE-4.1.1.BETA [ 13077 ]
              Fix Version/s EE-4.1.0.GA [ 12172 ]
              Fix Version/s EE-4.1.0.RC1 [ 13073 ]
              Ken Fyten made changes -
              Fix Version/s EE-4.2.0.GA [ 13074 ]
              Fix Version/s EE-4.1.1.BETA [ 13077 ]
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #50745 Mon Feb 27 09:39:31 MST 2017 jack.van.ooststroom Reverted PUSH-403 fixes to ICEfaces-layer
              Files Changed
              Commit graph MODIFY /icefaces4/trunk/icefaces/core/src/main/java/org/icefaces/application/PushMessage.java

                People

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

                  Dates

                  • Created:
                    Updated:
                    Resolved: