debugbundle-python 2.0.0__py3-none-any.whl → 2.0.1__py3-none-any.whl

debugbundle/acknowledgement.py

@@ -19,8 +19,10 @@ class AcknowledgementDecision:
     reason: str | None = None
 
 
-def decide_acknowledgement(body: object | None, batch_length: int) -> AcknowledgementDecision:
+def decide_acknowledgement(body: object | None, batch_length: int, required: bool = False) -> AcknowledgementDecision:
     if not isinstance(body, dict) or not any(key in body for key in ("accepted", "rejected", "errors")):
+        if required:
+            return AcknowledgementDecision(kind="protocol_failure", reason="missing_acknowledgement")
         return AcknowledgementDecision(kind="legacy")
 
     accepted = body.get("accepted")

debugbundle/core.py

@@ -61,7 +61,7 @@ from .redaction import (
 )
 from .request_capture_policy import should_capture_request_event
 from .suppression import EventSuppressionTracker
-from .transport import HttpTransport, Transport, coerce_transport_response
+from .transport import HttpTransport, Transport, bounded_retry_after_ms, coerce_transport_response
 from .trigger_token import resolve_request_trigger_directives
 
 DEFAULT_BATCH_SIZE = 25
@@ -499,11 +499,13 @@ class DebugBundleSdk:
                     return
 
                 if 200 <= response.status_code < 300:
-                    acknowledgement = decide_acknowledgement(response.body, len(batch))
+                    acknowledgement = decide_acknowledgement(
+                        response.body, len(batch), isinstance(transport, HttpTransport)
+                    )
                     if acknowledgement.kind == "protocol_failure":
                         self._consecutive_failures += 1
-                        retry_after_ms = response.retry_after_ms if response.retry_after_ms is not None else 1_000
-                        self._retry_after = now + (retry_after_ms / 1000)
+                        retry_after_ms = bounded_retry_after_ms(response.retry_after_ms)
+                        self._retry_after = self._time_provider() + (retry_after_ms / 1000)
                         self._emit_diagnostic(
                             "ingestion_acknowledgement_invalid",
                             "sdk-python retained a batch after an invalid ingestion acknowledgement",
@@ -535,8 +537,8 @@ class DebugBundleSdk:
                         self._last_event_at = self._time_provider() * 1000
                     if acknowledgement.retryable_indices:
                         self._consecutive_failures += 1
-                        retry_after_ms = response.retry_after_ms if response.retry_after_ms is not None else 1_000
-                        self._retry_after = now + (retry_after_ms / 1000)
+                        retry_after_ms = bounded_retry_after_ms(response.retry_after_ms)
+                        self._retry_after = self._time_provider() + (retry_after_ms / 1000)
                         self._schedule_flush_locked(delay=retry_after_ms / 1000)
                         return
                     self._retry_after = 0.0
@@ -544,9 +546,9 @@ class DebugBundleSdk:
                     return
 
                 self._consecutive_failures += 1
-                if response.status_code == 429:
-                    retry_after_ms = response.retry_after_ms if response.retry_after_ms is not None else 1_000
-                    self._retry_after = now + (retry_after_ms / 1000)
+                if response.status_code == 429 or (response.status_code >= 500 and response.retry_after_ms is not None):
+                    retry_after_ms = bounded_retry_after_ms(response.retry_after_ms)
+                    self._retry_after = self._time_provider() + (retry_after_ms / 1000)
                     self._schedule_flush_locked(delay=retry_after_ms / 1000)
                     return
                 if 400 <= response.status_code < 500:

debugbundle/event_support.py

@@ -264,7 +264,7 @@ def sdk_version() -> str:
     try:
         return metadata.version("debugbundle-python")
     except metadata.PackageNotFoundError:
-        return "2.0.0"
+        return "2.0.1"
 
 
 def sdk_config_endpoint(events_endpoint: str) -> str:

debugbundle/transport.py

@@ -1,11 +1,40 @@
 from __future__ import annotations
 
+import math
+import time
 from collections.abc import Mapping
 from dataclasses import dataclass
+from email.utils import parsedate_to_datetime
 from typing import Any, Protocol
 
 import httpx
 
+MAX_RETRY_AFTER_MS = 300_000
+
+
+def bounded_retry_after_ms(value: object, default: int = 1_000) -> int:
+    # Clamp before conversion so arbitrarily large integer hints cannot overflow a float deadline.
+    if isinstance(value, bool) or not isinstance(value, (int, float)):
+        return default
+    if isinstance(value, float) and not math.isfinite(value):
+        return default
+    return int(min(MAX_RETRY_AFTER_MS, max(0, value)))
+
+
+def _parse_retry_after(value: str | None) -> int | None:
+    if value is None:
+        return None
+    try:
+        seconds = float(value)
+    except ValueError:
+        try:
+            seconds = parsedate_to_datetime(value).timestamp() - time.time()
+        except (ValueError, TypeError, OverflowError):
+            return None
+    if not math.isfinite(seconds):
+        return None
+    return bounded_retry_after_ms(min(300, max(0, seconds)) * 1_000)
+
 
 @dataclass
 class TransportResponse:
@@ -34,13 +63,7 @@ class HttpTransport:
             json={"events": request["events"]},
             headers=headers,
         )
