debugbundle-python 0.1.2__tar.gz → 0.1.5__tar.gz

{debugbundle_python-0.1.2/src/debugbundle_python.egg-info → debugbundle_python-0.1.5}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: debugbundle-python
-Version: 0.1.2
+Version: 0.1.5
 Summary: DebugBundle SDK for Python
 Author: DebugBundle
 License-Expression: AGPL-3.0-only

{debugbundle_python-0.1.2 → debugbundle_python-0.1.5}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "debugbundle-python"
-version = "0.1.2"
+version = "0.1.5"
 description = "DebugBundle SDK for Python"
 readme = "README.md"
 requires-python = ">=3.10"

{debugbundle_python-0.1.2 → debugbundle_python-0.1.5}/src/debugbundle/config.py

@@ -13,6 +13,7 @@ class CapturePolicy:
     capture_request_events: str
     capture_breadcrumbs: str
     capture_probe_events: str
+    immediate_client_error_statuses: tuple[int, ...]
 
 
 @dataclass(frozen=True)
@@ -40,6 +41,7 @@ BALANCED_CAPTURE_POLICY = CapturePolicy(
     capture_request_events="failures_only",
     capture_breadcrumbs="exception_only",
     capture_probe_events="buffer_only",
+    immediate_client_error_statuses=(),
 )
 
 MINIMAL_CAPTURE_POLICY = CapturePolicy(
@@ -48,6 +50,7 @@ MINIMAL_CAPTURE_POLICY = CapturePolicy(
     capture_request_events="failures_only",
     capture_breadcrumbs="local_only",
     capture_probe_events="buffer_only",
+    immediate_client_error_statuses=(),
 )
 
 
@@ -117,6 +120,9 @@ def _parse_capture_policy(payload: object) -> CapturePolicy | None:
     capture_request_events = _as_non_empty_string(payload.get("capture_request_events"))
     capture_breadcrumbs = _as_non_empty_string(payload.get("capture_breadcrumbs"))
     capture_probe_events = _as_non_empty_string(payload.get("capture_probe_events"))
+    immediate_client_error_statuses = _parse_immediate_client_error_statuses(
+        payload.get("immediate_client_error_statuses")
+    )
 
     if capture_logs not in {"off", "error", "warning", "info"}:
         return None
@@ -126,6 +132,8 @@ def _parse_capture_policy(payload: object) -> CapturePolicy | None:
         return None
     if capture_probe_events not in {"buffer_only", "standalone_when_activated"}:
         return None
+    if immediate_client_error_statuses is None:
+        return None
 
     return CapturePolicy(
         preset=preset,
@@ -133,9 +141,25 @@ def _parse_capture_policy(payload: object) -> CapturePolicy | None:
         capture_request_events=capture_request_events,
         capture_breadcrumbs=capture_breadcrumbs,
         capture_probe_events=capture_probe_events,
+        immediate_client_error_statuses=immediate_client_error_statuses,
     )
 
 
+def _parse_immediate_client_error_statuses(value: object) -> tuple[int, ...] | None:
+    if value is None:
+        return ()
+    if not isinstance(value, list) or len(value) > 12:
+        return None
+
+    statuses: list[int] = []
+    for item in value:
+        if not isinstance(item, int) or isinstance(item, bool) or item < 400 or item > 499:
+            return None
+        statuses.append(item)
+
+    return tuple(sorted(set(statuses)))
+
+
 def _parse_directive(payload: object) -> RemoteProbeDirective | None:
     if not isinstance(payload, dict):
         return None

{debugbundle_python-0.1.2 → debugbundle_python-0.1.5}/src/debugbundle/core.py

@@ -53,6 +53,11 @@ LEVEL_RANKS = {
     "error": 40,
     "critical": 50,
 }
+BALANCED_IMMEDIATE_REQUEST_STATUSES = {408, 423, 424, 425, 429}
+INVESTIGATIVE_IMMEDIATE_REQUEST_STATUSES = BALANCED_IMMEDIATE_REQUEST_STATUSES | {409}
+BALANCED_STANDARD_ANOMALY_STATUSES = {401, 403, 404, 409, 422}
+BALANCED_HIGH_VOLUME_ANOMALY_STATUSES = {400, 410}
+INVESTIGATIVE_ANOMALY_STATUSES = BALANCED_STANDARD_ANOMALY_STATUSES | BALANCED_HIGH_VOLUME_ANOMALY_STATUSES
 
 
 @dataclass
@@ -604,7 +609,11 @@ class DebugBundleSdk:
             candidate = response.get("status_code") or response.get("response_status")
             if isinstance(candidate, int):
                 status_code = candidate
-        if status_code is not None and status_code >= 500:
+        if _is_immediate_request_incident_status(
+            status_code,
+            self._capture_policy.preset,
+            self._capture_policy.immediate_client_error_statuses,
+        ):
             return True
         if policy == "off":
             return False
@@ -615,7 +624,7 @@ class DebugBundleSdk:
         if status_code is None:
             return policy == "filtered"
         if policy == "failures_only":
-            return False
+            return _is_request_anomaly_candidate_status(status_code, self._capture_policy.preset)
         if policy == "filtered":
             return False
         return True
@@ -846,6 +855,34 @@ def _iso_now(time_provider: Callable[[], float]) -> str:
     return datetime.fromtimestamp(time_provider(), tz=timezone.utc).isoformat().replace("+00:00", "Z")
 
 
+def _is_immediate_request_incident_status(
+    status_code: int | None,
+    preset: str,
+    immediate_client_error_statuses: tuple[int, ...],
+) -> bool:
+    if status_code is None:
+        return False
+    if status_code >= 500:
+        return True
+    if status_code in immediate_client_error_statuses:
+        return True
+    if preset == "investigative":
+        return status_code in INVESTIGATIVE_IMMEDIATE_REQUEST_STATUSES
+    if preset == "balanced":
+        return status_code in BALANCED_IMMEDIATE_REQUEST_STATUSES
+    return False
+
+
+def _is_request_anomaly_candidate_status(status_code: int | None, preset: str) -> bool:
+    if status_code is None or status_code < 400 or status_code >= 500:
+        return False
+    if preset == "investigative":
+        return status_code in INVESTIGATIVE_ANOMALY_STATUSES
+    if preset == "balanced":
+        return status_code in BALANCED_STANDARD_ANOMALY_STATUSES or status_code in BALANCED_HIGH_VOLUME_ANOMALY_STATUSES
+    return False
+
+
 def _time_now() -> float:
     return datetime.now(tz=timezone.utc).timestamp()
 

{debugbundle_python-0.1.2 → debugbundle_python-0.1.5/src/debugbundle_python.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: debugbundle-python
-Version: 0.1.2
+Version: 0.1.5
 Summary: DebugBundle SDK for Python
 Author: DebugBundle
 License-Expression: AGPL-3.0-only

{debugbundle_python-0.1.2 → debugbundle_python-0.1.5}/tests/test_remote_config.py

@@ -226,4 +226,134 @@ def test_capture_policy_filters_logs_and_request_events_from_remote_config() ->
     events = _events(transport)
     assert [event["event_type"] for event in events] == ["log_event", "request_event"]
     assert events[0]["payload"]["message"] == "error kept"
-    assert events[1]["payload"]["response_status"] == 503
+    assert events[1]["payload"]["response_status"] == 503
+
+
+def test_balanced_capture_policy_keeps_request_failure_anomaly_candidates() -> None:
+    clock = ManualClock()
+    fetch = FakeFetch(
+        responses=[
+            FakeConfigResponse(
+                200,
+                {
+                    "probes_enabled": True,
+                    "remote_probes_enabled": True,
+                    "active_probes": [],
+                    "poll_interval_ms": 15000,
+                    "capture_policy": {
+                        "preset": "balanced",
+                        "capture_logs": "warning",
+                        "capture_request_events": "failures_only",
+                        "capture_breadcrumbs": "exception_only",
+                        "capture_probe_events": "buffer_only",
+                    },
+                },
+            )
+        ]
+    )
+    transport = FakeTransport()
+    sdk = DebugBundleSdk(transport=transport, time_provider=clock.time)
+    sdk.init(
+        project_token="dbundle_proj_test",
+        service="checkout-api",
+        environment="production",
+        fetch_impl=fetch,
+    )
+
+    sdk.capture_request({"method": "POST", "path": "/checkout", "headers": {}}, {"status_code": 429})
+    sdk.capture_request({"method": "POST", "path": "/checkout", "headers": {}}, {"status_code": 404})
+    sdk.capture_request({"method": "POST", "path": "/checkout", "headers": {}}, {"status_code": 409})
+    sdk.flush()
+
+    events = _events(transport)
+    request_statuses = [
+        event["payload"]["response_status"]
+        for event in events
+        if event["event_type"] == "request_event"
+    ]
+    assert request_statuses == [429, 404, 409]
+
+
+def test_investigative_capture_policy_promotes_409_even_when_request_capture_is_off() -> None:
+    clock = ManualClock()
+    fetch = FakeFetch(
+        responses=[
+            FakeConfigResponse(
+                200,
+                {
+                    "probes_enabled": True,
+                    "remote_probes_enabled": True,
+                    "active_probes": [],
+                    "poll_interval_ms": 15000,
+                    "capture_policy": {
+                        "preset": "investigative",
+                        "capture_logs": "info",
+                        "capture_request_events": "off",
+                        "capture_breadcrumbs": "standalone",
+                        "capture_probe_events": "standalone_when_activated",
+                    },
+                },
+            )
+        ]
+    )
+    transport = FakeTransport()
+    sdk = DebugBundleSdk(transport=transport, time_provider=clock.time)
+    sdk.init(
+        project_token="dbundle_proj_test",
+        service="checkout-api",
+        environment="production",
+        fetch_impl=fetch,
+    )
+
+    sdk.capture_request({"method": "POST", "path": "/checkout", "headers": {}}, {"status_code": 409})
+    sdk.capture_request({"method": "POST", "path": "/checkout", "headers": {}}, {"status_code": 404})
+    sdk.flush()
+
+    events = _events(transport)
+    assert [event["payload"]["response_status"] for event in events if event["event_type"] == "request_event"] == [409]
+
+
+def test_capture_policy_promotes_configured_client_error_statuses_when_request_capture_is_off() -> None:
+    clock = ManualClock()
+    fetch = FakeFetch(
+        responses=[
+            FakeConfigResponse(
+                200,
+                {
+                    "probes_enabled": True,
+                    "remote_probes_enabled": True,
+                    "active_probes": [],
+                    "poll_interval_ms": 15000,
+                    "capture_policy": {
+                        "preset": "minimal",
+                        "capture_logs": "error",
+                        "capture_request_events": "off",
+                        "capture_breadcrumbs": "local_only",
+                        "capture_probe_events": "buffer_only",
+                        "immediate_client_error_statuses": [422, 403, 403],
+                    },
+                },
+            )
+        ]
+    )
+    transport = FakeTransport()
+    sdk = DebugBundleSdk(transport=transport, time_provider=clock.time)
+    sdk.init(
+        project_token="dbundle_proj_test",
+        service="checkout-api",
+        environment="production",
+        fetch_impl=fetch,
+    )
+
+    sdk.capture_request({"method": "POST", "path": "/checkout", "headers": {}}, {"status_code": 403})
+    sdk.capture_request({"method": "POST", "path": "/checkout", "headers": {}}, {"status_code": 404})
+    sdk.capture_request({"method": "POST", "path": "/checkout", "headers": {}}, {"status_code": 422})
+    sdk.flush()
+
+    events = _events(transport)
+    request_statuses = [
+        event["payload"]["response_status"]
+        for event in events
+        if event["event_type"] == "request_event"
+    ]
+    assert request_statuses == [403, 422]