From afd87c3e9c70215c543f3f81e44f7ae8916e7588 Mon Sep 17 00:00:00 2001 From: Ihor Panasiuk Date: Thu, 26 Jun 2025 16:45:16 +0200 Subject: [PATCH 1/2] fix: Improve alert query error logging by serializing query object --- keep/api/core/alerts.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/keep/api/core/alerts.py b/keep/api/core/alerts.py index 507a12a9cb..3cd21a5928 100644 --- a/keep/api/core/alerts.py +++ b/keep/api/core/alerts.py @@ -393,9 +393,10 @@ def query_last_alerts(tenant_id, query: QueryDto) -> Tuple[list[Alert], int]: data_query = build_alerts_query(tenant_id, query_with_defaults) alerts_with_start = session.execute(data_query).all() + raise Exception("Shit, shit shit") except OperationalError as e: logger.warning( - f"Failed to query alerts for query object '{json.dumps(query_with_defaults)}': {e}" + f"Failed to query alerts for query object '{json.dumps(query_with_defaults.dict())}': {e}" ) return [], 0 From e9292d0f6a57726a3a5587a6f833db5eec3eecb2 Mon Sep 17 00:00:00 2001 From: Ihor Panasiuk Date: Thu, 26 Jun 2025 17:07:17 +0200 Subject: [PATCH 2/2] fix: Enhance alert query error logging by excluding unset fields from query object --- keep/api/core/alerts.py | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/keep/api/core/alerts.py b/keep/api/core/alerts.py index 3cd21a5928..bb2cf3ba52 100644 --- a/keep/api/core/alerts.py +++ b/keep/api/core/alerts.py @@ -393,10 +393,9 @@ def query_last_alerts(tenant_id, query: QueryDto) -> Tuple[list[Alert], int]: data_query = build_alerts_query(tenant_id, query_with_defaults) alerts_with_start = session.execute(data_query).all() - raise Exception("Shit, shit shit") except OperationalError as e: logger.warning( - f"Failed to query alerts for query object '{json.dumps(query_with_defaults.dict())}': {e}" + f"Failed to query alerts for query object '{json.dumps(query_with_defaults.dict(exclude_unset=True))}': {e}" ) return [], 0