resolved comment

Author: rkritika1508

backend/app/api/routes/banlist_configs.py

@@ -1,7 +1,7 @@
 from typing import Optional
 from uuid import UUID
 
-from fastapi import APIRouter
+from fastapi import APIRouter, HTTPException
 
 from app.api.deps import AuthDep, SessionDep
 from app.crud.banlist import banlist_crud
@@ -28,8 +28,13 @@ def create_banlist(
     project_id: int,
     _: AuthDep,
 ):
-    response_model = banlist_crud.create(session, payload, organization_id, project_id)
-    return APIResponse.success_response(data=response_model)
+    try:
+        response_model = banlist_crud.create(session, payload, organization_id, project_id)
+        return APIResponse.success_response(data=response_model)
+    except HTTPException as exc:
+        return APIResponse.failure_response(error=str(exc.detail))
+    except Exception as exc:
+        return APIResponse.failure_response(error=str(exc))
 
 @router.get(
         "/", 
@@ -42,8 +47,13 @@ def list_banlists(
     _: AuthDep,
     domain: Optional[str] = None,
 ):
-    response_model = banlist_crud.list(session, organization_id, project_id, domain)
-    return APIResponse.success_response(data=response_model)
+    try:
+        response_model = banlist_crud.list(session, organization_id, project_id, domain)
+        return APIResponse.success_response(data=response_model)
+    except HTTPException as exc:
+        return APIResponse.failure_response(error=str(exc.detail))
+    except Exception as exc:
+        return APIResponse.failure_response(error=str(exc))
 
 
 @router.get(
@@ -57,8 +67,13 @@ def get_banlist(
     session: SessionDep,
     _: AuthDep,
 ):
-    obj = banlist_crud.get(session, id, organization_id, project_id)
-    return APIResponse.success_response(data=obj)
+    try:
+        obj = banlist_crud.get(session, id, organization_id, project_id)
+        return APIResponse.success_response(data=obj)
+    except HTTPException as exc:
+        return APIResponse.failure_response(error=str(exc.detail))
+    except Exception as exc:
+        return APIResponse.failure_response(error=str(exc))
 
 
 @router.patch(
@@ -73,9 +88,19 @@ def update_banlist(
     session: SessionDep,
     _: AuthDep,
 ):
-    obj = banlist_crud.get(session, id, organization_id, project_id)
-    response_model = banlist_crud.update(session, obj=obj, data=payload)
-    return APIResponse.success_response(data=response_model)
+    try:
+        response_model = banlist_crud.update(
+            session,
+            id=id,
+            organization_id=organization_id,
+            project_id=project_id,
+            data=payload,
+        )
+        return APIResponse.success_response(data=response_model)
+    except HTTPException as exc:
+        return APIResponse.failure_response(error=str(exc.detail))
+    except Exception as exc:
+        return APIResponse.failure_response(error=str(exc))
 
 @router.delete(
         "/{id}",
@@ -88,6 +113,11 @@ def delete_banlist(
     session: SessionDep,
     _: AuthDep,
 ):
-    obj = banlist_crud.get(session, id, organization_id, project_id)
-    banlist_crud.delete(session, obj)
-    return APIResponse.success_response(data={"message": "Banlist deleted successfully"})
+    try:
+        obj = banlist_crud.get(session, id, organization_id, project_id)
+        banlist_crud.delete(session, obj)
+        return APIResponse.success_response(data={"message": "Banlist deleted successfully"})
+    except HTTPException as exc:
+        return APIResponse.failure_response(error=str(exc.detail))
+    except Exception as exc:
+        return APIResponse.failure_response(error=str(exc))

backend/app/crud/banlist.py

@@ -80,9 +80,12 @@ def list(
     def update(
         self,
         session: Session,
-        obj: BanList,
+        id: UUID,
+        organization_id: int,
+        project_id: int,
         data: BanListUpdate,
     ) -> BanList:
+        obj = self.get(session, id, organization_id, project_id)
         update_data = data.model_dump(exclude_unset=True)
 
         for k, v in update_data.items():