finalsa-common-http-client 0.0.8__tar.gz → 0.1.0__tar.gz

{finalsa_common_http_client-0.0.8 → finalsa_common_http_client-0.1.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: finalsa-common-http-client
-Version: 0.0.8
+Version: 0.1.0
 Summary: HTTP client library for common data types used in business applications
 Project-URL: Homepage, https://github.com/finalsa/finalsa-http-client
 Project-URL: Documentation, https://github.com/finalsa/finalsa-http-client#readme
@@ -45,11 +45,11 @@ Classifier: Topic :: Software Development :: Libraries :: Python Modules
 Classifier: Topic :: Software Development :: Localization
 Classifier: Typing :: Typed
 Requires-Python: >=3.10
-Requires-Dist: aiohttp>=3.10.5
-Requires-Dist: fastapi>=0.116.1
-Requires-Dist: finalsa-common-models>=2.0.4
+Requires-Dist: aiohttp>=3.13.3
+Requires-Dist: fastapi>=0.128.0
+Requires-Dist: finalsa-common-models>=2.1.1
 Requires-Dist: finalsa-traceability>=1.0.1
-Requires-Dist: requests>=2.32.4
+Requires-Dist: requests>=2.32.5
 Description-Content-Type: text/markdown
 
 # Finalsa HTTP Client

finalsa_common_http_client-0.1.0/finalsa/http/__init__.py

@@ -0,0 +1,24 @@
+from ._shared import (
+    ValidationError,
+    build_default_headers,
+    build_url,
+    ensure_scheme,
+    get_package_version,
+    merge_headers,
+    normalize_base_url,
+)
+from .async_client import BaseAsyncHttpClient
+from .sync_client import BaseSyncHttpClient
+
+__all__ = [
+    "BaseAsyncHttpClient",
+    "BaseSyncHttpClient",
+    "ValidationError",
+    "build_default_headers",
+    "build_url",
+    "ensure_scheme",
+    "get_package_version",
+    "merge_headers",
+    "normalize_base_url",
+]
+

{finalsa_common_http_client-0.0.8 → finalsa_common_http_client-0.1.0}/finalsa/http/_shared.py

@@ -2,7 +2,7 @@ from __future__ import annotations
 
 from collections.abc import Mapping
 from importlib import metadata
-from typing import Any, Dict
+from typing import Any, Dict, List
 from urllib.parse import urljoin
 
 from finalsa.common.models import BaseDomainException
@@ -11,6 +11,7 @@ from finalsa.traceability.functions import (
     HTTP_HEADER_TRACEPARENT,
     HTTP_HEADER_TRACESTATE,
 )
+from orjson import dumps
 
 Headers = Mapping[str, str]
 
@@ -27,6 +28,32 @@ class InternalHttpError(BaseDomainException):
         )
 
 
