fix: support falsy value in sprintf substitution

adonisjs · Feb 8, 2024 · 73b4bed · 73b4bed
1 parent da1dec4
commit 73b4bed
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 8 deletions.
diff --git a/src/logger.ts b/src/logger.ts
@@ -173,13 +173,7 @@ export class Logger<Config extends LoggerConfig = LoggerConfig> {
       return
     }
 
-    if (values.length) {
-      ;(this.pino[level] as any)(mergingObject, message, ...values)
-    } else if (message) {
-      ;(this.pino[level] as any)(mergingObject, message)
-    } else {
-      ;(this.pino[level] as any)(mergingObject)
-    }
+    ;(this.pino[level] as any)(mergingObject, message, ...values)
   }
 
   /**

diff --git a/tests/logger.spec.ts b/tests/logger.spec.ts
@@ -69,7 +69,7 @@ test.group('Logger', () => {
     )
   })
 
-  test('handle sprintf subsitutes', ({ assert }) => {
+  test('handle sprintf substitutes', ({ assert }) => {
     const messages: string[] = []
 
     const logger = new Logger({
@@ -86,6 +86,7 @@ test.group('Logger', () => {
     logger.info('hello %s', 'info')
     logger.info('hello %s %o', 'info', { url: '/' })
     logger.info('hello %s %j', 'info', { url: '/' })
+    logger.info('total: %d', 0)
 
     assert.deepEqual(
       messages.map((m) => {
@@ -105,6 +106,10 @@ test.group('Logger', () => {
           level: 30,
           msg: `hello info ${JSON.stringify({ url: '/' })}`,
         },
+        {
+          level: 30,
+          msg: 'total: 0',
+        },
       ]
     )
   })