diff --git a/src/base-middleware.ts b/src/base-middleware.ts index 87b2c5f..00d35c5 100644 --- a/src/base-middleware.ts +++ b/src/base-middleware.ts @@ -47,7 +47,7 @@ export function setupBaseMiddleware(ctx: AppContext, expressApp: Express) { req.ctx.addLoggerExtra('req', { id: req.id, method: req.method, - url: ctx.utils.redactSensitiveQueryParams(req.url), + url: ctx.utils.redactSensitiveQueryParams(req.path), }); const requestStartedExtra = ctx.config.appDevMode diff --git a/src/tests/logging.test.ts b/src/tests/logging.test.ts index 19bfbcd..d8d18bb 100644 --- a/src/tests/logging.test.ts +++ b/src/tests/logging.test.ts @@ -227,7 +227,7 @@ describe('log system', () => { req: { id: requestId, method: 'GET', - url: `/log?q=${query}`, + url: '/log', }, res: { statusCode: '200', @@ -250,9 +250,25 @@ describe('log system', () => { req: { id: requestId, method: 'GET', - url: `/log?q=${query}`, + url: '/log', }, query, }); + + // first log with request + log = JSON.parse(logger.write.mock.calls?.pop() || '{}'); + + // check request log + expect(log).toMatchObject({ + msg: `[Express GET] Request started [${requestId}]`, + level: 30, + name: APP_NAME, + time: expect.any(Number), + req: { + id: requestId, + method: 'GET', + url: `/log?q=${query}`, + }, + }); }); });