From 5584c92eff54487d5cb310ffb9fd2e8b968b886d Mon Sep 17 00:00:00 2001
From: Ishaan Jaffer <ishaanjaffer0324@gmail.com>
Date: Tue, 10 Mar 2026 14:40:41 -0700
Subject: [PATCH] fix: align fetchTools early-return guard with canFetchTools
 auth_type check for non-OpenAPI transports

---
 ui/litellm-dashboard/src/hooks/useTestMCPConnection.tsx | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/ui/litellm-dashboard/src/hooks/useTestMCPConnection.tsx b/ui/litellm-dashboard/src/hooks/useTestMCPConnection.tsx
index eff4ff5cd60..ea1b585b177 100644
--- a/ui/litellm-dashboard/src/hooks/useTestMCPConnection.tsx
+++ b/ui/litellm-dashboard/src/hooks/useTestMCPConnection.tsx
@@ -79,6 +79,11 @@ export const useTestMCPConnection = ({
       return;
     }
 
+    // For non-OpenAPI transports, auth_type must be selected (mirrors the canFetchTools check).
+    if (!isOpenAPITransport && !formValues.auth_type) {
+      return;
+    }
+
     // For OpenAPI transport, tools are derived from the spec — no OAuth token needed.
     if (requiresOAuthToken && !oauthAccessToken && !isOpenAPITransport) {
       return;