ICEpush
  1. ICEpush
  2. PUSH-366

Lower Some NaaS code.icepush Logging levels

          Details

          • Type: Improvement Improvement
          • Status: Closed
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: None
          • Fix Version/s: 4.1, EE-3.3.0.GA_P04
          • Component/s: Push Library
          • Labels:
            None
          • Environment:
            n/a

            Description

            The push blocking connection logging is overly verbose. Because the send and receive confirmation logging statements are being logged at info level, it's hard to filter these out from more important logging. We should lower this logging statements to debug level.

            eg. in http://dev.bridgeit.io/push/code.icepush:

            info(logger, 'send confirmation: ' + ids);

            and

            info(logger, 'received confirmation: ' + confirmedIDs);

              Activity

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

              Sending core/src/main/javascript/pushid.expiry.js
              Transmitting file data .
              Committed revision 45004.

              Show
              Jack Van Ooststroom added a comment - Sending core/src/main/javascript/pushid.expiry.js Transmitting file data . Committed revision 45004.
              Hide
              Permalink
              Jack Van Ooststroom added a comment -

              I lowered the logging level from info to debug for these two logging statements. Marking this one as FIXED.

              Show
              Jack Van Ooststroom added a comment - I lowered the logging level from info to debug for these two logging statements. Marking this one as FIXED.
              Hide
              Permalink
              Jack Van Ooststroom added a comment -

              This one should probably be moved to ICEpush instead of ICEnotify.

              Show
              Jack Van Ooststroom added a comment - This one should probably be moved to ICEpush instead of ICEnotify.

                People

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

                  Dates

                  • Created:
                    Updated:
                    Resolved: