fix: remove log query string from url for every ctx log (#56)

Author: goshander

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

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}`,
+            },
+        });
     });
 });