feat: add option to disable file logging

mytlogos · Oct 26, 2022 · ef2360c · ef2360c
1 parent 3517326
commit ef2360c
Show file tree

Hide file tree

Showing 2 changed files with 21 additions and 11 deletions.
diff --git a/packages/core/src/env.ts b/packages/core/src/env.ts
@@ -36,6 +36,7 @@ interface Config {
   development: boolean;
   stopScrapeEvents: boolean;
   lokiUrl?: string;
+  disableFileLogging: boolean;
 }
 
 /**
@@ -55,6 +56,8 @@ const appConfig: Config = {
   development: (process.env.NODE_ENV || config.parsed.NODE_ENV) !== "production",
   stopScrapeEvents: !!Number(process.env.stopScrapeEvents || config.parsed.stopScrapeEvents),
   lokiUrl: process.env.lokiUrl || config.parsed.lokiUrl,
+  disableFileLogging:
+    ((process.env.disableFileLogging || config.parsed.disableFileLogging) ?? "").toLocaleLowerCase() === "true",
 };
 
 const optionalVars = new Set(["lokiUrl"] as Array<keyof Config>);

diff --git a/packages/core/src/logger.ts b/packages/core/src/logger.ts
@@ -70,17 +70,6 @@ const logger = winston.createLogger({
     new winston.transports.Console(),
   ],
   transports: [
-    //
-    // - Write to all logs with level `info` and below to `combined.log`
-    // - Write all logs error (and below) to `error.log`.
-    //
-    new DailyRotateFile({
-      filename: `${filePrefix}combined.log-%DATE%`,
-    }),
-    new DailyRotateFile({
-      filename: `${filePrefix}error.log-%DATE%`,
-      level: "error",
-    }),
     new winston.transports.Console({
       format: format.combine(
         format.colorize(),
@@ -95,6 +84,24 @@ const logger = winston.createLogger({
   ],
 });
 
+if (!env.disableFileLogging) {
+  //
+  // - Write to all logs with level `info` and below to `combined.log`
+  // - Write all logs error (and below) to `error.log`.
+  //
+  logger.add(
+    new DailyRotateFile({
+      filename: `${filePrefix}combined.log-%DATE%`,
+    }),
+  );
+  logger.add(
+    new DailyRotateFile({
+      filename: `${filePrefix}error.log-%DATE%`,
+      level: "error",
+    }),
+  );
+}
+
 if (env.lokiUrl) {
   logger.add(
     new LokiTransport({