fal 1.50.0__tar.gz → 1.50.2__tar.gz

{fal-1.50.0/fal.egg-info → fal-1.50.2}/PKG-INFO

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

{fal-1.50.0 → fal-1.50.2/fal.egg-info}/PKG-INFO

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

{fal-1.50.0 → fal-1.50.2}/src/fal/_fal_version.py

@@ -28,7 +28,7 @@ version_tuple: VERSION_TUPLE
 commit_id: COMMIT_ID
 __commit_id__: COMMIT_ID
 
-__version__ = version = '1.50.0'
-__version_tuple__ = version_tuple = (1, 50, 0)
+__version__ = version = '1.50.2'
+__version_tuple__ = version_tuple = (1, 50, 2)
 
 __commit_id__ = commit_id = None

{fal-1.50.0 → fal-1.50.2}/src/fal/files.py

@@ -224,6 +224,7 @@ class FalFileSystem(AbstractFileSystem):
             "POST",
             f"/files/file/url/{abs_rpath}",
             json={"url": url},
+            timeout=10 * 60,  # 10 minutes in seconds
         )
         self.dircache.clear()
 

{fal-1.50.0 → fal-1.50.2}/src/fal/toolkit/file/providers/fal.py

@@ -56,44 +56,21 @@ def _should_retry(exc: Exception) -> bool:
     return False
 
 
-class _RetryingRequestContext:
-    def __init__(self, request: Request, kwargs: dict[str, Any]):
-        self.request = request
-        self.kwargs = kwargs
-        self._cm: Any | None = None
-        self._response: addinfourl | None = None
-
-    def __enter__(self) -> addinfourl:
-        def _enter_once() -> addinfourl:
-            # Obtain the original context manager and explicitly enter it
-            self._cm = _urlopen(self.request, **self.kwargs)
-            return self._cm.__enter__()
-
-        _enter_with_retry = retry(
-            max_retries=MAX_ATTEMPTS,
-            base_delay=BASE_DELAY,
-            max_delay=MAX_DELAY,
-            backoff_type="exponential",
-            jitter=True,
-            should_retry=_should_retry,
-        )(_enter_once)
-
-        self._response = _enter_with_retry()
-        assert self._response is not None
-        return self._response
-
-    def __exit__(self, exc_type, exc_val, exc_tb) -> bool:
-        if self._cm is not None:
-            return self._cm.__exit__(exc_type, exc_val, exc_tb)
-        return False
-
-
 @contextmanager
 def _maybe_retry_request(
     request: Request,
     **kwargs: Any,
 ) -> Generator[addinfourl, None, None]:
-    with _RetryingRequestContext(request, kwargs) as response:
+    _urlopen_with_retry = retry(
+        max_retries=MAX_ATTEMPTS,
+        base_delay=BASE_DELAY,
+        max_delay=MAX_DELAY,
+        backoff_type="exponential",
+        jitter=True,
+        should_retry=_should_retry,
+    )(_urlopen)
+
+    with _urlopen_with_retry(request, **kwargs) as response:
         yield response
 
 

{fal-1.50.0 → fal-1.50.2}/tests/toolkit/file/providers/test_fal_retry.py

@@ -1,7 +1,7 @@
 from __future__ import annotations
 
 from unittest import mock
-from urllib.error import HTTPError, URLError
+from urllib.error import HTTPError
 from urllib.request import Request
 
 import pytest
@@ -163,42 +163,3 @@ def test_max_retries_exhausted_for_retryable_errors():
 
         assert exc_info.value.code == 500
         assert call_count == 5
-
-
-def test_retry_on_urlerror_during_context_enter():
-    request = Request("https://example.com/test")
-
-    calls = {"open": 0, "enter": 0}
-
-    class EnterRaisesOnce:
-        def __init__(self):
-            self.data = b'{"result": "success"}'
-            self.headers = {"Content-Type": "application/json"}
-
-        def read(self):
-            return self.data
-
-        def __enter__(self):
-            calls["enter"] += 1
-            if calls["enter"] == 1:
-                raise URLError("Temporary failure in name resolution")
-            return self
-
-        def __exit__(self, exc_type, exc_val, exc_tb):
-            return False
-
-    def mock_urlopen_side_effect(*args, **kwargs):
-        calls["open"] += 1
-        return EnterRaisesOnce()
-
-    with mock.patch(
-        "fal.toolkit.file.providers.fal._urlopen", side_effect=mock_urlopen_side_effect
-    ):
-        with mock.patch("fal.toolkit.utils.retry.time.sleep"):
-            # Expect success after a retry (second __enter__ returns normally)
-            with _maybe_retry_request(request) as response:
-                assert response is not None
-
-    # Should have attempted to open at least twice due to retry
-    assert calls["open"] >= 2
-    assert calls["enter"] >= 2