Merge 70459c6 into 4bcc170

docs/api.md

@@ -15,6 +15,7 @@
   * [logger.level](#level)
   * [logger.isLevelEnabled()](#islevelenabled)
   * [logger.levels](#levels)
+  * [logger\[Symbol.for('pino.serializers')\]](#serializers)
   * [Event: 'level-change'](#level-change)
   * [logger.version](#version)
   * [logger.LOG_VERSION](#log_version)
@@ -543,6 +544,13 @@ $ node -p "require('pino')().levels"
 
 * See [`logger.level`](#level)
 
+<a id="serializers"></a>
+### logger\[Symbol.for('pino.serializers')\]
+
+Returns the serializers for the logger instance, as seen by the logger
+itself. Child loggers might reuse the same instance of their parent, if
+there are no changes.
+
 <a id="level-change"></a>
 ### Event: 'level-change'
 

lib/symbols.js

@@ -10,7 +10,6 @@ const parsedChindingsSym = Symbol('pino.parsedChindings')
 
 const asJsonSym = Symbol('pino.asJson')
 const writeSym = Symbol('pino.write')
-const serializersSym = Symbol('pino.serializers')
 const redactFmtSym = Symbol('pino.redactFmt')
 
 const timeSym = Symbol('pino.time')
@@ -21,6 +20,9 @@ const endSym = Symbol('pino.end')
 const formatOptsSym = Symbol('pino.formatOpts')
 const messageKeyStringSym = Symbol('pino.messageKeyString')
 
+// public symbols, no need to use the same pino
+// version for these
+const serializersSym = Symbol.for('pino.serializers')
 const wildcardGsym = Symbol.for('pino.*')
 const needsMetadataGsym = Symbol.for('pino.metadata')
 

test/serializers.test.js

@@ -91,6 +91,29 @@ test('child does not overwrite parent serializers', async ({is}) => {
   is((await o2).test, 'child')
 })
 
+test('Symbol.for(\'pino.serializers\')', async ({is, isNot}) => {
+  const stream = sink()
+  const parent = pino({ serializers: parentSerializers }, stream)
+  const child = parent.child({a: 'property'})
+
+  is(parent[Symbol.for('pino.serializers')], parentSerializers)
+  is(child[Symbol.for('pino.serializers')], parentSerializers)
+
+  const child2 = parent.child({
+    serializers: {
+      a
+    }
+  })
+
+  function a () {
+    return 'hello'
+  }
+
+  isNot(child2[Symbol.for('pino.serializers')], parentSerializers)
+  is(child2[Symbol.for('pino.serializers')].a, a)
+  is(child2[Symbol.for('pino.serializers')].test, parentSerializers.test)
+})
+
 test('children inherit parent serializers', async ({is}) => {
   const stream = sink()
   const parent = pino({ serializers: parentSerializers }, stream)