Skip to content

Commit bc5eb7c

Browse files
committed
[FIX] fix request_parameter
Signed-off-by: Red <redickbutay02@gmail.com>
1 parent 4307696 commit bc5eb7c

File tree

2 files changed

+15
-14
lines changed

2 files changed

+15
-14
lines changed

spp_api/controllers/api.py

Lines changed: 14 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -72,21 +72,21 @@ def wrapper(self, *args, **kwargs):
7272
request_log_val = initial_val.copy()
7373
request_log_val["http_type"] = "request"
7474
request_log_val["request_id"] = request_id
75-
if path.method in ["get"]:
76-
request_log_val["request_parameter"] = kwargs
75+
76+
request_log_val["request_parameter"] = request.httprequest.query_string.decode("utf-8", errors="replace")
77+
78+
# Try to get parsed JSON first
79+
# silent=True prevents Werkzeug from raising a 400 error on bad JSON
80+
json_payload = request.httprequest.get_json(silent=True)
81+
82+
if json_payload:
83+
request_data = json.dumps(json_payload)
7784
else:
78-
# Try to get parsed JSON first
79-
# silent=True prevents Werkzeug from raising a 400 error on bad JSON
80-
json_payload = request.httprequest.get_json(silent=True)
81-
82-
if json_payload:
83-
request_data = json.dumps(json_payload)
84-
else:
85-
# Fallback to raw data if not JSON
86-
# errors='replace' inserts a character instead of crashing on bad bytes
87-
request_data = request.httprequest.get_data().decode("utf-8", errors="replace")
88-
89-
request_log_val["request_data"] = request_data
85+
# Fallback to raw data if not JSON
86+
# errors='replace' inserts a character instead of crashing on bad bytes
87+
request_data = request.httprequest.get_data().decode("utf-8", errors="replace")
88+
89+
request_log_val["request_data"] = request_data
9090

9191
# Sanitize headers
9292
safe_headers = {

spp_api/views/openapi_view.xml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -46,6 +46,7 @@
4646
<field name="create_date" readonly="1" />
4747
<field name="request" readonly="1" />
4848
<field name="headers" readonly="1" />
49+
<field name="request_parameter" readonly="1" />
4950
<field name="request_data" readonly="1" />
5051
<field name="response_data" readonly="1" />
5152
</group>

0 commit comments

Comments
 (0)