arccos-api 0.2.0__tar.gz → 0.2.1__tar.gz

{arccos_api-0.2.0 → arccos_api-0.2.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: arccos-api
-Version: 0.2.0
+Version: 0.2.1
 Summary: Unofficial Python CLI and client library for the Arccos Golf API
 Author: Paul Frederiksen
 License: MIT

{arccos_api-0.2.0 → arccos_api-0.2.1}/arccos/__init__.py

@@ -9,6 +9,8 @@ Quick start:
     print(client.handicap.current())
 """
 
+import logging as _logging
+import re as _re
 from importlib.metadata import version as _pkg_version
 
 from .auth import ArccosAuth
@@ -21,6 +23,25 @@ from .exceptions import (
     ArccosRateLimitError,
 )
 
+
+class _SensitiveFilter(_logging.Filter):
+    """Redact tokens, access keys, and passwords from log messages."""
+
+    _PATTERNS = _re.compile(
+        r"(Bearer\s+)\S+|"
+        r"(eyJ[A-Za-z0-9_-]+\.eyJ[A-Za-z0-9_-]+\.)[A-Za-z0-9_-]+|"
+        r"(accessKey[\"']?\s*[:=]\s*[\"']?)\S+",
+        _re.IGNORECASE,
+    )
+
+    def filter(self, record: _logging.LogRecord) -> bool:
+        if isinstance(record.msg, str):
+            record.msg = self._PATTERNS.sub(r"\1\2\3[REDACTED]", record.msg)
+        return True
+
+
+_logging.getLogger("arccos").addFilter(_SensitiveFilter())
+
 __version__ = _pkg_version("arccos-api")
 __all__ = [
     "ArccosClient",

{arccos_api-0.2.0 → arccos_api-0.2.1}/arccos/_http.py

@@ -7,6 +7,7 @@ Retries transient failures (429, 5xx, timeouts) with exponential backoff.
 
 from __future__ import annotations
 
+import contextlib
 import logging
 import os
 import time
@@ -44,7 +45,13 @@ class HttpClient:
             "Content-Type": "application/json",
             "Accept":       "application/json",
         })
-        self._timeout = int(os.environ.get("ARCCOS_TIMEOUT", DEFAULT_TIMEOUT))
+        try:
+            self._timeout = int(os.environ.get("ARCCOS_TIMEOUT", DEFAULT_TIMEOUT))
+            if self._timeout <= 0:
+                raise ValueError
+        except (ValueError, TypeError):
+            logger.warning("Invalid ARCCOS_TIMEOUT, using default %ds", DEFAULT_TIMEOUT)
+            self._timeout = DEFAULT_TIMEOUT
 
     # ------------------------------------------------------------------
     # Public interface
@@ -125,7 +132,7 @@ class HttpClient:
                     raise_for_status(resp)
                 except Exception as exc:
                     last_exc = exc
-                    self._backoff(attempt)
+                    self._backoff(attempt, resp)
                     continue
 
             # Non-retryable error (4xx other than 429)
@@ -135,7 +142,13 @@ class HttpClient:
         raise last_exc  # type: ignore[misc]
 
     @staticmethod
-    def _backoff(attempt: int) -> None:
+    def _backoff(attempt: int, resp: requests.Response | None = None) -> None:
         delay = RETRY_BACKOFF * (2 ** attempt)
+        # Respect Retry-After header if present (429 responses)
+        if resp is not None:
+            retry_after = resp.headers.get("Retry-After")
+            if retry_after:
+                with contextlib.suppress(ValueError, TypeError):
+                    delay = max(float(retry_after), delay)
         logger.warning("Retrying in %.1fs (attempt %d/%d)", delay, attempt + 1, MAX_RETRIES)
         time.sleep(delay)

{arccos_api-0.2.0 → arccos_api-0.2.1}/arccos/auth.py

@@ -87,7 +87,7 @@ class Credentials:
             os.fchmod(fd, 0o600)
             with os.fdopen(fd, "w") as f:
                 json.dump(self.to_dict(), f, indent=2)
-        except BaseException:
+        except Exception:
             with contextlib.suppress(OSError):
                 os.close(fd)
             raise

{arccos_api-0.2.0 → arccos_api-0.2.1}/arccos/client.py

@@ -95,7 +95,7 @@ class ArccosClient:
             else:
                 raise ValueError(
                     "Provide email and password, or ensure cached credentials exist "
-                    f"at {creds_path}."
+                    "at ~/.arccos_creds.json."
                 )
 
         self._http = HttpClient(auth=self._auth, creds=self._creds)

{arccos_api-0.2.0 → arccos_api-0.2.1}/arccos/exceptions.py

@@ -58,10 +58,12 @@ def raise_for_status(response) -> None:
         body = response.json()
         error = body.get("error", {})
         code = error.get("code", status)
-        message = error.get("description", response.text[:200])
+        raw = response.text
+        message = error.get("description", raw[:200] + ("\u2026" if len(raw) > 200 else ""))
     except Exception:
         code = status
-        message = response.text[:200]
+        raw = response.text
+        message = raw[:200] + ("\u2026" if len(raw) > 200 else "")
 
     if status == 401 or (isinstance(code, int) and 40100 <= code <= 40199):
         raise ArccosAuthError(message, status_code=status, error_code=code)

{arccos_api-0.2.0 → arccos_api-0.2.1}/arccos_api.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: arccos-api
-Version: 0.2.0
+Version: 0.2.1
 Summary: Unofficial Python CLI and client library for the Arccos Golf API
 Author: Paul Frederiksen
 License: MIT

{arccos_api-0.2.0 → arccos_api-0.2.1}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "arccos-api"
-version = "0.2.0"
+version = "0.2.1"
 description = "Unofficial Python CLI and client library for the Arccos Golf API"
 readme = "README.md"
 license = {text = "MIT"}

{arccos_api-0.2.0 → arccos_api-0.2.1}/tests/test_exceptions.py

@@ -83,7 +83,8 @@ class TestRaiseForStatus:
         resp = _mock_response(500, text="x" * 500)
         with pytest.raises(ArccosError) as exc_info:
             raise_for_status(resp)
-        assert len(str(exc_info.value)) <= 200
+        # 200 chars + ellipsis indicator
+        assert len(str(exc_info.value)) <= 201
 
 
 class TestArccosError: