ICEpush
  1. ICEpush
  2. PUSH-271

Move getBrowserID (from Header or Cookie) into Browser class

          Details

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

            Description

            Currently ICEpush has various duplicate locations for getting the Browser-ID from a Header or a Cookie. This logic should be moved to the new Browser class and used throughout.

              Activity

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

              Sending src/main/java/org/icepush/Browser.java
              Sending src/main/java/org/icepush/PushContext.java
              Sending src/main/java/org/icepush/servlet/BrowserDispatcher.java
              Transmitting file data ...
              Committed revision 37780.

              Show
              Jack Van Ooststroom added a comment - Sending src/main/java/org/icepush/Browser.java Sending src/main/java/org/icepush/PushContext.java Sending src/main/java/org/icepush/servlet/BrowserDispatcher.java Transmitting file data ... Committed revision 37780.
              Hide
              Permalink
              Jack Van Ooststroom added a comment -

              A static getBrowserID(HttpServletRequest) : String has been added to the Browser class to consolidate the logic. Underneath the method gets the Browser-ID from the different spots. Marking this one as FIXED.

              Show
              Jack Van Ooststroom added a comment - A static getBrowserID(HttpServletRequest) : String has been added to the Browser class to consolidate the logic. Underneath the method gets the Browser-ID from the different spots. Marking this one as FIXED.

                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: