road24-artifacthub 0.1.1__py3-none-any.whl → 0.1.3__py3-none-any.whl

{road24_artifacthub-0.1.1.dist-info → road24_artifacthub-0.1.3.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: road24-artifacthub
-Version: 0.1.1
+Version: 0.1.3
 Summary: Shared logging and metrics library for Road24 FastAPI microservices
 Requires-Python: >=3.12
 Requires-Dist: build>=1.4.0
@@ -88,16 +88,6 @@ FastApiLoggingIntegration().setup(app)
 | `SqlalchemyLoggingIntegration` | `sqlalchemy` | Database query logging and metrics |
 | `RedisLoggingIntegration` | `redis` | Redis command logging and metrics |
 
-Pass integrations to `init()` for automatic class-level patching. All new client instances are auto-instrumented.
-
-The `.setup(client)` method is still available for per-instance patching (backwards compatible):
-
-```python
-# Per-instance patching (still works)
-client = AsyncClient(timeout=Timeout(25.0))
-HttpxLoggingIntegration().setup(client)
-```
-
 ## Manual Testing
 
 Standalone scripts to inspect structured JSON logs and metrics (no server needed):

{road24_artifacthub-0.1.1.dist-info → road24_artifacthub-0.1.3.dist-info}/RECORD

@@ -5,14 +5,14 @@ road24_sdk/_schemas.py,sha256=HOlBsIZSN2sm3s2MMbp-JoGGju_jjgZPmm8uNc4BCeI,3026
 road24_sdk/_types.py,sha256=OfRdYJhIAeMOtjPPCx-W9vXXAxYvuKjq37NzCubWIg8,1071
 road24_sdk/integrations/__init__.py,sha256=uCtjD-5z2lQYA1j4jN9pFw1azS095L7BVtoxzz-Rl3c,497
 road24_sdk/integrations/_base.py,sha256=IePXuVsvKmDDf3Aq411KzurXkkq5opelz9nqeXeCr2o,489
-road24_sdk/integrations/fastapi.py,sha256=7DN5jwR4SWMNKSDWu4yYecCP8w0pgktKqeuLMliBHB8,8677
-road24_sdk/integrations/httpx.py,sha256=TZySabu04UnUk0Q7LUlszQzTqpj_lZOvqPHN4QOdCT8,4223
+road24_sdk/integrations/fastapi.py,sha256=OZCb2iFULLDmID9MgyeQ6uGC0La8O8xvUPobI0CmQfA,9404
+road24_sdk/integrations/httpx.py,sha256=ojCBoAuB8oUTnDfNG1i6Kn5lerbWAYh-OcxLHThp9OY,5443
 road24_sdk/integrations/redis.py,sha256=KvLilhaZywGSrUJ4hL7T5BqfKGZWxSYQ7BADRVS5Fq0,7401
 road24_sdk/integrations/sqlalchemy.py,sha256=WOhFpUt7RfknhJL253MxNif16OKK8GJlXQ87sDsN9WQ,5055
 road24_sdk/metrics/__init__.py,sha256=vvXfTFdHYnYHkN9REgqvqiGsTkXnszVKXnqHkb5C1FQ,257
 road24_sdk/metrics/db.py,sha256=Lqqy7k1qF8D9sdxEl4qDP5pwqVHXgo1ucDHgUYD4exs,780
 road24_sdk/metrics/http.py,sha256=Ynbf-IvJwkzU6G2eRNWf99CH8ef-iR7CxfEvHR66OvQ,949
 road24_sdk/metrics/redis.py,sha256=qdDuG-2px7_Hp1eiFRc4x8F0MG7KbcuiDiFxw8QsavA,805
-road24_artifacthub-0.1.1.dist-info/METADATA,sha256=yrSPWG9udBlgOSzQv4EF9nkkOy8M7oGb5dhBcNNS4Pw,4072
-road24_artifacthub-0.1.1.dist-info/WHEEL,sha256=WLgqFyCfm_KASv4WHyYy0P3pM_m7J5L9k2skdKLirC8,87
-road24_artifacthub-0.1.1.dist-info/RECORD,,
+road24_artifacthub-0.1.3.dist-info/METADATA,sha256=H65x4vBDOVaQfYx-ApKsbu_4hDDO472wM4pvnZbZ7XM,3722
+road24_artifacthub-0.1.3.dist-info/WHEEL,sha256=WLgqFyCfm_KASv4WHyYy0P3pM_m7J5L9k2skdKLirC8,87
+road24_artifacthub-0.1.3.dist-info/RECORD,,

road24_sdk/integrations/fastapi.py

@@ -21,6 +21,17 @@ _request_exception: ContextVar[Exception | None] = ContextVar("request_exception
 
 
 class HttpInputLogger:
+    _BINARY_CONTENT_TYPES = (
+        "multipart/form-data",
+        "application/octet-stream",
+        "image/",
+        "audio/",
+        "video/",
+        "application/pdf",
+        "application/zip",
+        "application/gzip",
+    )
+
     def log(
         self,
         scope: "Scope",
@@ -37,6 +48,7 @@ class HttpInputLogger:
         headers = dict(scope.get("headers", []))
         host = headers.get(b"host", b"").decode() or ""
         url = f"{scheme}://{host}{path}" if host else path
+        content_type = headers.get(b"content-type", b"").decode() or ""
 
         if record_metrics:
             from road24_sdk.metrics.http import record_http_request
@@ -56,8 +68,8 @@ class HttpInputLogger:
             target=path,
             status_code=status_code,
             duration_seconds=duration_seconds,
-            request_body=self._decode_body(request_body),
-            response_body=self._decode_body(response_body),
+            request_body=self._decode_body(request_body, content_type),
+            response_body=self._decode_body(response_body, None),
             error_class=type(exc).__name__ if exc else "",
             error_message=str(exc) if exc else "",
         )
@@ -71,14 +83,21 @@ class HttpInputLogger:
         headers = dict(scope.get("headers", []))
         return headers.get(b"x-trace-id", b"").decode() or secrets.token_hex(16)
 
-    def _decode_body(self, body: bytes) -> str:
+    def _decode_body(self, body: bytes, content_type: str | None) -> str:
         if not body:
             return ""
+        if self._is_binary_content(content_type):
+            return f"<binary data: {len(body)} bytes>"
         try:
             decoded = body.decode("utf-8")
             return sanitize_body(decoded)
         except UnicodeDecodeError:
-            return "<binary data>"
+            return f"<binary data: {len(body)} bytes>"
+
+    def _is_binary_content(self, content_type: str | None) -> bool:
+        if not content_type or not isinstance(content_type, str):
+            return False
+        return any(bt in content_type.lower() for bt in self._BINARY_CONTENT_TYPES)
 
     def _get_log_level(self, status_code: int) -> str:
         if status_code < HTTPStatus.BAD_REQUEST:

road24_sdk/integrations/httpx.py

@@ -10,7 +10,7 @@ from road24_sdk._types import HttpDirection, LogType
 from road24_sdk.integrations._base import Integration
 
 if TYPE_CHECKING:
-    from httpx import AsyncClient, Request, Response
+    from httpx import AsyncClient, Request, RequestNotRead, Response
 
 logger = logging.getLogger(__name__)
 
@@ -46,8 +46,8 @@ class HttpOutputLogger:
                 direction=HttpDirection.OUTPUT,
             )
 
-        request_body = request.content.decode("utf-8") if request.content else ""
-        response_body = response.text if response.text else ""
+        request_body = self._get_request_body(request)
+        response_body = self._decode_body(response.content, response.headers.get("content-type"))
 
         http_attrs = HttpAttributes(
             direction=HttpDirection.OUTPUT,
@@ -63,6 +63,40 @@ class HttpOutputLogger:
         log_level = self._get_log_level(response.status_code)
         getattr(logger, log_level)(LogType.HTTP_REQUEST, extra=http_attrs.as_dict())
 
+    def _get_request_body(self, request: "Request") -> str:
+        from httpx import RequestNotRead
+
+        content_type = request.headers.get("content-type")
+        try:
+            return self._decode_body(request.content, content_type)
+        except RequestNotRead:
+            return "<streaming request>"
+
+    def _decode_body(self, content: bytes, content_type: str | None) -> str:
+        if not content:
+            return ""
+        if self._is_binary_content(content_type):
+            return f"<binary data: {len(content)} bytes>"
+        try:
+            return content.decode("utf-8")
+        except UnicodeDecodeError:
+            return f"<binary data: {len(content)} bytes>"
+
+    def _is_binary_content(self, content_type: str | None) -> bool:
+        if not content_type or not isinstance(content_type, str):
+            return False
+        binary_types = (
+            "multipart/form-data",
+            "application/octet-stream",
+            "image/",
+            "audio/",
+            "video/",
+            "application/pdf",
+            "application/zip",
+            "application/gzip",
+        )
+        return any(bt in content_type.lower() for bt in binary_types)
+
     def _get_log_level(self, status_code: int) -> str:
         if status_code < HTTPStatus.BAD_REQUEST:
             return "info"