ICEpush
  1. ICEpush
  2. PUSH-154

Avoid sending cloud push notifications during response-request gap

          Details

          • Type: Bug Bug
          • Status: Closed
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: 2.0-Alpha3
          • Fix Version/s: 3.0
          • Component/s: None
          • Labels:
            None
          • Environment:
            server
          • Affects:
            Documentation (User Guide, Ref. Guide, etc.)

            Description

            Avoid sending cloud push notifications when notifications are triggered during the server-response - client-request gap. During this gap the pushIds are temporarily parked (when blocking connection runs) which is falsely interpreted as stopped blocking connection.

              Activity

              Repository Revision Date User Message
              ICEsoft Public SVN Repository #26958 Mon Dec 19 15:31:57 MST 2011 mircea.toma PUSH-154 Stop scanning for expiry when adding (or re-adding) pushIDs to the group. The scanning is to be triggered only after the notification confirmation is received giving thus a chance to the server to touch the active pushIds and groups.
              Files Changed
              Commit graph MODIFY /icepush/trunk/icepush/core/src/main/java/org/icepush/LocalPushGroupManager.java
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #26914 Fri Dec 16 01:45:00 MST 2011 mircea.toma PUSH-154 Make sure that the timer is properly stopped on application restart.
              Files Changed
              Commit graph MODIFY /icepush/trunk/icepush/core/src/main/java/org/icepush/LocalPushGroupManager.java
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #26905 Thu Dec 15 15:36:32 MST 2011 mircea.toma PUSH-154 Stop sharing the timer used by the LocalPushGroupManager since its thread is used up.
              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/LocalPushGroupManager.java
              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/servlet/BrowserBoundServlet.java
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #26857 Tue Dec 13 12:31:11 MST 2011 ted.goddard trunk icepush.jar revision 29615 (PUSH-154)
              Files Changed
              Commit graph MODIFY /icemobile/trunk/icemobile/lib/icepush.jar
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #26823 Mon Dec 12 08:53:38 MST 2011 ken.fyten PUSH-154 - Fixed typo.
              Files Changed
              Commit graph MODIFY /icepush/trunk/icepush/core/src/main/java/org/icepush/LocalPushGroupManager.java
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #26821 Mon Dec 12 08:00:20 MST 2011 mircea.toma PUSH-154 Run email sending on a different thread to avoid blocking the NotificationProvider.send method.
              Files Changed
              Commit graph MODIFY /icepush/trunk/icepush/core/src/main/java/org/icepush/EmailNotificationProvider.java
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #26820 Mon Dec 12 07:36:00 MST 2011 mircea.toma PUSH-154 Removed unused class -- its functionality was rolled into BlockingConnectionServer.
              Files Changed
              Commit graph DEL /icepush/trunk/icepush/core/src/main/java/org/icepush/ParkIDsServer.java
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #26819 Mon Dec 12 07:29:12 MST 2011 mircea.toma PUSH-154 Refactored internal communication between PushGroupManager and BlockingConnectionServer. Introduced the concept of confirmation for the received notifications. Implemented notification queue to make sure the notifications are properly serialized and buffered when needed. Moved browser request timeout mechanism in BlockingConnectionServer that lets the PushGroupManager know when notifications were not received.
              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/servlet/MainServlet.java
              Commit graph MODIFY /icepush/trunk/icepush/core/src/main/java/org/icepush/LocalPushGroupManager.java
              Commit graph ADD /icepush/trunk/icepush/core/src/main/java/org/icepush/LocalNotificationBroadcaster.java
              Commit graph MODIFY /icepush/trunk/icepush/core/src/main/java/org/icepush/NoopPushGroupManager.java
              Commit graph ADD /icepush/trunk/icepush/core/src/main/java/org/icepush/NotificationBroadcaster.java
              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/servlet/BrowserBoundServlet.java

                People

                • Assignee:
                  Mircea Toma
                  Reporter:
                  Mircea Toma
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  0 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: