From 30a063d7ff9bc0ea35b4e40fd5ea326e540647ed Mon Sep 17 00:00:00 2001
From: Daniel McKnight <34697904+NeonDaniel@users.noreply.github.com>
Date: Tue, 18 Feb 2025 13:51:45 -0800
Subject: [PATCH] Handle empty API proxy responses (#38)

# Description
Set default values in `validate_api_proxy_response` to include more
details in error responses to clients

# Issues
Addresses uncaught exception logged in
https://neon-ai.sentry.io/issues/6302853101/events/fefd6b7410644e9bacab910f6349bd00/

# Other Notes
Deployed and tested in the `alpha` namespace
---
 neon_hana/mq_service_api.py | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/neon_hana/mq_service_api.py b/neon_hana/mq_service_api.py
index e1fd292..7dfc639 100644
--- a/neon_hana/mq_service_api.py
+++ b/neon_hana/mq_service_api.py
@@ -57,6 +57,11 @@ def __init__(self, config: dict):
 
     @staticmethod
     def _validate_api_proxy_response(response: dict, query_params: dict):
+        response.setdefault('content',
+                            {"error": "No response content was received",
+                             "raw_response": response,
+                             "raw_query": query_params})
+        response.setdefault('status_code', 500)
         if response['status_code'] == 200:
             try:
                 resp = json.loads(response['content'])