Add disableServerLogging option

README.md

@@ -130,6 +130,10 @@ var $logger = beaver.Logger({
 
     // Use sendBeacon if supported rather than XHR to send logs; defaults to false
     enableSendBeacon: true,
+
+    // Stop log events from being sent to server
+    disableServerLogging: false,
+
 });
 ```
 

src/logger.js

@@ -23,7 +23,8 @@ type LoggerOptions = {|
     logLevel? : $Values<typeof LOG_LEVEL>,
     transport? : Transport,
     flushInterval? : number,
-    enableSendBeacon? : boolean
+    enableSendBeacon? : boolean,
+    disableServerLogging? : boolean
 |};
 
 type ClientPayload = { [string] : ?string | ?boolean };
@@ -72,7 +73,7 @@ function extendIfDefined(target : { [string] : string | boolean }, source : { [s
     }
 }
 
-export function Logger({ url, prefix, logLevel = DEFAULT_LOG_LEVEL, transport = httpTransport, flushInterval = FLUSH_INTERVAL, enableSendBeacon = false } : LoggerOptions) : LoggerType {
+export function Logger({ url, prefix, logLevel = DEFAULT_LOG_LEVEL, transport = httpTransport, flushInterval = FLUSH_INTERVAL, enableSendBeacon = false, disableServerLogging = false } : LoggerOptions) : LoggerType {
 
     let events : Array<{| level : $Values<typeof LOG_LEVEL>, event : string, payload : Payload |}> = [];
     let tracking : Array<Payload> = [];
@@ -154,6 +155,10 @@ export function Logger({ url, prefix, logLevel = DEFAULT_LOG_LEVEL, transport =
 
     function enqueue(level : $Values<typeof LOG_LEVEL>, event : string, payload : Payload) {
 
+        if (disableServerLogging) {
+            return;
+        }
+
         events.push({
             level,
             event,

test/test.js

@@ -128,4 +128,48 @@ describe('beaver-logger tests', () => {
             }
         });
     });
+
+    it('should not send event to server if disableServerLogging is true', () => {
+        const $logger = Logger({
+            url:                  '/test/api/log',
+            disableServerLogging: true
+        });
+
+        const browserConsoleLogs = [];
+
+        window.console.info = (args) => browserConsoleLogs.push(args);
+
+        $logger.info('browser_only_log', {
+            foo: 'bar',
+            bar: true
+        });
+
+        let handlerCalled = false;
+
+        const logEndpoint = $mockEndpoint.register({
+            method:  'POST',
+            uri:     '/test/api/log',
+            handler: () => {
+                handlerCalled = true;
+                return {};
+            }
+        });
+
+        let apiCallsFired = true;
+        logEndpoint.expectCalls();
+        return $logger.flush().then(() => {
+            try {
+                logEndpoint.done(); // will throw if no API calls received by logEndpoint
+            } catch (e) {
+                apiCallsFired = false;
+            }
+            if (handlerCalled || apiCallsFired) {
+                throw new Error('Expected no API calls to be fired');
+            }
+
+            if (!browserConsoleLogs.includes('browser_only_log')) {
+                throw new Error('Expected events to be logged on browser');
+            }
+        });
+    });
 });