+class ValidationError(BaseDomainException):
+    """Exception raised when FastAPI/Pydantic returns validation errors (422)."""
+
+    response_code = 422
+
+    def __init__(
+        self,
+        fields: List[Dict[str, Any]],
+        error: List[Dict[str, Any]],
+        response_code: int | None = None,
+    ) -> None:
+        message = "Validation error"
+        if fields:
+            msg_parts = [
+                f"{'/'.join(str(loc) for loc in f.get('loc', []))}: {f.get('msg', '')}" for f in fields]
+            if msg_parts:
+                message = "; ".join(msg_parts)
+        super().__init__(
+            message=message,
+            response_code=response_code or self.response_code,
+            name="ValidationError",
+        )
+        self.fields = fields
+        self.error = error
+
+
 def normalize_base_url(
     base_url: str | None,
     default_scheme: str,
@@ -42,6 +69,20 @@ def normalize_base_url(
     return value.rstrip("/")
 
 
+def parse_params(params: Mapping[str, Any] | None ) -> dict[str, str | int | float | bytes] | None:
+    if params is None:
+        return None
+    result = {}
+    for key, value in params.items():
+        if value is not None:
+            result[key] = value
+        if isinstance(value, dict):
+            result[key] = dumps(value)
+        if isinstance(value, list):
+            result[key] = dumps(value)
+    return result
+
+
 def ensure_scheme(candidate: str, default_scheme: str) -> str:
     """Ensure that the provided candidate has an explicit scheme."""
     trimmed = candidate.strip()
@@ -107,7 +148,20 @@ def get_package_version(package_name: str = "finalsa-http-client") -> str:
         return "0.0.0"
 
 
+def _is_pydantic_validation_error(response: Dict[str, Any], status: int) -> bool:
+    return status == 422 and "detail" in response and isinstance(response["detail"], list)
+
+
 def raise_for_response(response: Dict[str, Any], status: int) -> None:
+    if _is_pydantic_validation_error(response, status):
+        detail = response["detail"]
+        fields = [d for d in detail if isinstance(
+            d, dict) and ("loc" in d or "msg" in d)]
+        raise ValidationError(
+            fields=fields,
+            error=detail,
+            response_code=status,
+        )
     if response.get("message") and response.get("name"):
         raise BaseDomainException(
             message=response.get("message"),

{finalsa_common_http_client-0.0.8 → finalsa_common_http_client-0.1.0}/finalsa/http/async_client/base.py

@@ -1,12 +1,16 @@
 from __future__ import annotations
 
+import logging
 import ssl
+import time
 from collections.abc import Mapping
-from typing import Any
+from typing import Any, Type
 
 import aiohttp
 import certifi
 from aiohttp import ClientSession, ClientTimeout
+from finalsa.common.models import PaginationRequest, PaginationResponse
+from pydantic import BaseModel
 
 from finalsa.http import _shared as shared
 
@@ -45,6 +49,7 @@ class BaseAsyncHttpClient:
             service_name,
         )
         self._raise_for_status = raise_for_status
+        self._logger = logging.getLogger("http")
 
     async def __aenter__(self) -> BaseAsyncHttpClient:
         await self._ensure_session()
@@ -86,7 +91,11 @@ class BaseAsyncHttpClient:
             return
         if "Content-Type" in headers and headers["Content-Type"] == "application/json":
             data = await response.json()
+            self._logger.error(f"Received response with status {response.status}", extra={
+                               "status": response.status, "headers": headers, "data": data})
             shared.raise_for_response(data, response.status)
+        self._logger.error(f"Received response with status {response.status}", extra={
+                           "status": response.status, "headers": headers})
         raise shared.InternalHttpError(response.status)
 
     async def request(
@@ -115,17 +124,25 @@ class BaseAsyncHttpClient:
         )
         merged_headers = shared.merge_headers(self._default_headers, headers)
         request_timeout = self._normalize_timeout(timeout) or self._timeout
+        self._logger.info(f"{method.upper()} request to {resolved_url} with headers {merged_headers} and timeout {request_timeout}", extra={
+                          "headers": merged_headers, "timeout": request_timeout, "params": params})
+        start_time = time.time()
         response = await session.request(
             method.upper(),
             resolved_url,
             headers=merged_headers,
-            params=params,
+            params=shared.parse_params(params),
             json=json,
             data=data,
             timeout=request_timeout,
             ssl=ssl_ctx,
             **kwargs,
         )
+        end_time = time.time()
+        duration = end_time - start_time
+        self._logger.info(
+            f"Received response from {resolved_url} with status {response.status}",
+            extra={"status": response.status, "duration": duration})
         return response
 
     async def get(self, path: str, **kwargs: Any) -> aiohttp.ClientResponse:
@@ -163,6 +180,14 @@ class BaseAsyncHttpClient:
             )
         return self._session
 
+    async def get_paginated(self, path: str, pagination_request: PaginationRequest, response_model: Type[BaseModel], **kwargs: Any) -> PaginationResponse[Type[BaseModel]]:
+        response = await self.get(path, params=pagination_request.model_dump(), **kwargs)
+        data = await response.json()
+        items = [response_model.model_validate_json(
+            item) for item in data["items"]]
+        data["items"] = items
+        return PaginationResponse[response_model](**data)
+
     def _normalize_timeout(
         self,
         timeout: float | ClientTimeout | None,

{finalsa_common_http_client-0.0.8 → finalsa_common_http_client-0.1.0}/finalsa/http/sync_client/base.py

@@ -1,9 +1,13 @@
 from __future__ import annotations
 
+import logging
+import time
 from collections.abc import Mapping
-from typing import Any
+from typing import Any, Type
 
 import requests
+from finalsa.common.models import PaginationRequest, PaginationResponse
+from pydantic import BaseModel
 
 from finalsa.http import _shared as shared
 
@@ -40,6 +44,7 @@ class BaseSyncHttpClient:
             default_headers,
             service_name,
         )
+        self._logger = logging.getLogger("http")
 
     def __enter__(self) -> BaseSyncHttpClient:
         self._ensure_session()
@@ -100,23 +105,35 @@ class BaseSyncHttpClient:
         merged_headers = shared.merge_headers(self._default_headers, headers)
         request_timeout = timeout if timeout is not None else self._timeout
         try:
+            self._logger.info(f"{method.upper()} request to {resolved_url} with headers {merged_headers} and timeout {request_timeout}", extra={
+                              "headers": merged_headers, "timeout": request_timeout, "params": params})
+            start_time = time.time()
             response = session.request(
                 method.upper(),
                 resolved_url,
                 headers=merged_headers,
-                params=params,
+                params=shared.parse_params(params),
                 json=json,
                 data=data,
                 timeout=request_timeout,
                 **kwargs,
             )
+            end_time = time.time()
+            duration = end_time - start_time
+            self._logger.info(f"Received response from {resolved_url} with status {response.status}", extra={
+                              "status": response.status, "duration": duration})
             if self._raise_for_status:
                 response.raise_for_status()
             return response
         except requests.RequestException as e:
             if e.response is not None and "Content-Type" in e.response.headers and e.response.headers["Content-Type"] == "application/json":
                 data = e.response.json()
+                self._logger.error(f"Received response with status {e.response.status_code}", extra={
+                                   "status": e.response.status_code, "headers": e.response.headers, "data": data})
                 shared.raise_for_response(data, e.response.status_code)
+
+            self._logger.error(f"Received response with status {e.response.status_code}", extra={
+                               "status": e.response.status_code, "headers": e.response.headers})
             raise e
 
     def get(self, path: str, **kwargs: Any) -> requests.Response:
@@ -140,6 +157,14 @@ class BaseSyncHttpClient:
             self._session.close()
             self._session = None
 
+    def get_paginated(self, path: str, pagination_request: PaginationRequest, response_model: Type[BaseModel], **kwargs: Any) -> PaginationResponse[Type[BaseModel]]:
+        response = self.get(path, params=pagination_request.model_dump(), **kwargs)
+        data = response.json()
+        items = [response_model.model_validate_json(
+            item) for item in data["items"]]
+        data["items"] = items
+        return PaginationResponse[response_model](**data)
+
     def _ensure_session(self) -> requests.Session:
         if self._session is None:
             session = requests.Session()

{finalsa_common_http_client-0.0.8 → finalsa_common_http_client-0.1.0}/pyproject.toml

@@ -1,15 +1,15 @@
 [project]
 name = "finalsa-common-http-client"
-version = "0.0.8"
+version = "0.1.0"
 description = "HTTP client library for common data types used in business applications"
 readme = "README.md"
 requires-python = ">=3.10"
 dependencies = [
-    "aiohttp>=3.10.5",
-    "fastapi>=0.116.1",
-    "finalsa-common-models>=2.0.4",
+    "aiohttp>=3.13.3",
+    "fastapi>=0.128.0",
+    "finalsa-common-models>=2.1.1",
     "finalsa-traceability>=1.0.1",
-    "requests>=2.32.4",
+    "requests>=2.32.5",
 ]
 authors = [
  {name = "Luis Jimenez", email = "luis@finalsa.com"},

finalsa_common_http_client-0.0.8/finalsa/http/__init__.py

@@ -1,13 +0,0 @@
-from ._shared import (
-    build_default_headers,
-    build_url,
-    ensure_scheme,
-    get_package_version,
-    merge_headers,
-    normalize_base_url,
-)
-from .async_client import BaseAsyncHttpClient
-from .sync_client import BaseSyncHttpClient
-
-__all__ = ["BaseSyncHttpClient", "BaseAsyncHttpClient", "normalize_base_url", "ensure_scheme", "build_url", "merge_headers", "build_default_headers", "get_package_version"]
-