Opened on 01/11/2016 at 09:44:42 AM

Closed on 01/22/2016 at 03:05:56 PM

Last modified on 02/23/2016 at 06:50:57 AM

#3497 closed change (fixed)

Convert logging statements to Lumberjack

Reported by: pavelz Assignee:
Priority: P3 Milestone: Adblock-Browser-for-iOS-1.3.0
Module: Adblock-Browser-for-iOS Keywords:
Cc: mario Blocked By:
Blocking: Platform: Adblock Browser for iOS
Ready: yes Confidential: no
Tester: Scheer Verified working: yes
Review URL(s):

Description (last modified by pavelz)

Background

ABB and Kitt core historically used Cocoa provided NSLog() statement, which is the default and only one available logging facility. It has numerous deficits, most notably no severity level assignable.

What to change

Adopt CocoaLumberjack - done.
Walk trough the 110 of old style logging statements in Kitt core and carefully switch them over to the Lumberjack style. The event severity has to be decided on each one - the old method had not severity recognition.

Remark for testers

As this issue just covered refactoring work regarding the logging system, no usage relevant changes have been applied. Thus this change can't be verified working per se - a general QA verifying general functionality of the app is sufficient.

Exceptionally curious tester who knows Xcode and Devices inspector, can observe a change in formatting of application logging. The former format is the NSLog default:

2016-02-10 11:25:59.483 AdblockBrowser[57133:2533133] CUICatalog: Invalid asset name supplied: (null)

while the new format is more compressed

.W.|112558.609|2533133() Settings.bundle does not exist, will not register defaults

Attachments (0)

Change History (6)

comment:1 Changed on 01/11/2016 at 09:50:48 AM by mario

  • Cc mario added
  • Priority changed from Unknown to P3

comment:2 Changed on 01/22/2016 at 03:05:56 PM by pavelz

  • Resolution set to fixed
  • Status changed from new to closed

comment:3 Changed on 01/22/2016 at 03:21:33 PM by mario

  • Description modified (diff)
  • Ready set

comment:4 Changed on 02/09/2016 at 04:25:18 PM by mario

  • Milestone set to Adblock-Browser-for-iOS-next

Batch modify: added Adblock-Browser-for-iOS-next milestone.

comment:5 Changed on 02/10/2016 at 10:28:42 AM by pavelz

  • Description modified (diff)

comment:6 Changed on 02/23/2016 at 06:50:57 AM by scheer

  • Tester changed from Unknown to Scheer
  • Verified working set
  • I was able to see the change in formatting via the Device log. For example, I observed the changed format to 'W AdblockBrowser[2537] <Warning>: I |074834.620|32753() Native event '13' has no callbacks' on one event.

Add Comment

Modify Ticket

Change Properties
Action
as closed .
The resolution will be deleted. Next status will be 'reopened'.
to The owner will be changed from (none).
 
Note: See TracTickets for help on using tickets.