Merge 266c3e2 into 11a6235

index.js

@@ -70,7 +70,7 @@ async function register (server, options) {
 
   // set a logger for each request
   server.ext('onRequest', (request, h) => {
-    if (options.ignorePaths && ignoreTable[request.url.path]) {
+    if (options.ignorePaths && ignoreTable[request.url.pathname]) {
       request.logger = nullLogger
       return h.continue
     }
@@ -106,7 +106,7 @@ async function register (server, options) {
 
   // log when a request completes
   tryAddEvent(server, options, 'on', 'response', function (request) {
-    if (options.ignorePaths && ignoreTable[request.url.path]) {
+    if (options.ignorePaths && ignoreTable[request.url.pathname]) {
       return
     }
 

test.js

@@ -5,6 +5,7 @@ const Lab = require('lab')
 const Hoek = require('hoek')
 const split = require('split2')
 const writeStream = require('flush-write-stream')
+const URL = require('url').URL
 const promisify = require('util').promisify
 const sleep = promisify(setTimeout)
 
@@ -1001,7 +1002,8 @@ experiment('ignore request logs for paths in ignorePaths', () => {
       resolver = resolve
     })
     const stream = sink((data) => {
-      expect(data.req.url).to.not.equal('/ignored')
+      const url = new URL(data.req.url)
+      expect(url.pathname).to.equal('/')
       resolver()
     })
     const logger = require('pino')(stream)
@@ -1037,8 +1039,9 @@ experiment('ignore response logs for paths in ignorePaths', () => {
       resolver = resolve
     })
     const stream = sink((data) => {
+      const url = new URL(data.req.url)
+      expect(url.pathname).to.equal('/')
       expect(data.msg).to.equal('request completed')
-      expect(data.req.url).to.not.equal('/ignored')
       resolver()
     })
     const logger = require('pino')(stream)