archive-gh-me/log4js-node
1
0
Fork 0
mirror of https://github.com/log4js-node/log4js-node.git synced 2025-12-08 19:26:01 +00:00
Code Issues Projects Releases Wiki Activity

Merge pull request #1179 from log4js-node/fixes-connectLogger

Browse Source 
fix: connectLogger not logging on close
This commit is contained in:
Lam Wei Li 2022-01-28 00:11:36 +08:00 committed by GitHub
commit 5f33c8ca24
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 11 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
lib/connect-logger.js
View File

@ -273,7 +273,12 @@ module.exports = function getLogger(logger4js, options) {
};
// hook on end request to emit the log entry of the HTTP request.
res.on("finish", () => {
let finished = false;
const handler = () => {
if (finished) {
return;
}
finished = true;
res.responseTime = new Date() - start;
// status code response level handling
if (res.statusCode && options.level === "auto") {
@ -293,7 +298,11 @@ module.exports = function getLogger(logger4js, options) {
thisLogger.log(level, format(fmt, combinedTokens));
}
if (options.context) thisLogger.removeContext("res");
});
};
res.on("end", handler);
res.on("finish", handler);
res.on("error", handler);
res.on("close", handler);
}
// ensure next gets always called