From 6f6a9ea7516fbd87b9b41c2a8878f8794cb07c3a Mon Sep 17 00:00:00 2001
From: Adri Van Houdt <adri.van.houdt@gmail.com>
Date: Wed, 9 Mar 2022 21:11:47 +0100
Subject: [PATCH] fix: allow logging numbers when mergeHapiLogData is true

---
 index.js |  2 +-
 test.js  | 24 ++++++++++++++++++++++++
 2 files changed, 25 insertions(+), 1 deletion(-)

diff --git a/index.js b/index.js
index 6d011d7..a9cb1af 100644
--- a/index.js
+++ b/index.js
@@ -252,7 +252,7 @@ async function register (server, options) {
 
     var logObject
     if (mergeHapiLogData) {
-      if (typeof data === 'string') {
+      if (typeof data === 'string' || typeof data === 'number') {
         data = { [messageKey]: data }
       }
 
diff --git a/test.js b/test.js
index aa3d5f0..8087a9b 100644
--- a/test.js
+++ b/test.js
@@ -1336,6 +1336,30 @@ experiment('logging with mergeHapiLogData option enabled', () => {
     await finish
   })
 
+  test('when data is a number, merge it as msg property', async () => {
+    const server = getServer()
+    let done
+    const finish = new Promise(function (resolve, reject) {
+      done = resolve
+    })
+    const stream = sink(data => {
+      expect(data).to.include({ msg: 1 })
+      done()
+    })
+    const plugin = {
+      plugin: Pino,
+      options: {
+        stream: stream,
+        level: 'info',
+        mergeHapiLogData: true
+      }
+    }
+
+    await server.register(plugin)
+    server.log(['info'], 1)
+    await finish
+  })
+
   test('respects `messageKey` option', async () => {
     const server = getServer()
     let done