structlog-config 0.4.1__tar.gz → 0.4.2__tar.gz

{structlog_config-0.4.1 → structlog_config-0.4.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.3
 Name: structlog-config
-Version: 0.4.1
+Version: 0.4.2
 Summary: A comprehensive structlog configuration with sensible defaults for development and production environments, featuring context management, exception formatting, and path prettification.
 Keywords: logging,structlog,json-logging,structured-logging
 Author: Michael Bianco

{structlog_config-0.4.1 → structlog_config-0.4.2}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "structlog-config"
-version = "0.4.1"
+version = "0.4.2"
 description = "A comprehensive structlog configuration with sensible defaults for development and production environments, featuring context management, exception formatting, and path prettification."
 keywords = ["logging", "structlog", "json-logging", "structured-logging"]
 readme = "README.md"

{structlog_config-0.4.1 → structlog_config-0.4.2}/structlog_config/__init__.py

@@ -9,9 +9,6 @@ from structlog.processors import ExceptionRenderer
 from structlog.tracebacks import ExceptionDictTransformer
 from structlog.typing import FilteringBoundLogger
 
-from structlog_config.fastapi_access_logger import (
-    client_ip_from_request,
-)
 from structlog_config.formatters import (
     PathPrettifier,
     add_fastapi_context,

{structlog_config-0.4.1 → structlog_config-0.4.2}/structlog_config/fastapi_access_logger.py

@@ -1,3 +1,7 @@
+"""
+Requires fastapi and is not loaded by default since fastapi is not a default dependency.
+"""
+
 from time import perf_counter
 from urllib.parse import quote
 
@@ -11,7 +15,8 @@ from starlette.routing import Match, Mount
 from starlette.types import Scope
 from starlette.websockets import WebSocket
 
-log = structlog.get_logger("access_log")
+# should name this access "access_log" or something
+log = structlog.get_logger()
 ipw = IpWare()
 
 
@@ -109,6 +114,8 @@ def is_static_assets_request(scope: Scope) -> bool:
     )
 
 
+# TODO issue with this approach is if there is an error we don't get a path logged :/
+#      maybe we should log a ERROR with the path information and wrap it with a try?
 def add_middleware(
     app: FastAPI,
 ) -> None: