💥 enforce apiKey parameters for loggers

Co-Authored-By: Malte Sehmer <[email protected]>

Author: pkiparra

src/models/controller.model.ts

@@ -96,6 +96,7 @@ export class Controller {
       const raceResult = await Promise.race([fetcherPromise, timeoutPromise]);
       if (raceResult === "TIMEOUT") {
         infoLogger(
+          "getContacts",
           `Fetching too slow, returning empty array.`,
           providerConfig.apiKey
         );

src/util/error.ts

@@ -9,7 +9,7 @@ import { errorLogger } from "./logger.util";
 export const throwAndDelegateError = (
   error: Error | AxiosError,
   source: string,
-  apiKey?: string,
+  apiKey: string | undefined,
   logMessage?: string
 ) => {
   const errorMessage =

src/util/logger.util.ts

@@ -10,7 +10,7 @@ import { anonymizeKey } from "./anonymize-key";
 export const infoLogger = (
   source: string,
   message: string,
-  apiKey?: string,
+  apiKey: string | undefined,
   ...args: unknown[]
 ): void => {
   logger(console.info, source, message, apiKey, args);
@@ -26,7 +26,7 @@ export const infoLogger = (
 export const errorLogger = (
   source: string,
   message: string,
-  apiKey?: string,
+  apiKey: string | undefined,
   ...args: unknown[]
 ): void => {
   logger(console.error, source, message, apiKey, args);
@@ -52,7 +52,7 @@ const logger = (
   logFn: (message?: any, ...optionalParams: any[]) => void,
   source: string,
   message: string,
-  apiKey?: string,
+  apiKey: string | undefined,
   ...args: unknown[]
 ): void => {
   // eslint-disable-next-line no-console