Keep bare HTTP origins on the service discovery path

Author: FrancescAlted

src/blosc2/b2view/ordinary_file.py

@@ -37,7 +37,10 @@ def ordinary_source(urlpath, remote_service="auto"):
         return False
     member = _source_member(str(base))
     parsed = urlsplit(member)
-    suffix = PurePosixPath(parsed.path or parsed.netloc).suffix.lower()
+    # HTTP authorities are hosts, not filenames (e.g. service.example.com).
+    # Archive/member protocols may instead carry the filename in the authority.
+    name = parsed.path if parsed.scheme in {"http", "https"} else parsed.path or parsed.netloc
+    suffix = PurePosixPath(name).suffix.lower()
     if suffix in NATIVE_SUFFIXES:
         return False
     if is_fsspec_url(urlpath):

tests/b2view/test_ordinary_files.py

@@ -256,6 +256,16 @@ def test_dataset_and_service_routing_unchanged(source):
     assert not ordinary_source(source)
 
 
+@pytest.mark.parametrize(
+    "source",
+    ["https://service.example.com", "https://service.example.com/", "http://service.example.com:8080"],
+)
+def test_bare_http_origins_keep_service_discovery(source):
+    assert not ordinary_source(source)
+    assert ordinary_source(source, remote_service="fsspec")
+    assert ordinary_source(source + "/README.md")
+
+
 def test_ipv6_http_and_explicit_fsspec_routing():
     assert ordinary_source("http://[::1]/README.md")
     assert ordinary_source("http://[::1]/no-extension", "fsspec")