fix: is_same_host returns False for bare hostnames (no scheme)

src/databricks/sql/auth/auth_utils.py

@@ -36,12 +36,18 @@ def is_same_host(url1: str, url2: str) -> bool:
         True if hosts are the same, False otherwise
     """
     try:
-        host1 = urlparse(url1).netloc
-        host2 = urlparse(url2).netloc
-        # Handle port differences (e.g., example.com vs example.com:443)
-        host1_without_port = host1.split(":")[0]
-        host2_without_port = host2.split(":")[0]
-        return host1_without_port == host2_without_port
+        def _extract_host(url: str) -> str:
+            parsed = urlparse(url)
+            netloc = parsed.netloc
+            if not netloc:
+                # Bare hostname with no scheme: urlparse puts the host in the
+                # path component and leaves netloc empty.  Add a dummy scheme so
+                # the parser can identify the netloc correctly.
+                netloc = urlparse(f"https://{url}").netloc
+            # Strip port (e.g. example.com:443 -> example.com)
+            return netloc.split(":")[0].lower()
+
+        return _extract_host(url1) == _extract_host(url2)
     except Exception as e:
         logger.debug("Failed to parse URLs: %s", e)
         return False

tests/unit/test_token_federation.py

@@ -322,6 +322,12 @@ def test_normalize_hostname(self, input_hostname, expected):
             ("https://test.databricks.com", "https://test.databricks.com:443", True),
             ("https://test1.databricks.com", "https://test2.databricks.com", False),
             ("https://login.microsoftonline.com", "https://test.databricks.com", False),
+            # Bare hostname (no scheme) — regression: urlparse("host").netloc == ""
+            # caused these to incorrectly return False
+            ("test.databricks.com", "https://test.databricks.com", True),
+            ("https://test.databricks.com", "test.databricks.com", True),
+            ("test.databricks.com", "test.databricks.com", True),
+            ("other.example.com", "test.databricks.com", False),
         ],
     )
     def test_is_same_host(self, url1, url2, expected):