fix: file name in signed URLs is not URI-encoded (#324)

Author: juancarlospaco

storage3/_async/file_api.py

@@ -180,8 +180,13 @@ async def create_signed_url(
             json=json,
         )
         data = response.json()
+
+        # Prepare URL
+        url = urllib.parse.urlparse(data["signedURL"])
+        url = urllib.parse.quote(url.path) + f"?{url.query}"
+
         data["signedURL"] = (
-            f"{self._client.base_url}{cast(str, data['signedURL']).lstrip('/')}{download_query}"
+            f"{self._client.base_url}{cast(str, url).lstrip('/')}{download_query}"
         )
         return data
 
@@ -216,8 +221,13 @@ async def create_signed_urls(
         )
         data = response.json()
         for item in data:
+
+            # Prepare URL
+            url = urllib.parse.urlparse(item["signedURL"])
+            url = urllib.parse.quote(url.path) + f"?{url.query}"
+
             item["signedURL"] = (
-                f"{self._client.base_url}{cast(str, item['signedURL']).lstrip('/')}{download_query}"
+                f"{self._client.base_url}{cast(str, url).lstrip('/')}{download_query}"
             )
         return data
 

storage3/_sync/file_api.py

@@ -180,8 +180,13 @@ def create_signed_url(
             json=json,
         )
         data = response.json()
+
+        # Prepare URL
+        url = urllib.parse.urlparse(data["signedURL"])
+        url = urllib.parse.quote(url.path) + f"?{url.query}"
+
         data["signedURL"] = (
-            f"{self._client.base_url}{cast(str, data['signedURL']).lstrip('/')}{download_query}"
+            f"{self._client.base_url}{cast(str, url).lstrip('/')}{download_query}"
         )
         return data
 
@@ -216,8 +221,13 @@ def create_signed_urls(
         )
         data = response.json()
         for item in data:
+
+            # Prepare URL
+            url = urllib.parse.urlparse(item["signedURL"])
+            url = urllib.parse.quote(url.path) + f"?{url.query}"
+
             item["signedURL"] = (
-                f"{self._client.base_url}{cast(str, item['signedURL']).lstrip('/')}{download_query}"
+                f"{self._client.base_url}{cast(str, url).lstrip('/')}{download_query}"
             )
         return data