Delete soc alerts bug fix (#151)
* Add session parameter to purge_alerts_route() function * precommit fixes
taylor_socfortress committed
Feb 14, 2024 at 15:29 UTC
9ed255fb421beddb56210460d411d37e527507eb
2 files changed
+6
-4
README.md
+2
-2
@@ -108,7 +108,7 @@ copilot-frontend:
108
ports:
109
- "80:80"
110
- "443:443"
111
-````
111
+```
112
113
### Upgrading Copilot
114
@@ -122,7 +122,7 @@ docker compose pull
122
123
# Start the container again
124
docker compose up -d
125
-````
125
+```
126
127
## Connectors
128
backend/app/connectors/dfir_iris/routes/alerts.py
+4
-2
@@ -241,7 +241,9 @@ async def delete_multiple_alerts_route(
241
description="Delete all alerts",
242
dependencies=[Security(AuthHandler().require_any_scope("admin", "analyst"))],
243
)
244
-async def purge_alerts_route() -> DeleteAlertResponse:
244
+async def purge_alerts_route(
245
+ session: AsyncSession = Depends(get_db),
246
+) -> DeleteAlertResponse:
247
"""
248
Delete all alerts.
249
@@ -249,7 +251,7 @@ async def purge_alerts_route() -> DeleteAlertResponse:
251
AlertResponse: The response containing the deleted alerts.
252
"""
253
logger.info("Purging all alerts, up to 1000")
252
- alerts = (await get_alerts(request=FilterAlertsRequest(per_page=1000))).alerts
254
+ alerts = (await get_alerts(request=FilterAlertsRequest(per_page=1000), session=session)).alerts
255
for alert in alerts:
256
await delete_alert(int(alert["alert_id"]))
257
return DeleteAlertResponse(success=True, message="Successfully deleted alerts.")