implement persistent debugLevel

lib/configuration.js

@@ -40,22 +40,50 @@ function Configuration(options) {
     self.agent = new Agent({
         log: self._log
     });
-
-
 }
 
 util.inherits(Configuration, EventEmitter);
 
 var confProto = Configuration.prototype;
 
-confProto.setDebugLevel = function (level) {
 
-    level = typeof level === 'string' ? level : level === true ? 'info' : null;
+var _DEBUG_LEVEL_KEY = 'http2-cache.configuration._debugLevel',
+    _DEBUG_LEVEL_DEFAULT = 'log';
+
+confProto.setDebugLevel = function (debugLevel, persist) {
+
+    debugLevel = typeof debugLevel === 'string' ? debugLevel : debugLevel === true ? 'log' : null;
+
+    // Setter with persist but not 'auto'
+    if (debugLevel !== 'auto' && persist === true) {
+
+        _DEBUG_LEVEL_DEFAULT = debugLevel;
+
+        // Use localStorage if available
+        if (typeof localStorage !== 'undefined') {
+            localStorage.setItem(_DEBUG_LEVEL_KEY, _DEBUG_LEVEL_DEFAULT);
+        }
+
+    // Compute default value from storet then const
+    } else if (debugLevel === 'auto') {
+
+        // Use localStorage if available
+        if (typeof localStorage !== 'undefined') {
+            debugLevel = localStorage.getItem(_DEBUG_LEVEL_KEY, _DEBUG_LEVEL_DEFAULT);
+        }
+
+        debugLevel = debugLevel || _DEBUG_LEVEL_DEFAULT;
+    }
 
     // Init debug/log
     if (this._log && this._log.hasOwnProperty('debugLevel')) {
-        this._log.debugLevel = level;
+        this._log.debugLevel = debugLevel;
     }
+
+    this.debug = debugLevel;
+
+    // Chainable
+    return this;
 };
 
 confProto.configuring = function () {

lib/http2-cache.js

@@ -3,13 +3,15 @@
     var Configuration = require('./configuration.js'),
         enableXHROverH2 = require('./xhr.js').enableXHROverH2;
 
-    var configuration = new Configuration({
-    	debug: false // true='info' or (info|debug|trace)
-    });
-    enableXHROverH2(XMLHttpRequest, configuration);
+    enableXHROverH2(XMLHttpRequest, new Configuration({
+    	debug: 'auto' // true='log' or (auto|log|debug|trace)
+    }));
 
     // To update debug level after injection:
     //- XMLHttpRequest.configuration.setDebugLevel('debug');
     //- XMLHttpRequest.configuration.setDebugLevel('trace');
 
+    // To persit update debug level after injection:
+    //- XMLHttpRequest.configuration.setDebugLevel('debug', true);
+
 }(this));

lib/logger.js

@@ -13,11 +13,25 @@ var defaultLogger = {
 };
 
 var consoleLogger = {
-  debugLevel: 'info', // 'info|debug|trace'
+  debugLevel: 'log', // 'log|debug|trace'
+  log: console.log,
   fatal: console.error,
   error: console.error,
   warn : console.warning || console.info,
-  info : console.info,
+  //info : console.info,
+  // Displayed when debugLevel is (trace|debug)
+  info: function (data, ctx) {
+    var info = console.info;
+    if (
+      consoleLogger.debugLevel === 'trace' || 
+        consoleLogger.debugLevel === 'debug'
+    ) {
+      var args = (typeof ctx === 'string' ? [ctx, data] : arguments);
+      info.apply(console, args);
+    } 
+    return this; 
+  },
+  // Displayed when debugLevel is (trace|debug)
   debug: function (data, ctx) {
     var debug = console.debug || console.info;
     if (
@@ -27,24 +41,26 @@ var consoleLogger = {
       var args = (typeof ctx === 'string' ? [ctx, data] : arguments);
       debug.apply(console, args);
     } 
+    return this; 
   },
-  // Trace only
+  // Displayed when debugLevel is trace only
   trace: function (data, ctx) {
     var debug = console.debug || console.info;
     if (consoleLogger.debugLevel === 'trace') {
-      consoleLogger.debug(data, ctx);
+      return consoleLogger.debug(data, ctx);
     } 
+    return this; 
   },
   // Trace only
   child: function(msg) {
     if (consoleLogger.debugLevel === 'trace') {
-      console.info(msg);
+      return console.info(msg);
     } 
-  	return this; 
+    return this; 
   }
 };
 
 module.exports = {
-	consoleLogger: consoleLogger,
-	defaultLogger: defaultLogger
+  consoleLogger: consoleLogger,
+  defaultLogger: defaultLogger
 };