Fix/84 Ignore webhook without user details

.gitignore

@@ -9,6 +9,7 @@ htmlcov/
 .venv
 env/
 venv/
+*.json
 
 .vscode
 .DS_Store

api/models/call_log_event.py

@@ -4,11 +4,8 @@
 
 
 class CallLogEventQuery(BaseQuery):
-    def call_sid_exist(self, form_data):
-        return (
-            self.filter(CallLogEvent.call_sid == form_data["CallSid"]).first()
-            is not None
-        )
+    def call_sid_exist(self, call_sid):
+        return self.filter(CallLogEvent.call_sid == call_sid).first() is not None
 
 
 class CallLogEvent(TimestampMixin, db.Model):

api/services/handle_event_service.py

@@ -9,7 +9,9 @@
 
 class HandleEventService:
     def handle_event_service(self, form_data):
-        call_sid_exist = models.CallLogEvent.query.call_sid_exist(form_data)
+        call_sid_exist = models.CallLogEvent.query.call_sid_exist(
+            form_data.get("CallSid")
+        )
 
         if call_sid_exist:
             return
@@ -23,6 +25,14 @@ def handle_event_service(self, form_data):
         if not system_phone_exists:
             return
 
+        user_details_exist = form_data.get("From")
+
+        if user_details_exist is None:
+            logger.warning(
+                f"User details are missing in the webhook payload: {form_data}"
+            )
+            return
+
         try:
             data = {}
             data["call_sid"] = form_data["CallSid"]

config.py

@@ -46,3 +46,5 @@
 TELCO_CODE_ANSWERED = 16
 DEFAULT_LANGUAGE_ID = 1
 MAX_RETRY_ATTEMPTS_FOR_LOGS = os.environ.get("MAX_RETRY_ATTEMPTS_FOR_LOGS", 3)
+
+GOOGLE_APPLICATION_CREDENTIALS = os.environ.get("GOOGLE_APPLICATION_CREDENTIALS")