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

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

          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: