road24-artifacthub 0.1.0__py3-none-any.whl → 0.1.2__py3-none-any.whl

{road24_artifacthub-0.1.0.dist-info → road24_artifacthub-0.1.2.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: road24-artifacthub
-Version: 0.1.0
+Version: 0.1.2
 Summary: Shared logging and metrics library for Road24 FastAPI microservices
 Requires-Python: >=3.12
 Requires-Dist: build>=1.4.0
@@ -56,28 +56,27 @@ pip install road24-artifacthub[dev]
 ```python
 import road24_sdk
 from fastapi import FastAPI
+from httpx import AsyncClient, Timeout
 from road24_sdk.integrations.fastapi import FastApiLoggingIntegration
 from road24_sdk.integrations.httpx import HttpxLoggingIntegration
 from road24_sdk.integrations.redis import RedisLoggingIntegration
 from road24_sdk.integrations.sqlalchemy import SqlalchemyLoggingIntegration
 
-road24_sdk.init(
-    service_name="my-service",
-    log_level="INFO",
-    integrations=[
-        FastApiLoggingIntegration(),
-        HttpxLoggingIntegration(),
-        SqlalchemyLoggingIntegration(),
-        RedisLoggingIntegration(),
-    ],
-)
-
-# FastAPI still requires .setup(app) since it needs the app instance
+def setup_road24_sdk():
+    road24_sdk.init(
+        service_name="my-service",
+        log_level="INFO",
+        integrations=[
+            HttpxLoggingIntegration(),
+            SqlalchemyLoggingIntegration(),
+            RedisLoggingIntegration(),
+        ],
+    )
+
+# FastAPI requires .setup(app) since it needs the app instance
 app = FastAPI()
+setup_road24_sdk()
 FastApiLoggingIntegration().setup(app)
-
-# All other clients are auto-instrumented — no .setup() needed
-client = AsyncClient(timeout=Timeout(25.0))  # already instrumented
 ```
 
 ## Available Integrations
@@ -89,16 +88,6 @@ client = AsyncClient(timeout=Timeout(25.0))  # already instrumented
 | `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.0.dist-info → road24_artifacthub-0.1.2.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=G5TgHJoA9Qkwb9BUhVzANo4HWARIRvGLe_-PG3MX3Ew,5152
 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.0.dist-info/METADATA,sha256=DnB13VHVn2yG1vY2WoRwYoP9QJR_SbTledydFRCBa50,4131
-road24_artifacthub-0.1.0.dist-info/WHEEL,sha256=WLgqFyCfm_KASv4WHyYy0P3pM_m7J5L9k2skdKLirC8,87
-road24_artifacthub-0.1.0.dist-info/RECORD,,
+road24_artifacthub-0.1.2.dist-info/METADATA,sha256=h52ypVoTSooqJ_JiC7VMbkCBh0De_8tmtmP6Ck_yzl0,3722
+road24_artifacthub-0.1.2.dist-info/WHEEL,sha256=WLgqFyCfm_KASv4WHyYy0P3pM_m7J5L9k2skdKLirC8,87
+road24_artifacthub-0.1.2.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

@@ -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._decode_body(request.content, request.headers.get("content-type"))
+        response_body = self._decode_body(response.content, response.headers.get("content-type"))
 
         http_attrs = HttpAttributes(
             direction=HttpDirection.OUTPUT,
@@ -63,6 +63,31 @@ class HttpOutputLogger:
         log_level = self._get_log_level(response.status_code)
         getattr(logger, log_level)(LogType.HTTP_REQUEST, extra=http_attrs.as_dict())
 
+    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"