Skip to content

Commit 1012cc5

Browse files
committed
[update] add check for null body
1 parent f735e84 commit 1012cc5

File tree

2 files changed

+15
-17
lines changed

2 files changed

+15
-17
lines changed
Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
__version__ = '1.0.1'
1+
__version__ = '1.0.2'

src/django_simple_api_proxy/views.py

Lines changed: 14 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -52,6 +52,16 @@ def update_payload(self, request, params):
5252
)
5353
return params
5454

55+
def preprocess_request_body(self, request):
56+
if request.body:
57+
params = json.loads(request.body)
58+
else:
59+
params = {}
60+
logger.debug(f"JSON Params: {params}")
61+
path = self.get_proxy_path(request)
62+
params = self.update_payload(request, params)
63+
return path, params
64+
5565
def send_request(
5666
self, method, url, params=None, data=None, json=None, timeout=180, verify=True
5767
):
@@ -85,10 +95,7 @@ def post(self, request, *args, **kwargs):
8595
logger.debug("----- Proxy POST")
8696
response = {"status": "error"}
8797
with logger.catch():
88-
params = json.loads(request.body)
89-
logger.debug(f"JSON Params: {params}")
90-
path = self.get_proxy_path(request)
91-
params = self.update_payload(request, params)
98+
path, params = self.preprocess_request_body(request)
9299
middle_resp_ = self.send_request("POST", path, json=params)
93100
response = middle_resp_.json()
94101
return self.response(response)
@@ -98,10 +105,7 @@ def patch(self, request, *args, **kwargs):
98105
logger.debug("----- Proxy PATCH")
99106
response = {"status": "error"}
100107
with logger.catch():
101-
params = json.loads(request.body)
102-
logger.debug(f"JSON Params: {params}")
103-
path = self.get_proxy_path(request)
104-
params = self.update_payload(request, params)
108+
path, params = self.preprocess_request_body(request)
105109
middle_resp_ = self.send_request("PATCH", path, json=params)
106110
response = middle_resp_.json()
107111
return self.response(response)
@@ -111,10 +115,7 @@ def delete(self, request, *args, **kwargs):
111115
logger.debug("----- Proxy DELETE")
112116
response = {"status": "error"}
113117
with logger.catch():
114-
params = json.loads(request.body)
115-
logger.debug(f"JSON Params: {params}")
116-
path = self.get_proxy_path(request)
117-
params = self.update_payload(request, params)
118+
path, params = self.preprocess_request_body(request)
118119
middle_resp_ = self.send_request("DELETE", path, json=params)
119120
response = middle_resp_.json()
120121
return self.response(response)
@@ -124,10 +125,7 @@ def put(self, request, *args, **kwargs):
124125
logger.debug("----- Proxy PUT")
125126
response = {"status": "error"}
126127
with logger.catch():
127-
params = json.loads(request.body)
128-
logger.debug(f"JSON Params: {params}")
129-
path = self.get_proxy_path(request)
130-
params = self.update_payload(request, params)
128+
path, params = self.preprocess_request_body(request)
131129
middle_resp_ = self.send_request("PUT", path, json=params)
132130
response = middle_resp_.json()
133131
return self.response(response)

0 commit comments

Comments
 (0)