From f8921630a29f74f4b695e1f10211d5d378be10f8 Mon Sep 17 00:00:00 2001
From: atlowChemi <chemi@testim.io>
Date: Mon, 10 Jul 2023 00:22:05 +0300
Subject: [PATCH] net: server add `asyncDispose`

PR-URL: https://github.com/nodejs/node/pull/48717
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: Moshe Atlow <moshe@atlow.co.il>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
---
 doc/api/net.md                                | 11 +++++++
 lib/net.js                                    | 11 ++++++-
 .../test-net-server-async-dispose.mjs         | 30 +++++++++++++++++++
 3 files changed, 51 insertions(+), 1 deletion(-)
 create mode 100644 test/parallel/test-net-server-async-dispose.mjs

diff --git a/doc/api/net.md b/doc/api/net.md
index eca3c1ffcfec83..92752005767909 100644
--- a/doc/api/net.md
+++ b/doc/api/net.md
@@ -355,6 +355,17 @@ The optional `callback` will be called once the `'close'` event occurs. Unlike
 that event, it will be called with an `Error` as its only argument if the server
 was not open when it was closed.
 
+### `server[Symbol.asyncDispose]()`
+
+<!-- YAML
+added: REPLACEME
+-->
+
+> Stability: 1 - Experimental
+
+Calls [`server.close()`][] and returns a promise that fulfills when the
+server has closed.
+
 ### `server.getConnections(callback)`
 
 <!-- YAML
diff --git a/lib/net.js b/lib/net.js
index f04f62cce15bef..f351fd00feb0e8 100644
--- a/lib/net.js
+++ b/lib/net.js
@@ -27,6 +27,7 @@ const {
   ArrayPrototypePush,
   Boolean,
   FunctionPrototypeBind,
+  FunctionPrototypeCall,
   MathMax,
   Number,
   NumberIsNaN,
@@ -35,6 +36,7 @@ const {
   ObjectSetPrototypeOf,
   Symbol,
   ObjectCreate,
+  SymbolAsyncDispose,
 } = primordials;
 
 const EventEmitter = require('events');
@@ -110,7 +112,7 @@ const {
 } = require('internal/errors');
 const { isUint8Array } = require('internal/util/types');
 const { queueMicrotask } = require('internal/process/task_queues');
-const { kEmptyObject } = require('internal/util');
+const { kEmptyObject, promisify } = require('internal/util');
 const {
   validateAbortSignal,
   validateBoolean,
@@ -2191,6 +2193,13 @@ Server.prototype.close = function(cb) {
   return this;
 };
 
+Server.prototype[SymbolAsyncDispose] = async function() {
+  if (!this._handle) {
+    return;
+  }
+  return FunctionPrototypeCall(promisify(this.close), this);
+};
+
 Server.prototype._emitCloseIfDrained = function() {
   debug('SERVER _emitCloseIfDrained');
 
diff --git a/test/parallel/test-net-server-async-dispose.mjs b/test/parallel/test-net-server-async-dispose.mjs
new file mode 100644
index 00000000000000..08765e9c719ca9
--- /dev/null
+++ b/test/parallel/test-net-server-async-dispose.mjs
@@ -0,0 +1,30 @@
+import * as common from '../common/index.mjs';
+import assert from 'node:assert';
+import net from 'node:net';
+import { describe, it } from 'node:test';
+
+describe('net.Server[Symbol.asyncDispose]()', () => {
+  it('should close the server', async () => {
+    const server = net.createServer();
+    const timeoutRef = setTimeout(common.mustNotCall(), 2 ** 31 - 1);
+
+    server.listen(0, common.mustCall(async () => {
+      await server[Symbol.asyncDispose]().then(common.mustCall());
+      assert.strictEqual(server.address(), null);
+      clearTimeout(timeoutRef);
+    }));
+
+    server.on('close', common.mustCall());
+  });
+
+  it('should resolve even if the server is already closed', async () => {
+    const server = net.createServer();
+    const timeoutRef = setTimeout(common.mustNotCall(), 2 ** 31 - 1);
+
+    server.listen(0, common.mustCall(async () => {
+      await server[Symbol.asyncDispose]().then(common.mustCall());
+      await server[Symbol.asyncDispose]().then(common.mustCall(), common.mustNotCall());
+      clearTimeout(timeoutRef);
+    }));
+  });
+});