From dd349516cb6ec5168baf537b607c3fb3a14d3d62 Mon Sep 17 00:00:00 2001
From: bromnhub <241785706+bromnhub@users.noreply.github.com>
Date: Mon, 15 Dec 2025 05:04:47 -0500
Subject: [PATCH] refactor: Use requests.raise_for_status for better HTTP error
 handling in create_web_call

---
 vapi_python/vapi_python.py | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/vapi_python/vapi_python.py b/vapi_python/vapi_python.py
index 69e9797..d6122b0 100644
--- a/vapi_python/vapi_python.py
+++ b/vapi_python/vapi_python.py
@@ -13,13 +13,17 @@ def create_web_call(api_url, api_key, payload):
         'Content-Type': 'application/json'
     }
     response = requests.post(url, headers=headers, json=payload)
+    response.raise_for_status()
     data = response.json()
     if response.status_code == 201:
         call_id = data.get('id')
         web_call_url = data.get('webCallUrl')
         return call_id, web_call_url
     else:
-        raise Exception(f"Error: {data['message']}")
+        # This else block is now redundant due to response.raise_for_status()
+        # but kept for safety if status code is 201 but data is missing.
+        # The raise_for_status() will handle non-2xx errors.
+        pass
 
 
 class Vapi: