From a527b3a5de86d8753f7881faff694fbe3797d93d Mon Sep 17 00:00:00 2001
From: Asher Gomez <ashersaupingomez@gmail.com>
Date: Sat, 31 Aug 2024 09:46:02 +1000
Subject: [PATCH] chore(net): remove `Deno.shutdown()` (#25253)

---
 ext/net/01_net.js                       |  7 +------
 ext/net/lib.deno_net.d.ts               | 18 ------------------
 runtime/js/90_deno_ns.js                |  8 --------
 runtime/js/99_main.js                   |  2 --
 tests/specs/future/runtime_api/main.js  |  1 -
 tests/specs/future/runtime_api/main.out |  1 -
 6 files changed, 1 insertion(+), 36 deletions(-)

diff --git a/ext/net/01_net.js b/ext/net/01_net.js
index a41a27543a..87bbcd476f 100644
--- a/ext/net/01_net.js
+++ b/ext/net/01_net.js
@@ -63,10 +63,6 @@ async function write(rid, data) {
   return await core.write(rid, data);
 }
 
-function shutdown(rid) {
-  return core.shutdown(rid);
-}
-
 async function resolveDns(query, recordType, options) {
   let cancelRid;
   let abortHandler;
@@ -164,7 +160,7 @@ class Conn {
   }
 
   closeWrite() {
-    return shutdown(this.#rid);
+    return core.shutdown(this.#rid);
   }
 
   get readable() {
@@ -647,7 +643,6 @@ export {
   Listener,
   listenOptionApiName,
   resolveDns,
-  shutdown,
   TcpConn,
   UnixConn,
   validatePort,
diff --git a/ext/net/lib.deno_net.d.ts b/ext/net/lib.deno_net.d.ts
index 13fbd35148..382a3ab8d9 100644
--- a/ext/net/lib.deno_net.d.ts
+++ b/ext/net/lib.deno_net.d.ts
@@ -528,22 +528,4 @@ declare namespace Deno {
     conn: TcpConn,
     options?: StartTlsOptions,
   ): Promise<TlsConn>;
-
-  /** Shutdown socket send operations.
-   *
-   * Matches behavior of POSIX shutdown(3).
-   *
-   * ```ts
-   * const listener = Deno.listen({ port: 80 });
-   * const conn = await listener.accept();
-   * Deno.shutdown(conn.rid);
-   * ```
-   *
-   * @deprecated This will be removed in Deno 2.0. See the
-   * {@link https://docs.deno.com/runtime/manual/advanced/migrate_deprecations | Deno 1.x to 2.x Migration Guide}
-   * for migration instructions.
-   *
-   * @category Network
-   */
-  export function shutdown(rid: number): Promise<void>;
 }
diff --git a/runtime/js/90_deno_ns.js b/runtime/js/90_deno_ns.js
index 7bfae0b7c5..1d44f30016 100644
--- a/runtime/js/90_deno_ns.js
+++ b/runtime/js/90_deno_ns.js
@@ -186,14 +186,6 @@ const denoNs = {
   connectTls: tls.connectTls,
   listenTls: tls.listenTls,
   startTls: tls.startTls,
-  shutdown(rid) {
-    internals.warnOnDeprecatedApi(
-      "Deno.shutdown()",
-      new Error().stack,
-      "Use `Deno.Conn.closeWrite()` instead.",
-    );
-    net.shutdown(rid);
-  },
   fstatSync(rid) {
     internals.warnOnDeprecatedApi(
       "Deno.fstatSync()",
diff --git a/runtime/js/99_main.js b/runtime/js/99_main.js
index 36c43ff14d..578bde6684 100644
--- a/runtime/js/99_main.js
+++ b/runtime/js/99_main.js
@@ -948,7 +948,6 @@ function bootstrapMainRuntime(runtimeOptions, warmup = false) {
       delete Deno.readSync;
       delete Deno.seek;
       delete Deno.seekSync;
-      delete Deno.shutdown;
       delete Deno.writeAll;
       delete Deno.writeAllSync;
       delete Deno.write;
@@ -1136,7 +1135,6 @@ function bootstrapWorkerRuntime(
       delete Deno.readSync;
       delete Deno.seek;
       delete Deno.seekSync;
-      delete Deno.shutdown;
       delete Deno.writeAll;
       delete Deno.writeAllSync;
       delete Deno.write;
diff --git a/tests/specs/future/runtime_api/main.js b/tests/specs/future/runtime_api/main.js
index 85c160094f..6bc83d30c5 100644
--- a/tests/specs/future/runtime_api/main.js
+++ b/tests/specs/future/runtime_api/main.js
@@ -23,7 +23,6 @@ console.log("Deno.read is", Deno.read);
 console.log("Deno.readSync is", Deno.readSync);
 console.log("Deno.seek is", Deno.seek);
 console.log("Deno.seekSync is", Deno.seekSync);
-console.log("Deno.shutdown is", Deno.shutdown);
 console.log("Deno.writeAll is", Deno.writeAll);
 console.log("Deno.writeAllSync is", Deno.writeAllSync);
 console.log("Deno.write is", Deno.write);
diff --git a/tests/specs/future/runtime_api/main.out b/tests/specs/future/runtime_api/main.out
index 18a39fc7a7..2a8167ef9a 100644
--- a/tests/specs/future/runtime_api/main.out
+++ b/tests/specs/future/runtime_api/main.out
@@ -20,7 +20,6 @@ Deno.read is undefined
 Deno.readSync is undefined
 Deno.seek is undefined
 Deno.seekSync is undefined
-Deno.shutdown is undefined
 Deno.writeAll is undefined
 Deno.writeAllSync is undefined
 Deno.write is undefined