fix: dont lose this reference for loggers (#708)

Author: Uzlopak

src/index.ts

@@ -41,12 +41,17 @@ export function createAppAuth(options: StrategyOptions): AuthInterface {
     );
   }
 
-  const log = Object.assign(
-    {
-      warn: console.warn.bind(console),
-    },
-    options.log,
-  );
+  /**
+   * Mutate the logger to ensure it has a `warn` method.
+   *
+   * Some Loggers like pino need that the `this` reference points
+   * to the original object, so we cannot use `Object.assign` here.
+   */
+  const log = options.log || ({} as NonNullable<StrategyOptions["log"]>);
+  if (typeof log.warn !== "function") {
+    log.warn = console.warn.bind(console);
+  }
+
   const request =
     options.request ||
     defaultRequest.defaults({