From b7b5fc993d4cece92833c95487efe63320537058 Mon Sep 17 00:00:00 2001
From: Sebastian Schmidt <mrschmidt@google.com>
Date: Mon, 13 Jan 2020 14:57:24 -0800
Subject: [PATCH] fix: don't format log message if logging is disable (#874)

---
 dev/src/logger.ts | 16 +++++++++-------
 1 file changed, 9 insertions(+), 7 deletions(-)

diff --git a/dev/src/logger.ts b/dev/src/logger.ts
index 7bc542e8c..28296f3f1 100644
--- a/dev/src/logger.ts
+++ b/dev/src/logger.ts
@@ -22,7 +22,7 @@ import {validateFunction} from './validate';
 let libVersion: string;
 
 /*! The external function used to emit logs. */
-let logFunction = (msg: string) => {};
+let logFunction: ((msg: string) => void) | undefined = undefined;
 
 /**
  * Log function to use for debug output. By default, we don't perform any
@@ -38,12 +38,14 @@ export function logger(
 ): void {
   requestTag = requestTag || '#####';
 
-  const formattedMessage = util.format(logMessage, ...additionalArgs);
-  const time = new Date().toISOString();
-  logFunction(
-    `Firestore (${libVersion}) ${time} ${requestTag} [${methodName}]: ` +
-      formattedMessage
-  );
+  if (logFunction) {
+    const formattedMessage = util.format(logMessage, ...additionalArgs);
+    const time = new Date().toISOString();
+    logFunction(
+      `Firestore (${libVersion}) ${time} ${requestTag} [${methodName}]: ` +
+        formattedMessage
+    );
+  }
 }
 
 /**