Don't remove trailing / with split_path. Keys with a trailing / are now properly removed.

s3fs/core.py

@@ -470,13 +470,15 @@ def split_path(self, path) -> tuple[str, str, str | None]:
         >>> split_path("s3://mybucket/path/to/versioned_file?versionId=some_version_id")
         ['mybucket', 'path/to/versioned_file', 'some_version_id']
         """
+        trail = path[len(path.rstrip("/")) :]
         path = self._strip_protocol(path)
         path = path.lstrip("/")
         if "/" not in path:
             return path, "", None
         else:
             bucket, keypart = self._find_bucket_key(path)
             key, _, version_id = keypart.partition("?versionId=")
+            key += trail  # restore trailing slashes removed by AbstractFileSystem._strip_protocol
             return (
                 bucket,
                 key,

s3fs/tests/test_s3fs.py

@@ -3085,3 +3085,14 @@ async def run_program(run):
     aiobotocore.httpsession.AIOHTTPSession
     asyncio.run(run_program(True))
     asyncio.run(run_program(False))
+
+
+def test_rm_recursive_prfix(s3):
+    prefix = "logs/"  # must end with "/"
+
+    # Create empty "directory" in S3
+    client = get_boto3_client()
+    client.put_object(Bucket=test_bucket_name, Key=prefix, Body=b"")
+    logs_path = f"s3://{test_bucket_name}/{prefix}"
+    s3.rm(logs_path, recursive=True)
+    assert not s3.isdir(logs_path)