@contrast/agent-bundle 5.47.0 → 5.48.0

package/node_modules/pino/test/broken-pipe.test.js

@@ -1,13 +1,15 @@
 'use strict'
 
-const t = require('tap')
-const { join } = require('path')
-const { fork } = require('child_process')
+const test = require('node:test')
+const assert = require('node:assert')
+const { join } = require('node:path')
+const { fork } = require('node:child_process')
+const tspl = require('@matteo.collina/tspl')
 const { once } = require('./helper')
 const pino = require('..')
 
 if (process.platform === 'win32') {
-  t.skip('skipping on windows')
+  console.log('skipping on windows')
   process.exit(0)
 }
 
@@ -17,31 +19,29 @@ if (process.env.CITGM) {
   // The failure does not reproduce locally or on our CI.
   // Skipping it is the only way to keep pino in CITGM.
   // https://github.com/nodejs/citgm/pull/1002#issuecomment-1751942988
-  t.skip('Skipping on Node.js core CITGM because it hangs on v18.x')
+  console.log('Skipping on Node.js core CITGM because it hangs on v18.x')
   process.exit(0)
 }
 
-function test (file) {
+function testFile (file) {
   file = join('fixtures', 'broken-pipe', file)
-  t.test(file, { parallel: true }, async ({ equal }) => {
+  test(file, async () => {
     const child = fork(join(__dirname, file), { silent: true })
     child.stdout.destroy()
 
     child.stderr.pipe(process.stdout)
 
     const res = await once(child, 'close')
-    equal(res, 0) // process exits successfully
+    assert.equal(res, 0) // process exits successfully
   })
 }
 
-t.jobs = 42
+testFile('basic.js')
+testFile('destination.js')
+testFile('syncfalse.js')
 
-test('basic.js')
-test('destination.js')
-test('syncfalse.js')
-
-t.test('let error pass through', ({ equal, plan }) => {
-  plan(3)
+test('let error pass through', async (t) => {
+  const plan = tspl(t, { plan: 3 })
   const stream = pino.destination({ sync: true })
 
   // side effect of the pino constructor is that it will set an
@@ -52,6 +52,8 @@ t.test('let error pass through', ({ equal, plan }) => {
   process.nextTick(() => stream.emit('error', new Error('kaboom')))
 
   stream.on('error', (err) => {
-    equal(err.message, 'kaboom')
+    plan.equal(err.message, 'kaboom')
   })
+
+  await plan
 })

package/node_modules/pino/test/browser-child.test.js

@@ -14,6 +14,18 @@ test('child has parent level', ({ end, same, is }) => {
   end()
 })
 
+test('child can set level at creation time', ({ end, same, is }) => {
+  const instance = pino({
+    level: 'error',
+    browser: {}
+  })
+
+  const child = instance.child({}, { level: 'info' }) // first bindings, then options
+
+  same(child.level, 'info')
+  end()
+})
+
 test('changing child level does not affect parent', ({ end, same, is }) => {
   const instance = pino({
     level: 'error',
@@ -94,6 +106,20 @@ test('changing child log level should not affect parent log behavior', ({ end, s
   end()
 })
 
+test('onChild callback should be called when new child is created', ({ end, pass, plan }) => {
+  plan(1)
+  const instance = pino({
+    level: 'error',
+    browser: {},
+    onChild: (_child) => {
+      pass('onChild callback was called')
+      end()
+    }
+  })
+
+  instance.child({})
+})
+
 function checkLogObjects (is, same, actual, expected) {
   is(actual.time <= Date.now(), true, 'time is greater than Date.now()')
 

package/node_modules/pino/test/browser-is-level-enabled.test.js

@@ -0,0 +1,101 @@
+'use strict'
+
+const { describe, test } = require('node:test')
+const assert = require('node:assert')
+const pino = require('../browser')
+
+const customLevels = {
+  trace: 10,
+  debug: 20,
+  info: 30,
+  warn: 40,
+  error: 50,
+  fatal: 60
+}
+
+describe('Default levels suite', () => {
+  test('can check if current level enabled', async () => {
+    const log = pino({ level: 'debug' })
+    assert.equal(true, log.isLevelEnabled('debug'))
+  })
+
+  test('can check if current level enabled when as object', async () => {
+    const log = pino({ asObject: true, level: 'debug' })
+    assert.equal(true, log.isLevelEnabled('debug'))
+  })
+
+  test('can check if level enabled after level set', async () => {
+    const log = pino()
+    assert.equal(false, log.isLevelEnabled('debug'))
+    log.level = 'debug'
+    assert.equal(true, log.isLevelEnabled('debug'))
+  })
+
+  test('can check if higher level enabled', async () => {
+    const log = pino({ level: 'debug' })
+    assert.equal(true, log.isLevelEnabled('error'))
+  })
+
+  test('can check if lower level is disabled', async () => {
+    const log = pino({ level: 'error' })
+    assert.equal(false, log.isLevelEnabled('trace'))
+  })
+
+  test('ASC: can check if child has current level enabled', async () => {
+    const log = pino().child({}, { level: 'debug' })
+    assert.equal(true, log.isLevelEnabled('debug'))
+    assert.equal(true, log.isLevelEnabled('error'))
+    assert.equal(false, log.isLevelEnabled('trace'))
+  })
+
+  test('can check if custom level is enabled', async () => {
+    const log = pino({
+      customLevels: { foo: 35 },
+      level: 'debug'
+    })
+    assert.equal(true, log.isLevelEnabled('foo'))
+    assert.equal(true, log.isLevelEnabled('error'))
+    assert.equal(false, log.isLevelEnabled('trace'))
+  })
+})
+
+describe('Custom levels suite', () => {
+  test('can check if current level enabled', async () => {
+    const log = pino({ level: 'debug', customLevels })
+    assert.equal(true, log.isLevelEnabled('debug'))
+  })
+
+  test('can check if level enabled after level set', async () => {
+    const log = pino({ customLevels })
+    assert.equal(false, log.isLevelEnabled('debug'))
+    log.level = 'debug'
+    assert.equal(true, log.isLevelEnabled('debug'))
+  })
+
+  test('can check if higher level enabled', async () => {
+    const log = pino({ level: 'debug', customLevels })
+    assert.equal(true, log.isLevelEnabled('error'))
+  })
+
+  test('can check if lower level is disabled', async () => {
+    const log = pino({ level: 'error', customLevels })
+    assert.equal(false, log.isLevelEnabled('trace'))
+  })
+
+  test('can check if child has current level enabled', async () => {
+    const log = pino().child({ customLevels }, { level: 'debug' })
+    assert.equal(true, log.isLevelEnabled('debug'))
+    assert.equal(true, log.isLevelEnabled('error'))
+    assert.equal(false, log.isLevelEnabled('trace'))
+  })
+
+  test('can check if custom level is enabled', async () => {
+    const log = pino({
+      customLevels: { foo: 35, ...customLevels },
+      level: 'debug'
+    })
+    assert.equal(true, log.isLevelEnabled('foo'))
+    assert.equal(true, log.isLevelEnabled('error'))
+    assert.equal(false, log.isLevelEnabled('trace'))
+  })
+})

package/node_modules/pino/test/browser-serializers.test.js

@@ -284,7 +284,6 @@ test('children inherit parent serializers', ({ end, is }) => {
 
 test('children serializers get called', ({ end, is }) => {
   const parent = pino({
-    test: 'this',
     browser: {
       serialize: true,
       write (o) {
@@ -301,7 +300,6 @@ test('children serializers get called', ({ end, is }) => {
 
 test('children serializers get called when inherited from parent', ({ end, is }) => {
   const parent = pino({
-    test: 'this',
     serializers: parentSerializers,
     browser: {
       serialize: true,

package/node_modules/pino/test/browser-transmit.test.js

@@ -347,3 +347,71 @@ test('extracts correct bindings and raw messages over multiple transmits', ({ en
 
   end()
 })
+
+test('does not log below configured level', ({ end, is }) => {
+  let message = null
+  const logger = pino({
+    level: 'info',
+    browser: {
+      write (o) {
+        message = o.msg
+      },
+      transmit: {
+        send () { }
+      }
+    }
+  })
+
+  logger.debug('this message is silent')
+  is(message, null)
+
+  end()
+})
+
+test('silent level prevents logging even with transmit', ({ end, fail }) => {
+  const logger = pino({
+    level: 'silent',
+    browser: {
+      write () {
+        fail('no data should be logged by the write method')
+      },
+      transmit: {
+        send () {
+          fail('no data should be logged by the send method')
+        }
+      }
+    }
+  })
+
+  Object.keys(pino.levels.values).forEach((level) => {
+    logger[level]('ignored')
+  })
+
+  end()
+})
+
+test('does not call send when transmit.level is set to silent', ({ end, fail, is }) => {
+  let c = 0
+  const logger = pino({
+    level: 'trace',
+    browser: {
+      write () {
+        c++
+      },
+      transmit: {
+        level: 'silent',
+        send () {
+          fail('no data should be logged by the transmit method')
+        }
+      }
+    }
+  })
+
+  const levels = Object.keys(pino.levels.values)
+  levels.forEach((level) => {
+    logger[level]('message')
+  })
+
+  is(c, levels.length, 'write must be called exactly once per level')
+  end()
+})

package/node_modules/pino/test/browser.test.js

@@ -166,7 +166,44 @@ test('opts.browser.asObject logs pino-like object to console', ({ end, ok, is })
   end()
 })
 
-test('opts.browser.formatters logs pino-like object to console', ({ end, ok, is }) => {
+test('opts.browser.asObject uses opts.messageKey in logs', ({ end, ok, is }) => {
+  const messageKey = 'message'
+  const instance = require('../browser')({
+    messageKey,
+    browser: {
+      asObject: true,
+      write: function (o) {
+        is(o.level, 30)
+        is(o[messageKey], 'test')
+        ok(o.time)
+      }
+    }
+  })
+
+  instance.info('test')
+  end()
+})
+
+test('opts.browser.asObjectBindingsOnly passes the bindings but keep the message unformatted', ({ end, ok, is, deepEqual }) => {
+  const messageKey = 'message'
+  const instance = require('../browser')({
+    messageKey,
+    browser: {
+      asObjectBindingsOnly: true,
+      write: function (o, msg, ...args) {
+        is(o.level, 30)
+        ok(o.time)
+        is(msg, 'test %s')
+        deepEqual(args, ['foo'])
+      }
+    }
+  })
+
+  instance.info('test %s', 'foo')
+  end()
+})
+
+test('opts.browser.formatters (level) logs pino-like object to console', ({ end, ok, is }) => {
   const info = console.info
   console.info = function (o) {
     is(o.level, 30)
@@ -189,7 +226,7 @@ test('opts.browser.formatters logs pino-like object to console', ({ end, ok, is
   end()
 })
 
-test('opts.browser.formatters logs pino-like object to console', ({ end, ok, is }) => {
+test('opts.browser.formatters (log) logs pino-like object to console', ({ end, ok, is }) => {
   const info = console.info
   console.info = function (o) {
     is(o.level, 30)
@@ -213,40 +250,88 @@ test('opts.browser.formatters logs pino-like object to console', ({ end, ok, is
   end()
 })
 
-test('opts.browser.write func log single string', ({ end, ok, is }) => {
-  const instance = pino({
+test('opts.browser.serialize and opts.browser.transmit only serializes log data once', ({ end, ok, is }) => {
+  const instance = require('../browser')({
+    serializers: {
+      extras (data) {
+        return { serializedExtras: data }
+      }
+    },
     browser: {
+      serialize: ['extras'],
+      transmit: {
+        level: 'info',
+        send (level, o) {
+          is(o.messages[0].extras.serializedExtras, 'world')
+        }
+      }
+    }
+  })
+
+  instance.info({ extras: 'world' }, 'test')
+  end()
+})
+
+test('opts.browser.serialize and opts.asObject only serializes log data once', ({ end, ok, is }) => {
+  const instance = require('../browser')({
+    serializers: {
+      extras (data) {
+        return { serializedExtras: data }
+      }
+    },
+    browser: {
+      serialize: ['extras'],
+      asObject: true,
       write: function (o) {
-        is(o.level, 30)
-        is(o.msg, 'test')
-        ok(o.time)
+        is(o.extras.serializedExtras, 'world')
       }
     }
   })
-  instance.info('test')
 
+  instance.info({ extras: 'world' }, 'test')
   end()
 })
 
-test('opts.browser.write func string joining', ({ end, ok, is }) => {
+test('opts.browser.serialize, opts.asObject and opts.browser.transmit only serializes log data once', ({ end, ok, is }) => {
+  const instance = require('../browser')({
+    serializers: {
+      extras (data) {
+        return { serializedExtras: data }
+      }
+    },
+    browser: {
+      serialize: ['extras'],
+      asObject: true,
+      transmit: {
+        send (level, o) {
+          is(o.messages[0].extras.serializedExtras, 'world')
+        }
+      }
+    }
+  })
+
+  instance.info({ extras: 'world' }, 'test')
+  end()
+})
+
+test('opts.browser.write func log single string', ({ end, ok, is }) => {
   const instance = pino({
     browser: {
       write: function (o) {
         is(o.level, 30)
-        is(o.msg, 'test test2 test3')
+        is(o.msg, 'test')
         ok(o.time)
       }
     }
   })
-  instance.info('test %s %s', 'test2', 'test3')
+  instance.info('test')
 
   end()
 })
 
-test('opts.browser.write func string joining when asObject is true', ({ end, ok, is }) => {
+test('opts.browser.write func string joining', ({ end, ok, is }) => {
   const instance = pino({
     browser: {
-      asObject: true,
       write: function (o) {
         is(o.level, 30)
         is(o.msg, 'test test2 test3')

package/node_modules/pino/test/complex-objects.test.js

@@ -1,11 +1,13 @@
 'use strict'
 
-const { test } = require('tap')
+const test = require('node:test')
+const assert = require('node:assert')
+const { PassThrough } = require('node:stream')
+
 const { sink, once } = require('./helper')
-const { PassThrough } = require('stream')
 const pino = require('../')
 
-test('Proxy and stream objects', async ({ equal }) => {
+test('Proxy and stream objects', async () => {
   const s = new PassThrough()
   s.resume()
   s.write('', () => {})
@@ -16,10 +18,10 @@ test('Proxy and stream objects', async ({ equal }) => {
 
   const result = await once(stream, 'data')
 
-  equal(result.obj, '[unable to serialize, circular reference is too complex to analyze]')
+  assert.equal(result.obj, '[unable to serialize, circular reference is too complex to analyze]')
 })
 
-test('Proxy and stream objects', async ({ equal }) => {
+test('Proxy and stream objects', async () => {
   const s = new PassThrough()
   s.resume()
   s.write('', () => {})
@@ -30,5 +32,5 @@ test('Proxy and stream objects', async ({ equal }) => {
 
   const result = await once(stream, 'data')
 
-  equal(result.p, '[unable to serialize, circular reference is too complex to analyze]')
+  assert.equal(result.p, '[unable to serialize, circular reference is too complex to analyze]')
 })

package/node_modules/pino/test/crlf.test.js

@@ -1,6 +1,8 @@
 'use strict'
 
-const { test } = require('tap')
+const test = require('node:test')
+const assert = require('node:assert')
+
 const writer = require('flush-write-stream')
 const pino = require('../')
 
@@ -13,20 +15,20 @@ function capture () {
   return ws
 }
 
-test('pino uses LF by default', async ({ ok }) => {
+test('pino uses LF by default', async () => {
   const stream = capture()
   const logger = pino(stream)
   logger.info('foo')
   logger.error('bar')
-  ok(/foo[^\r\n]+\n[^\r\n]+bar[^\r\n]+\n/.test(stream.data))
+  assert.ok(/foo[^\r\n]+\n[^\r\n]+bar[^\r\n]+\n/.test(stream.data))
 })
 
-test('pino can log CRLF', async ({ ok }) => {
+test('pino can log CRLF', async () => {
   const stream = capture()
   const logger = pino({
     crlf: true
   }, stream)
   logger.info('foo')
   logger.error('bar')
-  ok(/foo[^\n]+\r\n[^\n]+bar[^\n]+\r\n/.test(stream.data))
+  assert.ok(/foo[^\n]+\r\n[^\n]+bar[^\n]+\r\n/.test(stream.data))
 })