fix(make_request): don't blindly try to check the content-type

Ensure that the response has one first

Signed-off-by: Akhil Narang <me@akhilnarang.dev>
(cherry picked from commit 152c4d4)

frappe/integrations/utils.py

@@ -21,15 +21,15 @@ def make_request(method, url, auth=None, headers=None, data=None, json=None, par
 		)
 		response.raise_for_status()
 
-		content_type = response.headers.get("content-type")
-		if content_type == "text/plain; charset=utf-8":
-			return parse_qs(response.text)
-		elif content_type.startswith("application/") and content_type.split(";")[0].endswith("json"):
-			return response.json()
-		elif response.text:
-			return response.text
-		else:
-			return
+		# Check whether the response has a content-type, before trying to check what it is
+		if content_type := response.headers.get("content-type"):
+			if content_type == "text/plain; charset=utf-8":
+				return parse_qs(response.text)
+			elif content_type.startswith("application/") and content_type.split(";")[0].endswith("json"):
+				return response.json()
+			elif response.text:
+				return response.text
+		return
 	except Exception as exc:
 		frappe.log_error()
 		raise exc