Merge pull request #398 from Gary-Community-Ventures/feat/webhook_imp…

…rovements Add Logging if Web Hook Fails
Gary-Community-Ventures · Jun 6, 2024 · 093827f · 093827f
2 parents 812d2f3 + fd73db1
commit 093827f
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/benefits/settings.py b/benefits/settings.py
@@ -219,7 +219,7 @@
 DEFAULT_AUTO_FIELD = "django.db.models.BigAutoField"
 CSRF_FAILURE_VIEW = "benefits.views.catch_403_view"
 
-SWAGGER_SETTINGS = {"SUPPORTED_SUBMIT_METHODS": ("get",)}
+SWAGGER_SETTINGS = {"SUPPORTED_SUBMIT_METHODS": ("get",), "DEEP_LINKING": True}
 
 # Enable logging with Sentry if it is enabled
 if config("SENTRY_DSN", None) is not None:

diff --git a/screener/webhooks.py b/screener/webhooks.py
@@ -1,6 +1,7 @@
 from .models import Screen
 from programs.models import Referrer
 from .serializers import ScreenSerializer
+from sentry_sdk import capture_exception
 import requests
 
 
@@ -22,9 +23,8 @@ def send(self, screen: Screen, results: dict):
 
         try:
             requests.post(self.hook.webhook_url, json=request_data)
-        except requests.exceptions.RequestException:
-            # TODO: add logging
-            pass
+        except requests.exceptions.RequestException as e:
+            capture_exception(e)
 
     def screen_data(self, screen: Screen):
         screen_dict = ScreenSerializer(screen).data