From dd0279a6c918769fdfc1f8025d2aec29a7b847a7 Mon Sep 17 00:00:00 2001
From: Sonia Appasamy <sonia@tailscale.com>
Date: Tue, 12 Dec 2023 14:31:49 -0500
Subject: [PATCH] client/web: fix ts connection check

Updates #10261

Signed-off-by: Sonia Appasamy <sonia@tailscale.com>
---
 client/web/src/components/login-toggle.tsx | 6 ++----
 client/web/src/hooks/auth.ts               | 3 ++-
 2 files changed, 4 insertions(+), 5 deletions(-)

diff --git a/client/web/src/components/login-toggle.tsx b/client/web/src/components/login-toggle.tsx
index 08386d20e..850a2d8e2 100644
--- a/client/web/src/components/login-toggle.tsx
+++ b/client/web/src/components/login-toggle.tsx
@@ -106,10 +106,8 @@ function LoginPopoverContent({
     }
     setIsRunningCheck(true)
     fetch(`http://${node.IPv4}:5252/ok`, { mode: "no-cors" })
-      .then((r) => {
-        if (r.ok) {
-          setCanConnectOverTS(true)
-        }
+      .then(() => {
+        setCanConnectOverTS(true)
         setIsRunningCheck(false)
       })
       .catch(() => setIsRunningCheck(false))
diff --git a/client/web/src/hooks/auth.ts b/client/web/src/hooks/auth.ts
index a79171c2e..6cc0e6f62 100644
--- a/client/web/src/hooks/auth.ts
+++ b/client/web/src/hooks/auth.ts
@@ -83,7 +83,8 @@ export default function useAuth() {
 
   useEffect(() => {
     loadAuth() // Refresh auth state after syno auth runs
-  }, [loadAuth, ranSynoAuth])
+    // eslint-disable-next-line react-hooks/exhaustive-deps
+  }, [ranSynoAuth])
 
   return {
     data,