base logging setup

Author: Karthikeyannhs

src/eligibility_signposting_api/app.py

@@ -8,11 +8,13 @@
 from mangum.types import LambdaContext, LambdaEvent
 
 from eligibility_signposting_api import repos, services
-from eligibility_signposting_api.config import LOG_LEVEL, config, init_logging
+from eligibility_signposting_api.config import LOG_LEVEL, config, setup_logging
 from eligibility_signposting_api.error_handler import handle_exception
 from eligibility_signposting_api.views.eligibility import eligibility
 from eligibility_signposting_api.views.hello import hello
 
+setup_logging()
+logger = logging.getLogger(__name__)
 
 def main() -> None:  # pragma: no cover
     """Run the Flask app as a local process."""
@@ -27,10 +29,9 @@ def lambda_handler(event: LambdaEvent, context: LambdaContext) -> dict[str, Any]
 
 
 def create_app() -> Flask:
-    init_logging()
 
     app = Flask(__name__)
-    app.logger.info("app created")
+    logger.info("app created")
 
     # Register views & error handler
     app.register_blueprint(eligibility, url_prefix="/eligibility")
@@ -41,7 +42,7 @@ def create_app() -> Flask:
     container = wireup.create_container(service_modules=[services, repos], parameters=config())
     wireup.integration.flask.setup(container, app, import_flask_config=True)
 
-    app.logger.info("app ready")
+    logger.info("app ready")
     return app
 
 

src/eligibility_signposting_api/config.py

@@ -1,6 +1,5 @@
 import logging
 import os
-from logging.config import dictConfig
 from typing import Any, NewType
 
 from yarl import URL
@@ -21,23 +20,13 @@ def config() -> dict[str, Any]:
     }
 
 
-def init_logging() -> None:
-    level = logging.getLevelName(LOG_LEVEL)
+def setup_logging():
     log_format = "%(asctime)s %(levelname)-8s %(name)s %(module)s.py:%(funcName)s():%(lineno)d %(message)s"
-    dictConfig(
-        {
-            "version": 1,
-            "formatters": {
-                "default": {
-                    "format": log_format,
-                }
-            },
-            "handlers": {
-                "wsgi": {"class": "logging.StreamHandler", "stream": "ext://sys.stdout", "formatter": "default"}
-            },
-            "root": {"level": level, "handlers": ["wsgi"]},
-            "loggers": {
-                "eligibility_signposting_api.app": {"level": level, "handlers": ["wsgi"], "propagate": False},
-            },
-        }
+    # Define the root logger configuration
+    logging.basicConfig(
+        level=logging.DEBUG,  # Set the base log level
+        format=log_format,
+        handlers=[
+            logging.StreamHandler()
+        ]
     )

src/eligibility_signposting_api/repos/person_repo.py

@@ -1,4 +1,5 @@
 import logging
+from logging.config import dictConfig
 from typing import Annotated, Any
 
 from boto3.resources.base import ServiceResource
@@ -7,13 +8,18 @@
 from eligibility_signposting_api.model.person import Name, Person
 from eligibility_signposting_api.repos.exceptions import NotFoundError
 
-logger = logging.getLogger(__name__)
 
+logger = logging.getLogger(__name__)
 
 @service(qualifier="people_table")
 def people_table_factory(dynamodb_resource: Annotated[ServiceResource, Inject(qualifier="dynamodb")]) -> Any:
     table = dynamodb_resource.Table("People")  # type: ignore[reportAttributeAccessIssue]
+    print("hi ash 999")
     logger.info("built people_table: %r", table)
+    logger.warning("built people_table: %r", table)
+    logger.debug("built people_table: %r", table)
+    logger.error("built people_table: %r", table)
+    print("hi ash 1000")
     return table