Skip to content

Commit dbec8ad

Browse files
CodeQL warning
1 parent 1ae8c18 commit dbec8ad

File tree

1 file changed

+8
-12
lines changed

1 file changed

+8
-12
lines changed

dbclient/dbclient.py

Lines changed: 8 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -268,21 +268,17 @@ def http_req(self, http_type, endpoint, json_params, version='2.0', print_json=F
268268
message = "Error: {0} request failed with code {1}\n{2}".format(
269269
http_type, http_status_code, raw_results.text
270270
)
271+
error_json = {
272+
'http_status_code': raw_results.status_code,
273+
'error': raw_results.text,
274+
'url': full_endpoint,
275+
'json': json_params,
276+
}
271277
if self.is_skip_failed():
272278
logging.error(message)
273-
return {
274-
'http_status_code': raw_results.status_code,
275-
'error': raw_results.text,
276-
'url': full_endpoint,
277-
'json': json_params,
278-
}
279+
return error_json
279280
else:
280-
logging.error({
281-
'http_status_code': raw_results.status_code,
282-
'error': raw_results.text,
283-
'url': full_endpoint,
284-
'json': json_params,
285-
})
281+
logging.error(error_json)
286282
raise Exception(message)
287283
results = raw_results.json()
288284
if logging_utils.check_error(results):

0 commit comments

Comments
 (0)