AP-4411 Allow to provide redact option to loggers (#141)

* AP-4411 Test for redacted config.

* AP-4411 cleanup

* AP-4411 Code coverage.

README.md

@@ -24,6 +24,9 @@ The library provides methods to resolve the default logging configuration. Publi
   const loggerConfig = resolveLoggerConfiguration({
     logLevel: 'warn',
     nodeEnv: 'production',
+    redact: {
+      paths: ['path1', 'path2'],
+    },
   })
 
   const logger = pino(loggerConfig)

package.json

@@ -50,6 +50,7 @@
     "eslint": "^8.57.0",
     "eslint-plugin-import": "^2.29.1",
     "eslint-plugin-vitest": "0.4.1",
+    "pino-test": "^1.0.1",
     "prettier": "^3.3.1",
     "tmp": "^0.2.3",
     "typescript": "^5.4.5",

src/logging/loggerConfigResolver.spec.ts

@@ -1,4 +1,5 @@
 import { pino } from 'pino'
+import pinoTest from 'pino-test'
 import { expect } from 'vitest'
 
 import { resolveLoggerConfiguration } from './loggerConfigResolver'
@@ -17,6 +18,7 @@ describe('loggerConfigResolver', () => {
             "level": [Function],
           },
           "level": "warn",
+          "redact": undefined,
         }
       `)
     })
@@ -61,5 +63,25 @@ describe('loggerConfigResolver', () => {
         logger.warn('test')
       }).not.toThrow()
     })
+
+    it('redacts logs via provided config', () =>
+      new Promise((done) => {
+        const loggerConfig = resolveLoggerConfiguration({
+          logLevel: 'info',
+          nodeEnv: 'production',
+          redact: {
+            paths: ['password'],
+          },
+        })
+
+        const stream = pinoTest.sink()
+        stream.on('data', (obj) => {
+          expect(obj.password).toBe('[Redacted]')
+          done(true)
+        })
+
+        const logger = pino(loggerConfig, stream)
+        logger.info({ password: 'super password' }, 'Auth attempt.')
+      }))
   })
 })

src/logging/loggerConfigResolver.ts

@@ -1,11 +1,12 @@
-import type { Level, Logger, LoggerOptions } from 'pino'
+import type { Level, Logger, LoggerOptions, redactOptions } from 'pino'
 import { levels, pino } from 'pino'
 import pretty from 'pino-pretty'
 
 export type AppLoggerConfig = {
   logLevel: 'fatal' | 'error' | 'warn' | 'info' | 'debug' | 'trace' | 'silent'
   nodeEnv: 'production' | 'development' | 'test'
   base?: Record<string, unknown>
+  redact?: redactOptions
 }
 
 export type MonorepoAppLoggerConfig = AppLoggerConfig & {
@@ -14,7 +15,7 @@ export type MonorepoAppLoggerConfig = AppLoggerConfig & {
 }
 
 // Note that transports do not work in vitest, likely because pino attempts to run them in a separate worker
-/* c8 ignore next 24 */
+/* c8 ignore next 25 */
 export function resolveMonorepoLoggerConfiguration(
   appConfig: MonorepoAppLoggerConfig,
 ): LoggerOptions | Logger | boolean {
@@ -29,6 +30,7 @@ export function resolveMonorepoLoggerConfiguration(
         return { level: label }
       },
     },
+    redact: appConfig.redact,
     transport: {
       target: 'pino/file',
       options: {
@@ -63,5 +65,6 @@ export function resolveLoggerConfiguration(
         return { level }
       },
     },
+    redact: appConfig.redact,
   } satisfies LoggerOptions
 }