From b84f745b276009cca1f120dbd70e9de59b842965 Mon Sep 17 00:00:00 2001
From: atlowChemi <chemi@testim.io>
Date: Sun, 25 Jun 2023 20:19:40 +0300
Subject: [PATCH] http: server add async dispose

PR-URL: https://github.com/nodejs/node/pull/48548
Reviewed-By: Moshe Atlow <moshe@atlow.co.il>
Reviewed-By: Paolo Insogna <paolo@cowtech.it>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
---
 doc/api/http.md                                 | 12 ++++++++++++
 lib/_http_server.js                             |  9 +++++++++
 test/parallel/test-http-server-async-dispose.js | 14 ++++++++++++++
 3 files changed, 35 insertions(+)
 create mode 100644 test/parallel/test-http-server-async-dispose.js

diff --git a/doc/api/http.md b/doc/api/http.md
index 10909e8ebaa913..82c2e4909d0a10 100644
--- a/doc/api/http.md
+++ b/doc/api/http.md
@@ -1652,6 +1652,17 @@ to 8.0.0, which did not have a keep-alive timeout.
 The socket timeout logic is set up on connection, so changing this value only
 affects new connections to the server, not any existing connections.
 
+### `server[Symbol.asyncDispose]()`
+
+<!-- YAML
+added: REPLACEME
+-->
+
+> Stability: 1 - Experimental
+
+Calls [`server.close()`][] and returns a promise that fulfills when the
+server has closed.
+
 ## Class: `http.ServerResponse`
 
 <!-- YAML
@@ -3842,6 +3853,7 @@ Set the maximum number of idle HTTP parsers.
 [`response.write(data, encoding)`]: #responsewritechunk-encoding-callback
 [`response.writeContinue()`]: #responsewritecontinue
 [`response.writeHead()`]: #responsewriteheadstatuscode-statusmessage-headers
+[`server.close()`]: #serverclosecallback
 [`server.headersTimeout`]: #serverheaderstimeout
 [`server.listen()`]: net.md#serverlisten
 [`server.requestTimeout`]: #serverrequesttimeout
diff --git a/lib/_http_server.js b/lib/_http_server.js
index c010d8fa96a1c2..1d23ea0e86ab22 100644
--- a/lib/_http_server.js
+++ b/lib/_http_server.js
@@ -24,12 +24,14 @@
 const {
   ArrayIsArray,
   Error,
+  FunctionPrototypeCall,
   MathMin,
   ObjectKeys,
   ObjectSetPrototypeOf,
   RegExpPrototypeExec,
   ReflectApply,
   Symbol,
+  SymbolAsyncDispose,
   SymbolFor,
 } = primordials;
 
@@ -80,6 +82,9 @@ const {
   ERR_INVALID_ARG_VALUE,
   ERR_INVALID_CHAR,
 } = codes;
+const {
+  promisify,
+} = require('internal/util');
 const {
   validateInteger,
   validateBoolean,
@@ -548,6 +553,10 @@ Server.prototype.close = function() {
   ReflectApply(net.Server.prototype.close, this, arguments);
 };
 
+Server.prototype[SymbolAsyncDispose] = async function() {
+  return FunctionPrototypeCall(promisify(this.close), this);
+};
+
 Server.prototype.closeAllConnections = function() {
   const connections = this[kConnections].all();
 
diff --git a/test/parallel/test-http-server-async-dispose.js b/test/parallel/test-http-server-async-dispose.js
new file mode 100644
index 00000000000000..8af11dcb7b0c3c
--- /dev/null
+++ b/test/parallel/test-http-server-async-dispose.js
@@ -0,0 +1,14 @@
+'use strict';
+const common = require('../common');
+const assert = require('assert');
+const { createServer } = require('http');
+const { kConnectionsCheckingInterval } = require('_http_server');
+
+const server = createServer();
+
+server.listen(0, common.mustCall(() => {
+  server.on('close', common.mustCall());
+  server[Symbol.asyncDispose]().then(common.mustCall(() => {
+    assert(server[kConnectionsCheckingInterval]._destroyed);
+  }));
+}));