Merge pull request #50 from eoinsha/v2.x.x

Backport ignorePaths support to v2.x.x

index.js

@@ -1,6 +1,7 @@
 'use strict'
 
 const pino = require('pino')
+const nullLogger = require('abstract-logging')
 
 const levels = ['trace', 'debug', 'info', 'warn', 'error']
 module.exports.levelTags = {
@@ -39,13 +40,24 @@ function register (server, options, next) {
     return next(new Error('invalid tag levels'))
   }
 
+  var ignoreTable = {}
+  if (options.ignorePaths) {
+    for (let i = 0; i < options.ignorePaths.length; i++) {
+      ignoreTable[options.ignorePaths[i]] = true
+    }
+  }
+
   const mergeHapiLogData = options.mergeHapiLogData
 
   // expose logger as 'server.logger()'
   server.decorate('server', 'logger', () => logger)
 
   // set a logger for each request
   server.ext('onRequest', (request, reply) => {
+    if (options.ignorePaths && ignoreTable[request.url.path]) {
+      request.logger = nullLogger
+      return reply.continue()
+    }
     request.logger = logger.child({ req: request })
     reply.continue()
   })

package.json

@@ -30,6 +30,7 @@
     "standard": "^10.0.0"
   },
   "dependencies": {
+    "abstract-logging": "^1.0.0",
     "pino": "^4.10.2"
   },
   "repository": {

test.js

@@ -721,3 +721,33 @@ experiment('logging with request payload', () => {
     })
   })
 })
+
+experiment('ignore request logs for paths in ignorePaths', () => {
+  test('when path matches entry in ignorePaths, nothing should be logged', (done) => {
+    const server = getServer()
+    const stream = sink((data) => {
+      expect(data.req.url).to.not.equal('/ignored')
+      done()
+    })
+    const logger = require('pino')(stream)
+    const plugin = {
+      register: Pino.register,
+      options: {
+        instance: logger,
+        ignorePaths: ['/ignored']
+      }
+    }
+
+    server.register(plugin, (err) => {
+      expect(err).to.be.undefined()
+      server.inject({
+        method: 'PUT',
+        url: '/ignored'
+      })
+      server.inject({
+        method: 'PUT',
+        url: '/'
+      })
+    })
+  })
+})