-        retry_after_header = response.headers.get("retry-after")
-        retry_after_ms = None
-        if retry_after_header is not None:
-            try:
-                retry_after_ms = int(float(retry_after_header) * 1000)
-            except ValueError:
-                retry_after_ms = None
+        retry_after_ms = _parse_retry_after(response.headers.get("retry-after"))
 
         try:
             body: object | None = response.json()

{debugbundle_python-2.0.0.dist-info → debugbundle_python-2.0.1.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: debugbundle-python
-Version: 2.0.0
+Version: 2.0.1
 Summary: DebugBundle SDK for Python
 Author: DebugBundle
 License-Expression: Apache-2.0
@@ -357,3 +357,9 @@ CI validates Ruff, mypy, pytest, package build, event schema fixtures, and cover
 ## License
 
 Apache-2.0. See `LICENSE`.
+
+## Delivery acknowledgement and retry limits
+
+The built-in HTTP transport requires the canonical ingestion acknowledgement (`accepted`, `rejected`, and `errors`). Empty responses, unrelated JSON, malformed counts, or invalid rejection indices retain the full batch with backoff. Valid acknowledgements remove accepted and terminally rejected events and retry only the indexed retryable rejections; an all-rejected batch does not advance `lastEventAt`.
+
+File transports and explicitly supplied custom/legacy transports retain their documented bodyless success fallback. A custom transport that returns acknowledgement fields must return the complete canonical shape. Retry hints (delay seconds or HTTP dates) are bounded to five minutes and measured from response receipt, including partial/protocol acknowledgements and retryable server failures. Without a server hint, existing retry timing is preserved. Failures remain contained within the SDK's existing delivery path.

{debugbundle_python-2.0.0.dist-info → debugbundle_python-2.0.1.dist-info}/RECORD

@@ -1,11 +1,11 @@
 debugbundle/__init__.py,sha256=QH5RY2Cdp3Ry5PNSAhiV9akOMoS2pERRqowFnZ94roc,5131
-debugbundle/acknowledgement.py,sha256=tyI8XLRG9N3-Weo6uiGrAlf_O4tjptVe1x1cG7RjrMc,2314
+debugbundle/acknowledgement.py,sha256=2miOWbuIfElrbkktJHEuL7a2-IqKYUZCpO1UjSaouYw,2461
 debugbundle/before_send.py,sha256=_tmV6ovHUBIxfQPMQ8a0C6r91IAnd4DVs8UcjsKhqX0,7792
 debugbundle/capture_hooks.py,sha256=70Q6ske8JulyuYpeE9FohvMreytf4DUrj4QW9f2-o2Q,1940
 debugbundle/config.py,sha256=ENRSR5jUI7xYdBqUakWws3GIjBk00GWFmwXVE_iQp2Y,9296
-debugbundle/core.py,sha256=XgRx92jSzrP6PjvHHRI0NlwdD53cRmBQ6qN4UtnQqIA,39310
+debugbundle/core.py,sha256=uu2_VPvXwmccwwCcK1Y1kJgfehMOCrhFjV4L14X0cok,39467
 debugbundle/delivery_hooks.py,sha256=BUqtHuEv1MDOSLv0sUP6D582c06JofsbXhi7I7Y5IGc,2691
-debugbundle/event_support.py,sha256=MtPg2CO7B91U7-zE9XGro0-fynaEnJZmQ6Ms9arfRlg,9046
+debugbundle/event_support.py,sha256=keOpma92Hxer2mOD9ELw-w5MRFYNOsj5A9C9goLdkDE,9046
 debugbundle/exception_snapshot.py,sha256=_2p55F572sL7_ogBL6Raa137HRyUqvSaHVMwpKiC8fs,2831
 debugbundle/flush_aggregates.py,sha256=ukHsIUykeGp4YJsesA_9NCUMarg_BAXVuzO4bqJKH3Y,1464
 debugbundle/logger_integrations.py,sha256=T6LR10_DgDS8C-CUPxtC1kZz3qfX5j0B2YUV7LxDovk,3572
@@ -19,7 +19,7 @@ debugbundle/relay_delivery.py,sha256=VL-nIgJR6mYXqKy-EbA0USWeY_iS_uAr1KBzrnwBnnk
 debugbundle/request_capture_policy.py,sha256=fl1EilWAnBQCdGTafRpY0tMlYUoSJK_7hZyJmPHLkeI,1583
 debugbundle/structlog_integration.py,sha256=hDwCGXTQ8GAjnx2jwlRbpnvLE0y798kFFMsSB3iqOrM,6061
 debugbundle/suppression.py,sha256=V_JvDvcV7tN3EHkfpEJGK1WB2G4nR3GhXvaKd400iFQ,4418
-debugbundle/transport.py,sha256=V289FwFWL6KV5gyzOsesluEMbpRobm_FHu3x8BpOYKU,2014
+debugbundle/transport.py,sha256=_6L6XdjSSm-YzWsQLH8E5xku6yMS8wBYFpOwuYtY2cg,2764
 debugbundle/trigger_token.py,sha256=YUwIWnnxu1klAVaB8Ltgk_PwWW_PPjq9rzmEbWXeFeM,4455
 debugbundle/integrations/__init__.py,sha256=Jr87ImWXzUXOuFP4WlGleK14VARajSSdGZN_nHt5emw,1161
 debugbundle/integrations/common.py,sha256=iiwf5wDlpujFuWfbO-ziFTn4MtiHBXSIZNXtamhCavg,2014
@@ -29,8 +29,8 @@ debugbundle/integrations/flask.py,sha256=cgyHbsAH96gpNPf_5IGJYzp2va28JdDgXROBsF-
 debugbundle/integrations/relay_django.py,sha256=Wj8BE9D5otU2KmtjgqdKIJ-BvXVcagCKgpLYvnPZ6WE,2281
 debugbundle/integrations/relay_fastapi.py,sha256=-Zl6bvhYMLii__mP6-UvSdwxS5VKsb9OZvk-yy-3hEw,2227
 debugbundle/integrations/relay_flask.py,sha256=VFHkDTJy4LkZzL_Ry_Ba-e_gW6UTG4PBwnLcvB_oXuQ,2011
-debugbundle_python-2.0.0.dist-info/licenses/LICENSE,sha256=z8d0m5b2O9McPEK1xHG_dWgUBT6EfBDz6wA0F7xSPTA,11358
-debugbundle_python-2.0.0.dist-info/METADATA,sha256=oX9_gQfmoPs8PCSNucy0tTany71DEiuHsu71IPmYBmg,15231
-debugbundle_python-2.0.0.dist-info/WHEEL,sha256=YVMoNqKzERt-wjUZwJ33xBGAwnFl-4cqbYkTtWa4itE,91
-debugbundle_python-2.0.0.dist-info/top_level.txt,sha256=RCB9STTFnl1OKdojxz-xhaks2zkRFs1meZXsKnm18LM,12
-debugbundle_python-2.0.0.dist-info/RECORD,,
+debugbundle_python-2.0.1.dist-info/licenses/LICENSE,sha256=z8d0m5b2O9McPEK1xHG_dWgUBT6EfBDz6wA0F7xSPTA,11358
+debugbundle_python-2.0.1.dist-info/METADATA,sha256=5el7IjAErunxEkDXAZNfD8eIirCrVCgB5I4kAYBxC88,16205
+debugbundle_python-2.0.1.dist-info/WHEEL,sha256=YVMoNqKzERt-wjUZwJ33xBGAwnFl-4cqbYkTtWa4itE,91
+debugbundle_python-2.0.1.dist-info/top_level.txt,sha256=RCB9STTFnl1OKdojxz-xhaks2zkRFs1meZXsKnm18LM,12
+debugbundle_python-2.0.1.dist-info/RECORD,,