From 25f3ca2be22f49b316e63bf1e70dfb6d19e29adf Mon Sep 17 00:00:00 2001 From: Brian Gough Date: Mon, 10 Jan 2022 14:51:38 +0000 Subject: [PATCH] Merge pull request #6295 from overleaf/bg-fix-error-in-request-logger [filestore] add missing params property to request logger GitOrigin-RevId: 528558db5a3350c319d561dc5bae6f8ce1f6aa7a --- services/filestore/app/js/RequestLogger.js | 22 +++------------------- 1 file changed, 3 insertions(+), 19 deletions(-) diff --git a/services/filestore/app/js/RequestLogger.js b/services/filestore/app/js/RequestLogger.js index 94a014eeca..af4b4c787f 100644 --- a/services/filestore/app/js/RequestLogger.js +++ b/services/filestore/app/js/RequestLogger.js @@ -45,25 +45,9 @@ class RequestLogger { const level = res.statusCode >= 500 ? 'err' : 'log' logger[level]( { - req: { - url: req.originalUrl || req.url, - route: routePath, - method: req.method, - referrer: req.headers.referer || req.headers.referrer, - 'remote-addr': - req.ip || - (req.socket && req.socket.remoteAddress) || - (req.socket && - req.socket.socket && - req.socket.socket.remoteAddress), - 'user-agent': req.headers['user-agent'], - 'content-length': req.headers['content-length'], - }, - res: { - 'content-length': res._headers['content-length'], - statusCode: res.statusCode, - 'response-time': responseTime, - }, + req, + res, + responseTimeMs: responseTime, info: req.requestLogger._logInfo, }, req.requestLogger._logMessage