fix: remove duplicate server name in cache path

identified in https://github.com/openml/openml-python/actions/runs/23148612114/job/67243650521?pr=1609

Author: geetu040

openml/_api/clients/http.py

@@ -50,8 +50,8 @@ def get_key(self, url: str, params: dict[str, Any]) -> str:
         """
         Generate a filesystem-safe cache key for a request.
 
-        The key is constructed from the reversed domain components, URL path
-        segments, and URL-encoded query parameters (excluding ``api_key``).
+        The key is constructed from URL path segments and
+        URL-encoded query parameters (excluding ``api_key``).
 
         Parameters
         ----------
@@ -66,13 +66,12 @@ def get_key(self, url: str, params: dict[str, Any]) -> str:
             A relative path string representing the cache key.
         """
         parsed_url = urlparse(url)
-        netloc_parts = parsed_url.netloc.split(".")[::-1]
         path_parts = parsed_url.path.strip("/").split("/")
 
         filtered_params = {k: v for k, v in params.items() if k != "api_key"}
         params_part = [urlencode(filtered_params)] if filtered_params else []
 
-        return str(Path(*netloc_parts, *path_parts, *params_part))
+        return str(Path(*path_parts, *params_part))
 
     def _key_to_path(self, key: str) -> Path:
         """

tests/test_api/test_http.py

@@ -42,22 +42,31 @@ def sample_download_url_v1(test_server_v1) -> str:
 def test_cache(cache, sample_url_v1):
     params = {"param1": "value1", "param2": "value2"}
 
+    # validate key
+
     parsed_url = urlparse(sample_url_v1)
-    netloc_parts = parsed_url.netloc.split(".")[::-1]
     path_parts = parsed_url.path.strip("/").split("/")
     params_key = "&".join([f"{k}={v}" for k, v in params.items()])
 
-
     key = cache.get_key(sample_url_v1, params)
 
     expected_key = os.path.join(
-        *netloc_parts,
         *path_parts,
         params_key,
     )
 
     assert key == expected_key
 
+    # validate path
+
+    path = cache._key_to_path(key)
+    expected_path = Path(openml.config.get_cache_directory()).joinpath(key)
+
+    assert path == expected_path
+    assert ":" not in str(path)
+
+    # validate save/load
+
     # mock response
     req = Request("GET", sample_url_v1).prepare()
     response = Response()