Bug 1447173 - Don't log addons on sync failure unless trace level logging is enabled. r=tcsc
☠☠ backed out by 3232f89ec8d8 ☠ ☠
authorui.manish <1991manish.kumar@gmail.com>
Tue, 27 Mar 2018 22:51:18 +0200
changeset 411282 3a278fcf0571eb06c8c1fcc074315ec6ff9c4b1e
parent 411281 2e425dfec285c730c7a43301a2f968d905a0cdf0
child 411283 d18246f06ea5e0dce155c786c08bd03aa7f67427
push id101601
push usernbeleuzu@mozilla.com
push dateSun, 01 Apr 2018 23:33:57 +0000
treeherdermozilla-inbound@d18246f06ea5 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewerstcsc
bugs1447173
milestone61.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1447173 - Don't log addons on sync failure unless trace level logging is enabled. r=tcsc
services/sync/modules/policies.js
--- a/services/sync/modules/policies.js
+++ b/services/sync/modules/policies.js
@@ -763,19 +763,19 @@ ErrorHandler.prototype = {
     let addons = [];
     try {
       addons = await AddonManager.getAddonsByTypes(["extension"]);
     } catch (e) {
       this._log.warn("Failed to dump addons", e);
     }
 
     let relevantAddons = addons.filter(x => x.isActive && !x.hidden);
-    this._log.debug("Addons installed", relevantAddons.length);
+    this._log.trace("Addons installed", relevantAddons.length);
     for (let addon of relevantAddons) {
-      this._log.debug(" - ${name}, version ${version}, id ${id}", addon);
+      this._log.trace(" - ${name}, version ${version}, id ${id}", addon);
     }
   },
 
   /**
    * Generate a log file for the sync that just completed
    * and refresh the input & output streams.
    */
   async resetFileLog() {