Add exception for swift in fsspec path handling

http(s) and swift filesystems need the scheme in the path.
This commit adds a special case for the swift filesystem,
when checking for the scheme presence.

Also remove one if for the http(s) schemes. Here all schemes can
call fs._strip_protocol(path). For the filesystems that
need the scheme in the path, this function will not remove it.

Author: fjansson

src/zarr/storage/_fsspec.py

@@ -139,8 +139,8 @@ def __init__(
                 f"fs ({fs}) was not created with `asynchronous=True`, this may lead to surprising behavior",
                 stacklevel=2,
             )
-        if "://" in path and not path.startswith("http"):
-            # `not path.startswith("http")` is a special case for the http filesystem (¯\_(ツ)_/¯)
+        if "://" in path and not path.startswith("http") and not path.startswith("swift"):
+            # special cases for the http(s) and swift filesystems (¯\_(ツ)_/¯)
             scheme, _ = path.split("://", maxsplit=1)
             raise ValueError(f"path argument to FsspecStore must not include scheme ({scheme}://)")
 
@@ -249,9 +249,8 @@ def from_url(
 
         # fsspec is not consistent about removing the scheme from the path, so check and strip it here
         # https://github.com/fsspec/filesystem_spec/issues/1722
-        if "://" in path and not path.startswith("http"):
-            # `not path.startswith("http")` is a special case for the http filesystem (¯\_(ツ)_/¯)
-            path = fs._strip_protocol(path)
+        # for http(s) and swift, _strip_protocol() does nothing, since they need the protocol in the path
+        path = fs._strip_protocol(path)
 
         return cls(fs=fs, path=path, read_only=read_only, allowed_exceptions=allowed_exceptions)