ICEpush
  1. ICEpush
  2. PUSH-390

Add new .icepush requests for adding, querying and removing Notify-Back-URIs

    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
    • Labels:
      None
    • Environment:
      ICEpush JavaScript Bridge, ICEpush Core
    • Assignee Priority:
      P1

      Description

      Instead of lazily adding the ice.notifyBack to the listen.icepush requests upon registering for cloud push and resulting in the eventual server-side NotifyBackURI representation, we should introduce new .icepush requests specifically for this:

      1. add-notify-back-uri.icepush
      2. has-notify-back-uri.icepush
      3. remove-notify-back-uri.icepush

      All these requests should be kept quite simple. The add request should at a minimum include the browserID and notifyBackURI as only the client-side is able to obtain the notifyBackURI. The response to the add request can likely just be a 200 OK or 204 No Content status message. The has request should at a minimum include the browserID. This request can be used by the cloud/mobile component to check if its browserID already has a notifyBackURI associated with it. The response to the has request can likely just be a 200 OK or 204 No Content status message indicating its browserID has a notifyBackURI associated with it, or it can likely just be a 404 Not Found status message indicating its browserID has no notifyBackURI associated with it.

      For this both the client-side and server-side need to include support for these new .icepush requests.
      1. PUSH-390.client-side.patch
        10 kB
        Mircea Toma
      2. PUSH-390.patch
        29 kB
        Jack Van Ooststroom

        Activity

        Repository Revision Date User Message
        ICEsoft Public SVN Repository #50761 Mon Feb 27 16:30:58 MST 2017 jack.van.ooststroom Reverted PUSH-390 fixes to ICEfaces-layer
        Files Changed
        Commit graph DEL /icefaces4/trunk/icefaces/core/src/main/resources/META-INF/resources/add-notify-back-uri.icepush.txt
        Commit graph DEL /icefaces4/trunk/icefaces/core/src/main/resources/META-INF/resources/has-notify-back-uri.icepush.txt
        Commit graph DEL /icefaces4/trunk/icefaces/core/src/main/resources/META-INF/resources/remove-notify-back-uri.icepush.txt
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #50746 Mon Feb 27 09:46:53 MST 2017 jack.van.ooststroom Reverted PUSH-390 fixes to ICEfaces-layer
        Files Changed
        Commit graph MODIFY /icefaces4/trunk/icefaces/core/src/main/java/org/icefaces/impl/push/servlet/ICEpushResourceHandler.java
        Commit graph MODIFY /icefaces4/trunk/icefaces/core/src/main/java/org/icefaces/impl/event/BridgeSetup.java
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #48629 Thu Apr 14 13:31:10 MDT 2016 mircea.toma PUSH-390 Add resource dummy files that correspond to add/remove/has notify back URI requests.
        Files Changed
        Commit graph ADD /icefaces4/trunk/icefaces/core/src/main/resources/META-INF/resources/add-notify-back-uri.icepush.txt
        Commit graph ADD /icefaces4/trunk/icefaces/core/src/main/resources/META-INF/resources/has-notify-back-uri.icepush.txt
        Commit graph ADD /icefaces4/trunk/icefaces/core/src/main/resources/META-INF/resources/remove-notify-back-uri.icepush.txt
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #48626 Wed Apr 13 16:35:27 MDT 2016 jack.van.ooststroom Fixed JIRA PUSH-390 : Add new .icepush requests for adding, querying and removing Notify-Back-URIs
        Files Changed
        Commit graph MODIFY /icefaces4/trunk/icefaces/core/src/main/java/org/icefaces/impl/push/servlet/ICEpushResourceHandler.java
        Commit graph MODIFY /icefaces4/trunk/icefaces/core/src/main/java/org/icefaces/impl/event/BridgeSetup.java
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #48625 Wed Apr 13 16:34:30 MDT 2016 jack.van.ooststroom Fixed JIRA PUSH-390 : Add new .icepush requests for adding, querying and removing Notify-Back-URIs
        Files Changed
        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/PushGroupManager.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/servlet/HasNotifyBackURI.java
        Commit graph MODIFY /icepush/trunk/icepush/core/src/main/java/org/icepush/NoopPushGroupManager.java
        Commit graph MODIFY /icepush/trunk/icepush/core/src/main/java/org/icepush/PushContext.java
        Commit graph MODIFY /icepush/trunk/icepush/core/src/main/javascript/application.js
        Commit graph ADD /icepush/trunk/icepush/core/src/main/java/org/icepush/servlet/AddNotifyBackURI.java
        Commit graph MODIFY /icepush/trunk/icepush/core/src/main/java/org/icepush/servlet/BrowserBoundServlet.java
        Commit graph ADD /icepush/trunk/icepush/core/src/main/java/org/icepush/servlet/RemoveNotifyBackURI.java

          People

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

            Dates

            • Created:
              Updated:
              Resolved: