ICEpush
  1. ICEpush
  2. PUSH-272

Move ConfigurationServer up the chain

          Details

          • Type: Bug Bug
          • Status: Closed
          • Priority: Major Major
          • Resolution: Duplicate
          • Affects Version/s: 3.3, EE-3.3.0.GA_P01
          • Fix Version/s: 4.0.BETA, EE-3.3.0.GA_P02
          • Component/s: Push Library
          • Labels:
            None
          • Environment:
            ICEpush

            Description

            It is possible for creating a BlockingConnectionServer that maps to a BrowserID set to null. This is incorrect and due to the ConfigurationServer being down the chain too far. It should be moved up to before a BlockingConnectionServer is created to ensure the incoming Request contains the ice.push.browser.

              Activity

              Ascending order - Click to sort in descending order
              Hide
              Permalink
              Jack Van Ooststroom added a comment -

              Deleting core/src/main/java/org/icepush/ConfigurationServer.java
              Sending core/src/main/java/org/icepush/servlet/AsyncAdaptingServlet.java
              Sending core/src/main/java/org/icepush/servlet/BrowserBoundServlet.java
              Adding core/src/main/java/org/icepush/servlet/ConfigurationServlet.java
              Sending core/src/main/java/org/icepush/servlet/MainServlet.java
              Transmitting file data ....
              Committed revision 37782.

              Sending core-ee/src/main/java/com/icesoft/icepush/BrowserBoundServlet.java
              Sending eps/src/main/java/com/icesoft/push/BlockingConnectionServer.java
              Transmitting file data ..
              Committed revision 35344.

              Show
              Jack Van Ooststroom added a comment - Deleting core/src/main/java/org/icepush/ConfigurationServer.java Sending core/src/main/java/org/icepush/servlet/AsyncAdaptingServlet.java Sending core/src/main/java/org/icepush/servlet/BrowserBoundServlet.java Adding core/src/main/java/org/icepush/servlet/ConfigurationServlet.java Sending core/src/main/java/org/icepush/servlet/MainServlet.java Transmitting file data .... Committed revision 37782. Sending core-ee/src/main/java/com/icesoft/icepush/BrowserBoundServlet.java Sending eps/src/main/java/com/icesoft/push/BlockingConnectionServer.java Transmitting file data .. Committed revision 35344.
              Hide
              Permalink
              Jack Van Ooststroom added a comment -

              Sending eps/src/main/java/com/icesoft/push/BlockingConnectionServer.java
              Sending eps/src/main/java/com/icesoft/push/servlet/BrowserBoundServlet.java
              Transmitting file data ..
              Committed revision 35345.

              Show
              Jack Van Ooststroom added a comment - Sending eps/src/main/java/com/icesoft/push/BlockingConnectionServer.java Sending eps/src/main/java/com/icesoft/push/servlet/BrowserBoundServlet.java Transmitting file data .. Committed revision 35345.
              Hide
              Permalink
              Jack Van Ooststroom added a comment -

              The fix to PUSH-276 seemed to have addressed the same issue. Parts of the commits for this JIRA have been reverted accordingly. Closing this one.

              Show
              Jack Van Ooststroom added a comment - The fix to PUSH-276 seemed to have addressed the same issue. Parts of the commits for this JIRA have been reverted accordingly. Closing this one.

                People

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

                  Dates

                  • Created:
                    Updated:
                    Resolved: