feat: as a dev I want control over using local storage to store the s…

…ession id
Rebelizer · Apr 14, 2015 · fde95a0 · fde95a0
1 parent afd29ac
commit fde95a0
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 2 deletions.
diff --git a/bin/config/common.json b/bin/config/common.json
@@ -10,6 +10,7 @@
         //"filter": "^(statsd|warn|error|info|uncaught-exception)$",  list of types sent back to the server from the client if not set return all
         "debug": "^(statsd|warn|error|info|uncaught-exception)$", // list of type that should be sent to console.debug if not set do not debug
         "cookie": "_sid",
+        "lstorage": true, // use local storage and a cookie
         "url": "/__idat" // or //foobar.com/__idat
       }
       // "abnDashboardUrl": "/__abn"

diff --git a/src/instrumentation/local.js b/src/instrumentation/local.js
@@ -23,7 +23,10 @@ if(process.env.BROWSER_ENV) {
 
       sessionId = pellet.cookie.get(sessionKey);
       if (!sessionId || forceRegenerate) {
-        sessionId = localStorage.getItem(sessionKey);
+        if(pellet.config.instrumentation.lstorage) {
+          sessionId = localStorage.getItem(sessionKey);
+        }
+
         if(!sessionId || forceRegenerate) {
           sessionId = 'pID:' + s4() + s4() + s4() + '-' + s4();
         }
@@ -44,7 +47,10 @@ if(process.env.BROWSER_ENV) {
         }
       }
 
-      localStorage.setItem(sessionKey, sessionId);
+      if(pellet.config.instrumentation.lstorage) {
+        localStorage.setItem(sessionKey, sessionId);
+      }
+
       pellet.cookie.set(sessionKey, sessionId);
 
       return true;