From e43cc2214c909db28ee5e42ae9923bf10bc38f6e Mon Sep 17 00:00:00 2001
From: Sunil Pai <threepointone@gmail.com>
Date: Tue, 8 Mar 2022 00:54:35 +0530
Subject: [PATCH] feat: disable tunnel in `wrangler dev`

Disables sharing local development server on the internet. We will bring this back after it's more polished/ready.

Fixes https://github.com/cloudflare/wrangler2/issues/550
---
 .changeset/eleven-bananas-bow.md |  9 +++++++++
 packages/wrangler/src/dev.tsx    | 15 +++++----------
 2 files changed, 14 insertions(+), 10 deletions(-)
 create mode 100644 .changeset/eleven-bananas-bow.md

diff --git a/.changeset/eleven-bananas-bow.md b/.changeset/eleven-bananas-bow.md
new file mode 100644
index 000000000000..8b916d697006
--- /dev/null
+++ b/.changeset/eleven-bananas-bow.md
@@ -0,0 +1,9 @@
+---
+"wrangler": patch
+---
+
+feat: disable tunnel in `wrangler dev`
+
+Disables sharing local development server on the internet. We will bring this back after it's more polished/ready.
+
+Fixes https://github.com/cloudflare/wrangler2/issues/550
diff --git a/packages/wrangler/src/dev.tsx b/packages/wrangler/src/dev.tsx
index 4e012d86b57f..1867e7c88c89 100644
--- a/packages/wrangler/src/dev.tsx
+++ b/packages/wrangler/src/dev.tsx
@@ -153,9 +153,7 @@ function Dev(props: DevProps): JSX.Element {
       )}
       <Box borderStyle="round" paddingLeft={1} paddingRight={1}>
         <Text>
-          {`B to open a browser, D to open Devtools, S to ${
-            toggles.tunnel ? "turn off" : "turn on"
-          } (experimental) sharing, L to ${
+          {`B to open a browser, D to open Devtools, L to ${
             toggles.local ? "turn off" : "turn on"
           } local mode, X to exit`}
         </Text>
@@ -835,6 +833,10 @@ async function findTunnelHostname() {
   return hostName;
 }
 
+/**
+ * Create a tunnel to the remote worker.
+ * We've disabled this for now until we figure out a better user experience.
+ */
 function useTunnel(toggle: boolean) {
   const tunnel = useRef<ReturnType<typeof spawn>>();
   const removeSignalExitListener = useRef<() => void>();
@@ -924,13 +926,6 @@ function useHotkeys(
           );
           break;
         }
-        // toggle tunnel
-        case "s":
-          setToggles((previousToggles) => ({
-            ...previousToggles,
-            tunnel: !previousToggles.tunnel,
-          }));
-          break;
         // toggle local
         case "l":
           setToggles((previousToggles) => ({