http: fix close return value mismatch between doc and implementation

nodejs · Feb 19, 2024 · 56d4c2b · 56d4c2b
1 parent ec3040f
commit 56d4c2b
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 0 deletions.
diff --git a/lib/_http_server.js b/lib/_http_server.js
@@ -570,6 +570,7 @@ ObjectSetPrototypeOf(Server, net.Server);
 Server.prototype.close = function() {
   httpServerPreClose(this);
   ReflectApply(net.Server.prototype.close, this, arguments);
+  return this;
 };
 
 Server.prototype[SymbolAsyncDispose] = async function() {

diff --git a/lib/https.js b/lib/https.js
@@ -111,6 +111,7 @@ Server.prototype.setTimeout = HttpServer.prototype.setTimeout;
 Server.prototype.close = function() {
   httpServerPreClose(this);
   ReflectApply(tls.Server.prototype.close, this, arguments);
+  return this;
 };
 
 Server.prototype[SymbolAsyncDispose] = async function() {

diff --git a/test/parallel/test-http-server.js b/test/parallel/test-http-server.js
@@ -139,4 +139,5 @@ process.on('exit', function() {
   assert.match(server_response, quit);
 
   assert.strictEqual(client_got_eof, true);
+  assert.strictEqual(server.close(), server);
 });