mirror of
https://github.com/nowcommunity/nowdatabase.git
synced 2026-10-12 19:13:52 +03:00
Remove json-override from response logger as res.json calls .send causing double logging
This commit is contained in:
@@ -32,6 +32,7 @@ const app = express()
|
||||
app.use(express.json({ limit: '50mb' }))
|
||||
app.use(cors())
|
||||
app.use(compression())
|
||||
|
||||
app.use(responseLogger)
|
||||
|
||||
if (!ENABLE_WRITE && RUNNING_ENV !== 'dev') {
|
||||
|
||||
@@ -14,15 +14,10 @@ export const requestLogger: Middleware = (req, _res, next) => {
|
||||
}
|
||||
|
||||
export const responseLogger: Middleware = (req, res, next) => {
|
||||
const originalJson = res.json
|
||||
const originalSend = res.send
|
||||
req.start = new Date().getTime()
|
||||
const getLogString = (req: express.Request) =>
|
||||
`${res.statusCode} ${req.method}\t${new Date().getTime() - (req.start ?? 0)} ms\t${req.baseUrl}${req.url} ${req.user ? `Username: '${req.user.username}'` : ''}`
|
||||
res.json = function (body) {
|
||||
logger.info(getLogString(req))
|
||||
return originalJson.call(this, body)
|
||||
}
|
||||
res.send = function (body) {
|
||||
logger.info(getLogString(req))
|
||||
return originalSend.call(this, body)
|
||||
|
||||
Reference in New Issue
Block a user