fix: Removing default logger prefix [BS] - fixes #1607

BrowserSync · Oct 6, 2018 · 257fba6 · 257fba6
1 parent cc8dfad
commit 257fba6
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 3 deletions.
diff --git a/packages/browser-sync-client/lib/log.ts b/packages/browser-sync-client/lib/log.ts
@@ -11,7 +11,7 @@ import { Inputs } from "./index";
 import { pluck } from "rxjs/operators/pluck";
 
 export function initLogger(options: IBrowserSyncOptions) {
-    const log = new Nanologger(options.logPrefix, {
+    const log = new Nanologger(options.logPrefix || "", {
         colors: { magenta: "#0F2634" }
     });
     return of(log);

diff --git a/packages/browser-sync-client/lib/messages/Connection.ts b/packages/browser-sync-client/lib/messages/Connection.ts
@@ -14,9 +14,14 @@ export function incomingConnection(
     return xs.pipe(
         withLatestFrom(inputs.option$.pipe(pluck("logPrefix"))),
         mergeMap(([x, logPrefix]) => {
+
+            const prefix = logPrefix
+                ? `${logPrefix}: `
+                : '';
+
             return of<any>(
                 setOptions(x),
-                Log.overlayInfo(`${logPrefix}: connected`)
+                Log.overlayInfo(`${prefix}connected`)
             );
         })
     );

diff --git a/packages/browser-sync/lib/logger.js b/packages/browser-sync/lib/logger.js
@@ -7,7 +7,6 @@ var _ = require("./lodash.custom");
 var template = "[{blue:%s}] ";
 
 var logger = require("eazy-logger").Logger({
-    prefix: template.replace("%s", "BS"),
     useLevelPrefixes: false
 });