Closed Bug 954116 Opened 8 years ago Closed 8 years ago
Reduce the amount of data the twitter plugin logs into the Error Console
*** Original post on bio 681 at 2011-02-04 13:20:00 UTC *** Currently all the data from the twitter plugin (requests sent, received response) is logged. That's sometimes huge messages in the error console, and they are not really human readable. The messages come from the HTTP request implementation in jsProtoHelper, that uses the LOG function. I think that function should use a preference to determine what to do with logged debug data.
*** Original post on bio 681 at 2011-02-16 15:19:26 UTC *** This would be useful to all protocols to make the LOG command be able to handle debug levels, etc. Summary of ideas from IRC: All js protocol plugins can call LOG with some debug junk, and preferences to select which junk should appear (all, none, only important messages, only one protocol, ...) flo suggested: maybe just put the LOG function of jsProtoHelper into an Utils exported object, and then at the top of each protocol plugin code: const LOG = Utils.LOG.bind(null, "IRC");
Whiteboard: [0.3-blocking] → [0.3-blocking-final]
*** Original post on bio 681 at 2011-06-07 23:01:22 UTC *** The plan is to make a simple LOG command that checks the purple logging pref: http://lxr.instantbird.org/instantbird/source/instantbird/app/profile/all-instantbird.js#293
Assignee: nobody → clokep
Status: NEW → ASSIGNED
*** Original post on bio 681 as attmnt 709 at 2011-06-14 23:35:00 UTC *** This adds a DEBUG, LOG, WARN and ERROR statement that abide by the purple.debug.loglevel preference. WARN and ERROR include the line number and file.
*** Original post on bio 681 as attmnt 711 at 2011-06-15 13:18:00 UTC *** After some discussion on IRC and several iterations, I think this is what we agreed on.
Comment on attachment 8352451 [details] [diff] [review] Patch *** Original change on bio 681 attmnt 709 at 2011-06-15 13:18:36 UTC was without comment, so any subsequent comment numbers will be shifted ***
*** Original post on bio 681 as attmnt 712 at 2011-06-15 13:21:00 UTC *** Let's attach the right patch this time :)
Attachment #8352454 - Flags: review?(clokep)
Comment on attachment 8352453 [details] [diff] [review] Patch v2 *** Original change on bio 681 attmnt 711 at 2011-06-15 13:21:06 UTC was without comment, so any subsequent comment numbers will be shifted ***
Comment on attachment 8352454 [details] [diff] [review] Patch v2 *** Original change on bio 681 attmnt 712 at 2011-06-15 13:33:22 UTC *** Looks good! I like this a lot better than what I had. Removing the "Error: " is up to you, but it's probably not necessary anymore. Can't wait to convert my JS-IRC to use this! Thanks for cleaning it up.
Attachment #8352454 - Flags: review?(clokep) → review+
*** Original post on bio 681 at 2011-06-15 18:00:49 UTC *** https://hg.instantbird.org/instantbird/rev/f6ae36370e55
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → 1.0
You need to log in before you can comment on or make changes to this bug.