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 #1401 from MamfTheKramf/http-response-chaining

Browse Source 
fix: enable method chaining for after writeHead
This commit is contained in:
Lam Wei Li 2024-04-05 13:55:29 +08:00 committed by GitHub
commit bc59c87050
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
4 changed files with 5 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
lib/connect-logger.js
View File

@ -273,6 +273,7 @@ module.exports = function getLogger(logger4js, options) {
res.__statusCode = code;
res.__headers = headers || {};
return res;
};
// hook on end request to emit the log entry of the HTTP request.

1
test/tap/connect-context-test.js
View File

@ -58,6 +58,7 @@ class MockResponse extends EE {
writeHead(code /* , headers */) {
this.statusCode = code;
return this;
}
}

4
test/tap/connect-logger-test.js
View File

@ -58,6 +58,7 @@ class MockResponse extends EE {
writeHead(code /* , headers */) {
this.statusCode = code;
return this;
}
}
@ -87,8 +88,7 @@ function request(
next = () => {};
}
cl(req, res, next);
res.writeHead(code, resHeaders);
res.end('chunk', 'encoding');
res.writeHead(code, resHeaders).end('chunk', 'encoding');
}
test('log4js connect logger', (batch) => {

1
test/tap/connect-nolog-test.js
View File

@ -49,6 +49,7 @@ class MockResponse extends EE {
writeHead(code /* , headers */) {
this.statusCode = code;
return this;
}
}