fal 1.5.5__tar.gz → 1.5.7__tar.gz

{fal-1.5.5/fal.egg-info → fal-1.5.7}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: fal
-Version: 1.5.5
+Version: 1.5.7
 Summary: fal is an easy-to-use Serverless Python Framework
 Author: Features & Labels <support@fal.ai>
 Requires-Python: >=3.8

{fal-1.5.5 → fal-1.5.7/fal.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: fal
-Version: 1.5.5
+Version: 1.5.7
 Summary: fal is an easy-to-use Serverless Python Framework
 Author: Features & Labels <support@fal.ai>
 Requires-Python: >=3.8

{fal-1.5.5 → fal-1.5.7}/src/fal/_fal_version.py

@@ -12,5 +12,5 @@ __version__: str
 __version_tuple__: VERSION_TUPLE
 version_tuple: VERSION_TUPLE
 
-__version__ = version = '1.5.5'
-__version_tuple__ = version_tuple = (1, 5, 5)
+__version__ = version = '1.5.7'
+__version_tuple__ = version_tuple = (1, 5, 7)

{fal-1.5.5 → fal-1.5.7}/src/fal/toolkit/file/file.py

@@ -54,7 +54,7 @@ def get_builtin_repository(id: RepositoryId) -> FileRepository:
 
 get_builtin_repository.__module__ = "__main__"
 
-DEFAULT_REPOSITORY: FileRepository | RepositoryId = "fal_v2"
+DEFAULT_REPOSITORY: FileRepository | RepositoryId = "fal_v3"
 FALLBACK_REPOSITORY: FileRepository | RepositoryId = "cdn"
 OBJECT_LIFECYCLE_PREFERENCE_KEY = "x-fal-object-lifecycle-preference"
 

{fal-1.5.5 → fal-1.5.7}/src/fal/toolkit/utils/download_utils.py

@@ -84,6 +84,9 @@ def _get_remote_file_properties(
             url_path = parsed_url.path
             file_name = Path(url_path).name or _hash_url(url)
 
+    # file name can still contain a forward slash if the server returns a relative path
+    file_name = Path(file_name).name
+
     return file_name, content_length
 
 
@@ -159,6 +162,7 @@ def download_file(
     try:
         file_name = _get_remote_file_properties(url, request_headers)[0]
     except Exception as e:
+        print(f"GOt error: {e}")
         raise DownloadError(f"Failed to get remote file properties for {url}") from e
 
     if "/" in file_name:

{fal-1.5.5 → fal-1.5.7}/tests/integration_test.py

@@ -1,6 +1,7 @@
 from __future__ import annotations
 
 import tempfile
+import unittest.mock
 from pathlib import Path
 from typing import Callable
 from uuid import uuid4
@@ -562,3 +563,21 @@ def test_fal_cdn(isolated_client):
     uploaded_image = upload_to_fal_cdn()
 
     assert uploaded_image
+
+
+def test_download_file_with_slash_in_filename():
+    from fal.toolkit.utils.download_utils import _get_remote_file_properties
+
+    test_url = "https://example.com/file/with/slash.txt"
+
+    with unittest.mock.patch(
+        "fal.toolkit.utils.download_utils.urlopen"
+    ) as mock_urlopen:
+        # urlopen is a context manager, so we need to mock the __enter__ method
+        mock_response = mock_urlopen.return_value.__enter__.return_value
+        mock_response.headers.get_filename.return_value = "file/with/slash.txt"
+        mock_response.headers.get.return_value = "100"
+
+        file_name, _ = _get_remote_file_properties(test_url)
+
+    assert "/" not in file_name