fix: accept case-insensitive HTTPS schemes in Thrift connections

src/databricks/sql/backend/thrift_backend.py

@@ -9,6 +9,7 @@
 from uuid import UUID
 
 from databricks.sql.common.unified_http_client import UnifiedHttpClient
+from databricks.sql.common.url_utils import normalize_host_with_protocol
 from databricks.sql.result_set import ThriftResultSet
 from databricks.sql.telemetry.models.event import StatementType
 
@@ -156,10 +157,10 @@ def __init__(
             uri = kwargs.get("_connection_uri")
         elif server_hostname and http_path:
             uri = "{host}:{port}/{path}".format(
-                host=server_hostname.rstrip("/"), port=port, path=http_path.lstrip("/")
+                host=normalize_host_with_protocol(server_hostname),
+                port=port,
+                path=http_path.lstrip("/"),
             )
-            if not uri.startswith("https://"):
-                uri = "https://" + uri
         else:
             raise ValueError("No valid connection settings.")
 

tests/unit/test_thrift_backend.py

@@ -70,6 +70,28 @@ class ThriftBackendTestSuite(unittest.TestCase):
         ttypes.TGetColumnsResp,
     ]
 
+    def test_server_hostname_https_scheme_is_case_insensitive(self):
+        for scheme in ("", "https://", "HTTPS://", "HtTpS://"):
+            with self.subTest(scheme=scheme):
+                backend = ThriftDatabricksClient(
+                    scheme + "httpbin.invalid/",
+                    8443,
+                    "/sql/1.0/warehouses/test",
+                    [],
+                    auth_provider=AuthProvider(),
+                    ssl_options=SSLOptions(),
+                    http_client=MagicMock(),
+                )
+                try:
+                    self.assertEqual(backend._transport.scheme, "https")
+                    self.assertEqual(backend._transport.host, "httpbin.invalid")
+                    self.assertEqual(backend._transport.port, 8443)
+                    self.assertEqual(
+                        backend._transport.path, "/sql/1.0/warehouses/test"
+                    )
+                finally:
+                    backend._transport.close()
+
     def test_make_request_checks_thrift_status_code(self):
         mock_response = Mock()
         mock_response.status.statusCode = ttypes.TStatusCode.ERROR_STATUS