fix: config deep merge

examples/plugin-with-logger.js

@@ -0,0 +1,19 @@
+'use strict'
+
+module.exports = function (fastify, options, next) {
+  fastify.get('/', function (req, reply) {
+    reply.send({ hello: 'world' })
+  })
+  next()
+}
+
+module.exports.options = {
+  logger: {
+    redact: {
+      censor: '***',
+      paths: [
+        'foo',
+      ]
+    },
+  }
+}

start.js

@@ -5,6 +5,10 @@
 require('dotenv').config()
 const isDocker = require('is-docker')
 const closeWithGrace = require('close-with-grace')
+const deepmerge = require('@fastify/deepmerge')({ 
+  cloneProtoObject(obj){ return obj }
+ })
+
 const listenAddressDocker = '0.0.0.0'
 const watch = require('./lib/watch')
 const parseArgs = require('./args')
@@ -99,7 +103,7 @@ async function runFastify (args, additionalOptions, serverOptions) {
   const defaultLogger = {
     level: opts.logLevel
   }
-  const options = {
+  let options = {
     logger: logger || defaultLogger,
 
     pluginTimeout: opts.pluginTimeout
@@ -127,12 +131,14 @@ async function runFastify (args, additionalOptions, serverOptions) {
   }
 
   if (serverOptions) {
-    Object.assign(options, serverOptions)
+    options = deepmerge(options, serverOptions)
+  }
+
+  if (opts.options && file.options) {
+    options = deepmerge(options, file.options)
   }
 
-  const fastify = Fastify(
-    opts.options ? Object.assign(options, file.options) : options
-  )
+  const fastify = Fastify(options)
 
   if (opts.prefix) {
     opts.pluginOptions.prefix = opts.prefix

test/helper.test.js

@@ -124,3 +124,29 @@ test('should start fastify with custom logger configuration', async t => {
   t.same(lines.length, 1)
   t.same(app.log.level, 'warn')
 })
+
+test('should merge the CLI and FILE configs', async t => {
+  const argv = ['./examples/plugin-with-logger.js', '--options']
+
+  const lines = []
+  const dest = new stream.Writable({
+    write: function (chunk, enc, cb) {
+      lines.push(JSON.parse(chunk))
+      cb()
+    }
+  })
+
+  const app = await helper.listen(argv, {}, {
+    logger: {
+      level: 'warn',
+      stream: dest
+    }
+  })
+  t.teardown(() => app.close())
+  app.log.info('test')
+  t.same(lines.length, 0)
+  app.log.warn({ foo: 'test' })
+  t.same(app.log.level, 'warn')
+  t.same(lines.length, 1)
+  t.same(lines[0].foo, '***')
+})