iii-helpers 0.21.0.dev1__tar.gz → 0.21.2.dev1__tar.gz

{iii_helpers-0.21.0.dev1 → iii_helpers-0.21.2.dev1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: iii-helpers
-Version: 0.21.0.dev1
+Version: 0.21.2.dev1
 Summary: Shared helper primitives across iii SDKs.
 Project-URL: Homepage, https://github.com/iii-hq/iii
 Project-URL: Repository, https://github.com/iii-hq/iii

{iii_helpers-0.21.0.dev1 → iii_helpers-0.21.2.dev1}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "iii-helpers"
-version = "0.21.0.dev1"
+version = "0.21.2.dev1"
 description = "Shared helper primitives across iii SDKs."
 authors = [{ name = "III" }]
 license = { text = "Apache-2.0" }

{iii_helpers-0.21.0.dev1 → iii_helpers-0.21.2.dev1}/src/iii_helpers/observability/__init__.py

@@ -1,6 +1,6 @@
 """iii_helpers.observability: shared OTel + Logger primitives."""
 
-from .baggage_span_processor import DEFAULT_ALLOWLIST, BaggageSpanProcessor
+from .baggage_span_processor import BaggageSpanProcessor
 from .http_instrumentation import execute_traced_request
 from .logger import Logger
 from .payload import (
@@ -32,7 +32,6 @@ from .telemetry_types import OtelConfig
 
 __all__ = [
     "BaggageSpanProcessor",
-    "DEFAULT_ALLOWLIST",
     "Logger",
     "OtelConfig",
     "REDACTED_PLACEHOLDER",

iii_helpers-0.21.2.dev1/src/iii_helpers/observability/baggage_span_processor.py

@@ -0,0 +1,40 @@
+"""Baggage -> span attribute processor.
+
+Copies EVERY baggage entry onto every span started in its scope,
+unconditionally -- the same contract as the upstream OTel contrib
+``BaggageSpanProcessor``. There is deliberately no key filtering here: a
+filtering policy baked into worker binaries has to be kept in lockstep
+across every SDK language and every deployed worker, and a stale binary
+silently drops newer keys. Which attributes *mean* something (e.g. the
+``iii.tag.*`` trace-tag namespace, ``iii.session.*``) is a query-side
+convention owned by the engine's traces API, where it can evolve without
+touching workers.
+"""
+
+from __future__ import annotations
+
+from opentelemetry import baggage
+from opentelemetry.context import Context
+from opentelemetry.sdk.trace import ReadableSpan, Span, SpanProcessor
+
+
+class BaggageSpanProcessor(SpanProcessor):
+
+    def on_start(self, span: Span, parent_context: Context | None = None) -> None:
+        # NoOp guard: skip allocation when sampler drops the span.
+        if not span.is_recording():
+            return
+
+        for key, value in baggage.get_all(parent_context).items():
+            if value is None:
+                continue
+            span.set_attribute(key, str(value))
+
+    def on_end(self, span: ReadableSpan) -> None:  # noqa: ARG002
+        pass
+
+    def shutdown(self) -> None:
+        pass
+
+    def force_flush(self, timeout_millis: int = 30000) -> bool:  # noqa: ARG002
+        return True

iii_helpers-0.21.0.dev1/src/iii_helpers/observability/baggage_span_processor.py

@@ -1,42 +0,0 @@
-"""Baggage -> span attribute processor."""
-
-from __future__ import annotations
-
-from typing import Sequence
-
-from opentelemetry import baggage
-from opentelemetry.context import Context
-from opentelemetry.sdk.trace import ReadableSpan, Span, SpanProcessor
-
-#: DEFAULT_ALLOWLIST drift across languages would break worker chains;
-#: lockstep tests in each SDK pin this constant at CI time.
-DEFAULT_ALLOWLIST: tuple[str, ...] = (
-    "iii.session.id",
-    "iii.message.id",
-    "iii.function.id",
-)
-
-
-class BaggageSpanProcessor(SpanProcessor):
-
-    def __init__(self, allowlist: Sequence[str] = DEFAULT_ALLOWLIST) -> None:
-        self._allowlist: tuple[str, ...] = tuple(allowlist)
-
-    def on_start(self, span: Span, parent_context: Context | None = None) -> None:
-        # NoOp guard: skip allocation when sampler drops the span.
-        if not span.is_recording():
-            return
-
-        for key in self._allowlist:
-            value = baggage.get_baggage(key, parent_context)
-            if value is not None:
-                span.set_attribute(key, str(value))
-
-    def on_end(self, span: ReadableSpan) -> None:  # noqa: ARG002
-        pass
-
-    def shutdown(self) -> None:
-        pass
-
-    def force_flush(self, timeout_millis: int = 30000) -> bool:  # noqa: ARG002
-        return True