fix exception logging

Author: Phil Varner

CHANGELOG.md

@@ -29,7 +29,7 @@ none
 
 ### Fixed
 
-none
+- Exception logging
 
 ### Security
 

src/stapi_fastapi/routers/product_router.py

@@ -2,7 +2,7 @@
 
 import logging
 from typing import TYPE_CHECKING, Self
-
+import traceback
 from fastapi import APIRouter, HTTPException, Request, Response, status
 from geojson_pydantic.geometries import Geometry
 from returns.result import Failure, Success
@@ -22,6 +22,8 @@
 if TYPE_CHECKING:
     from stapi_fastapi.routers import RootRouter
 
+logger = logging.getLogger(__name__)
+
 
 class ProductRouter(APIRouter):
     def __init__(
@@ -182,7 +184,10 @@ async def search_opportunities(
             case Failure(e) if isinstance(e, ConstraintsException):
                 raise e
             case Failure(e):
-                logging.exception("An error occurred while searching opportunities", e)
+                logger.exception(
+                    "An error occurred while searching opportunities: %s",
+                    traceback.format_exception(e),
+                )
                 raise HTTPException(
                     status_code=status.HTTP_500_INTERNAL_SERVER_ERROR,
                     detail="Error searching opportunities",
@@ -221,7 +226,10 @@ async def create_order(
             case Failure(e) if isinstance(e, ConstraintsException):
                 raise e
             case Failure(e):
-                logging.exception("An error occurred while creating order", e)
+                logger.exception(
+                    "An error occurred while creating order: %s",
+                    traceback.format_exception(e),
+                )
                 raise HTTPException(
                     status_code=status.HTTP_500_INTERNAL_SERVER_ERROR,
                     detail="Error creating order",

src/stapi_fastapi/routers/root_router.py

@@ -1,6 +1,6 @@
 import logging
 from typing import Self
-
+import traceback
 from fastapi import APIRouter, HTTPException, Request, status
 from fastapi.datastructures import URL
 from fastapi.responses import Response
@@ -23,6 +23,8 @@
 from stapi_fastapi.responses import GeoJSONResponse
 from stapi_fastapi.routers.product_router import ProductRouter
 
+logger = logging.getLogger(__name__)
+
 
 class RootRouter(APIRouter):
     def __init__(
@@ -176,7 +178,10 @@ async def get_orders(self, request: Request) -> OrderCollection:
                     )
                 return orders
             case Failure(e):
-                logging.exception("An error occurred while retrieving orders", e)
+                logger.exception(
+                    "An error occurred while retrieving orders: %s",
+                    traceback.format_exception(e),
+                )
                 raise HTTPException(
                     status_code=status.HTTP_500_INTERNAL_SERVER_ERROR,
                     detail="Error finding Orders",
@@ -195,8 +200,9 @@ async def get_order(self: Self, order_id: str, request: Request) -> Order:
             case Success(Maybe.empty):
                 raise NotFoundException("Order not found")
             case Failure(e):
-                logging.exception(
-                    f"An error occurred while retrieving order '{order_id}'", e
+                logger.exception(
+                    "An error occurred while retrieving order '%s': %s", order_id,
+                    traceback.format_exception(e),
                 )
                 raise HTTPException(
                     status_code=status.HTTP_500_INTERNAL_SERVER_ERROR,
@@ -226,8 +232,9 @@ async def get_order_statuses(
                     ],
                 )
             case Failure(e):
-                logging.exception(
-                    "An error occurred while retrieving order statuses", e
+                logger.exception(
+                    "An error occurred while retrieving order statuses: %s",
+                    traceback.format_exception(e),
                 )
                 raise HTTPException(
                     status_code=status.HTTP_500_INTERNAL_SERVER_ERROR,
@@ -243,7 +250,10 @@ async def set_order_status(
             case Success(_):
                 return Response(status_code=status.HTTP_202_ACCEPTED)
             case Failure(e):
-                logging.exception("An error occurred while setting order status", e)
+                logger.exception(
+                    "An error occurred while setting order status: %s",
+                    traceback.format_exception(e),
+                )
                 raise HTTPException(
                     status_code=status.HTTP_500_INTERNAL_SERVER_ERROR,
                     detail="Error setting Order Status",