ICEpush
  1. ICEpush
  2. PUSH-398

Modify ICEpush bridge to use add/remove/has/NotifyBackURI functions

          Details

          • Type: New Feature New Feature
          • Status: Resolved
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: EE-4.0.0.GA
          • Fix Version/s: EE-4.2.0.GA
          • Component/s: Push Server
          • Labels:
            None
          • Environment:
            supported browser

            Description

            Need to change the bridge to use the 'addNotifyBackURI' function when enabling cloud push and stop sending the 'notifyBackURI' parameter with each 'listen.icepush' request.

            Use 'hasNotifyBackURI' function to test when to invoke the 'ice.push.onMissingNotifyBackURI' registered callback.

              Issue Links

              blocks

              Bug - A problem which impairs or prevents the functions of the product. ICE-10690 Showcase -> mobi:cloudPush demo has button state problems on iOS

              • Minor - Minor loss of function, or other problem where easy workaround is present.
              • Closed

                Activity

                Ascending order - Click to sort in descending order
                Mircea Toma created issue -
                Mircea Toma made changes -
                Field Original Value New Value
                Assignee Mircea Toma [ mircea.toma ]
                Mircea Toma made changes -
                Fix Version/s EE-4.1.0.GA [ 12172 ]
                Hide
                Permalink
                Mircea Toma added a comment - - edited

                Modified ICEpush bridge to use the notify-back API. Modified hasNotifyBackURI function to use asynchronous POST request since modern browsers block synchronous requests made on the main thread.

                Show
                Mircea Toma added a comment - - edited Modified ICEpush bridge to use the notify-back API. Modified hasNotifyBackURI function to use asynchronous POST request since modern browsers block synchronous requests made on the main thread.
                Mircea Toma made changes -
                Description Need to change the bridge to use the _addNotifyBackURI_ function when enabling cloud push and stop sending the _notifyBackURI_ parameter with each _listen.icepush_ request.

                Use _hasNotifyBackURI_ function to test when to invoke the _ice.push.onMissingNotifyBackURI_ registered callback.                 		Need to change the bridge to use the 'addNotifyBackURI' function when enabling cloud push and stop sending the 'notifyBackURI' parameter with each 'listen.icepush' request.

                Use 'hasNotifyBackURI' function to test when to invoke the 'ice.push.onMissingNotifyBackURI' registered callback.
                Repository Revision Date User Message
                ICEsoft Public SVN Repository #48643 Wed Apr 20 14:46:27 MDT 2016 mircea.toma PUSH-398 Modified ICEpush bridge to use the notify-back API.
                Files Changed
                Commit graph MODIFY /icepush/trunk/icepush/core/src/main/javascript/application.js
                Hide
                Permalink
                Mircea Toma added a comment -

                Removed usages of ice.notifyBack related methods and the methods themselves.

                Show
                Mircea Toma added a comment - Removed usages of ice.notifyBack related methods and the methods themselves.
                Mircea Toma made changes -
                Status Open [ 1 ] Resolved [ 5 ]
                Resolution Fixed [ 1 ]
                Repository Revision Date User Message
                ICEsoft Public SVN Repository #48645 Wed Apr 20 17:16:09 MDT 2016 mircea.toma PUSH-398 Removed usages of ice.notifyBack related methods and the methods themselves.
                Files Changed
                Commit graph MODIFY /icepush/trunk/icepush/samples/pushservice/src/main/webapp/WEB-INF/web.xml
                Commit graph MODIFY /icepush/trunk/icepush/core/src/main/java/org/icepush/http/standard/PushRequestProxy.java
                Commit graph MODIFY /icepush/trunk/icepush/core/src/main/java/org/icepush/servlet/ServletPushRequest.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/http/PushRequest.java
                Commit graph MODIFY /icepush/trunk/icepush/core/src/main/java/org/icepush/servlet/ServletPushRequestResponse.java
                Hide
                Permalink
                Jack Van Ooststroom added a comment -

                Restored the ICEpush metric logging in the BlockingConnectionServer.

                Show
                Jack Van Ooststroom added a comment - Restored the ICEpush metric logging in the BlockingConnectionServer.
                Repository Revision Date User Message
                ICEsoft Public SVN Repository #48650 Fri Apr 22 08:40:21 MDT 2016 jack.van.ooststroom Fixed JIRA PUSH-398 : Modify ICEpush bridge to use add/remove/has/NotifyBackURI functions; Restored the ICEpush metric logging.
                Files Changed
                Commit graph MODIFY /icepush/trunk/icepush/core/src/main/java/org/icepush/BlockingConnectionServer.java
                Ken Fyten made changes -
                Link This issue blocks ICE-10690 [ ICE-10690 ]
                Ken Fyten made changes -
                Fix Version/s EE-4.1.0.BETA [ 13073 ]
                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 ]

                  People

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

                    Dates

                    • Created:
                      Updated:
                      Resolved: