ICEpush
  1. ICEpush
  2. PUSH-244

Turning on Level.FINE logging breaks setting the NotifyBackURI

          Details

          • Type: Bug Bug
          • Status: Closed
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: EE-3.2.0.GA, 3.3
          • Fix Version/s: EE-3.3.0.GA, 4.0.BETA, 4.0
          • Component/s: Push Library
          • Labels:
            None
          • Environment:
            ICEpush, Mobile Device
          • Workaround Exists:
            Yes
          • Workaround Description:
            Turn off Level.FINE logging for org.icepush.BlockingConnectionServer

            Description

            If the Level.FINE logging is switched on for ICEpush, the NotifyBackURI appears not to be set on the PushID object.

              Activity

              Ascending order - Click to sort in descending order
              Jack Van Ooststroom created issue -
              Jack Van Ooststroom made changes -
              Field Original Value New Value
              Assignee Jack Van Ooststroom [ jack.van.ooststroom ]
              Hide
              Permalink
              Jack Van Ooststroom added a comment -

              Sending src/main/java/org/icepush/BlockingConnectionServer.java
              Transmitting file data .
              Committed revision 35650.

              Show
              Jack Van Ooststroom added a comment - Sending src/main/java/org/icepush/BlockingConnectionServer.java Transmitting file data . Committed revision 35650.
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #35650 Thu May 30 07:32:33 MDT 2013 jack.van.ooststroom Fixed JIRA PUSH-244 : Turning on Level.FINE logging breaks setting the NotifyBackURI
              Files Changed
              Commit graph MODIFY /icepush/trunk/icepush/core/src/main/java/org/icepush/BlockingConnectionServer.java
              Jack Van Ooststroom made changes -
              Workaround Description Turn off Level.FINE logging for org.icepush.BlockingConnectionServer
              Workaround Exists Yes [ 10007 ]
              Jack Van Ooststroom made changes -
              Status Open [ 1 ] In Progress [ 3 ]
              Hide
              Permalink
              Jack Van Ooststroom added a comment -

              In one of the logging blocks in BlockingConnectionServer, namely for logging the "ICEpush metric", within this block the NotifyBackURI is also trying to be set, but only on the BlockingConnectionServer and not on the corresponding PushID instances. Retrieving the NotifyBackURI from the Request or the PushGroupManager is now moved into a separate private method, and both the logging block and the handling of the request now invoke this method, ensuring the new NotifyBackURI is set on both the BlockingConnectionServer and PushID. Even though I find it strange that we have this logic inside a logging block as well, I decided to leave the invocation of the new method in there for timing purposes of when the logging is printed. Marking this one as FIXED.

              Show
              Jack Van Ooststroom added a comment - In one of the logging blocks in BlockingConnectionServer, namely for logging the "ICEpush metric", within this block the NotifyBackURI is also trying to be set, but only on the BlockingConnectionServer and not on the corresponding PushID instances. Retrieving the NotifyBackURI from the Request or the PushGroupManager is now moved into a separate private method, and both the logging block and the handling of the request now invoke this method, ensuring the new NotifyBackURI is set on both the BlockingConnectionServer and PushID. Even though I find it strange that we have this logic inside a logging block as well, I decided to leave the invocation of the new method in there for timing purposes of when the logging is printed. Marking this one as FIXED.
              Jack Van Ooststroom made changes -
              Status In Progress [ 3 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Ken Fyten made changes -
              Fix Version/s 3.4 [ 10971 ]
              Ken Fyten made changes -
              Fix Version/s 4.0 [ 11383 ]
              Ken Fyten made changes -
              Status Resolved [ 5 ] Closed [ 6 ]

                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: