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

        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
        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
        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
        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
        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: