solari-browser 0.2.0__tar.gz → 0.2.2__tar.gz

{solari_browser-0.2.0 → solari_browser-0.2.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: solari-browser
-Version: 0.2.0
+Version: 0.2.2
 Summary: Python SDK for Solari Browser — managed, stealthy remote Chromium over the Playwright wire protocol / raw CDP.
 Project-URL: Homepage, https://getsolari.com
 Author: Solari

{solari_browser-0.2.0 → solari_browser-0.2.2}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "solari-browser"
-version = "0.2.0"
+version = "0.2.2"
 description = "Python SDK for Solari Browser — managed, stealthy remote Chromium over the Playwright wire protocol / raw CDP."
 readme = "README.md"
 requires-python = ">=3.9"

{solari_browser-0.2.0 → solari_browser-0.2.2}/solari_browser/__init__.py

@@ -37,7 +37,7 @@ from .types import (
     Session,
 )
 
-__version__ = "0.1.2"
+__version__ = "0.2.2"
 
 __all__ = [
     "Solari",

{solari_browser-0.2.0 → solari_browser-0.2.2}/solari_browser/client.py

@@ -81,6 +81,21 @@ def _says_retryable(res: httpx.Response) -> bool:
         return False
     return isinstance(body, dict) and body.get("retryable") is True
 
+# A few 503s are answers, not outages: retrying cannot change them, and swallowing the body would hide the
+# code from the caller. `ProxyTierUnavailable` (explicitly requested proxy tier not servable) is one.
+_TERMINAL_503_CODES = frozenset({"ProxyTierUnavailable"})
+
+
+def _is_terminal_503(res: httpx.Response) -> bool:
+    if res.status_code != 503:
+        return False
+    try:
+        body = res.json()
+    except Exception:  # noqa: BLE001 - non-JSON body is an ordinary outage
+        return False
+    return isinstance(body, dict) and body.get("code") in _TERMINAL_503_CODES
+
+
 # Mirrors `isLikelyTransientConnect`: which connect failures are worth relaunching.
 _TRANSIENT_CONNECT = re.compile(
     r"ECONNRESET|ECONNREFUSED|ETIMEDOUT|EPIPE|socket hang up|WebSocket|connect|"
@@ -284,7 +299,10 @@ class _Sessions:
         )
 
     async def download_replay(self, session_id: str) -> bytes:
-        """Download the replay as NDJSON bytes (gzip-encoded per `content_encoding`)."""
+        """Download the replay as NDJSON bytes. May or may not actually be
+        gzip despite what was uploaded — GCS decompresses transparently on
+        an ordinary GET, so check `content_encoding` (now provider-accurate)
+        before attempting to decompress."""
         replay = await self.get_replay_url(session_id)
         async with httpx.AsyncClient(timeout=self._client._timeout_s) as http:
             res = await http.get(replay.url)
@@ -468,7 +486,7 @@ class Solari:
                 res = await self._http.request(
                     method, url, headers=headers, content=payload
                 )
-                retry_this = res.status_code in _RETRYABLE_STATUS or (
+                retry_this = (res.status_code in _RETRYABLE_STATUS and not _is_terminal_503(res)) or (
                     _is_safe_to_replay(method, idempotency_key)
                     and _says_retryable(res)
                 )

{solari_browser-0.2.0 → solari_browser-0.2.2}/solari_browser/errors.py

@@ -17,6 +17,9 @@ BROWSER_UNHEALTHY = "BrowserUnhealthy"
 # `_release_rejection` in client.py for why a 404 is not blanket-success on
 # DELETE /sessions/:id.
 INVALID_SESSION_ID = "InvalidSessionId"
+# PROXY_TIER_UNAVAILABLE — POST /sessions 503: a proxy tier you explicitly requested cannot be served right now.
+# No session was created. Not retried by the client (the answer cannot change within a retry).
+PROXY_TIER_UNAVAILABLE = "ProxyTierUnavailable"
 
 
 class SolariError(Exception):

{solari_browser-0.2.0 → solari_browser-0.2.2}/tests/test_retry_hint.py

@@ -83,3 +83,35 @@ def test_507_is_retryable_even_though_this_gateway_never_emits_one():
     # Negative control: permanent statuses must stay out.
     assert 500 not in _RETRYABLE_STATUS
     assert 501 not in _RETRYABLE_STATUS
+
+
+def test_proxy_tier_unavailable_503_is_not_retried_and_keeps_its_code(gw):  # noqa: F811
+    gw.route("POST", "/sessions", 503,
+             {"error": "Requested proxy tier 'residential' is unavailable", "code": "ProxyTierUnavailable", "tier": "residential"})
+    gw.route("POST", "/sessions", 201, {"sessionId": "s-1", "wsEndpoint": "ws://127.0.0.1:1/ws"})
+
+    c = client(gw, backoff_ms=1)
+
+    @sync
+    async def create():
+        return await c.sessions.create(proxy={"tier": "residential", "country": "us"})
+
+    with pytest.raises(SolariError) as info:
+        create()
+    assert info.value.code == "ProxyTierUnavailable"
+    assert info.value.status == 503
+    assert len(gw.seen) == 1, "a terminal 503 must not be retried"
+
+
+def test_ordinary_503_is_still_retried(gw):  # noqa: F811
+    gw.route("POST", "/sessions", 503, {"error": "No stealth pool available", "kind": "stealth"})
+    gw.route("POST", "/sessions", 201, {"sessionId": "s-1", "wsEndpoint": "ws://127.0.0.1:1/ws"})
+
+    c = client(gw, backoff_ms=1)
+
+    @sync
+    async def create():
+        return await c.sessions.create()
+
+    create()
+    assert len(gw.seen) == 2