adafruit-mirror/zephyr
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 9

mgmt: hawkbit: check http status code in response

Browse source 
check http status code in response

Signed-off-by: Fin Maaß <f.maass@vogl-electronic.com>
This commit is contained in:
Fin Maaß 2024-03-19 10:52:30 +01:00 committed by Fabio Baltieri
parent 976da5da5c
commit b61d6ce46e
1 changed files with 12 additions and 11 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

23
subsys/mgmt/hawkbit/hawkbit.c
View file

@ -645,6 +645,16 @@ static void response_cb(struct http_response *rsp, enum http_final_call final_da
type = enum_for_http_req_string(userdata);
if (rsp->http_status_code != 200) {
LOG_ERR("HTTP request denied (%s): %d", (char *)userdata, rsp->http_status_code);
if (rsp->http_status_code == 401 || rsp->http_status_code == 403) {
hb_context.code_status = HAWKBIT_PERMISSION_ERROR;
} else {
hb_context.code_status = HAWKBIT_METADATA_ERROR;
}
return;
}
switch (type) {
case HAWKBIT_PROBE:
if (hb_context.dl.http_content_size == 0) {
@ -680,12 +690,6 @@ static void response_cb(struct http_response *rsp, enum http_final_call final_da
break;
}
if (rsp->http_status_code / 100 == 4) {
LOG_ERR("HTTP request denied: %d", rsp->http_status_code);
hb_context.code_status = HAWKBIT_PERMISSION_ERROR;
break;
}
hb_context.response_data[hb_context.dl.downloaded_size] = '\0';
ret = json_obj_parse(hb_context.response_data,
hb_context.dl.downloaded_size, json_ctl_res_descr,
@ -701,10 +705,6 @@ static void response_cb(struct http_response *rsp, enum http_final_call final_da
case HAWKBIT_CLOSE:
case HAWKBIT_REPORT:
case HAWKBIT_CONFIG_DEVICE:
if (strcmp(rsp->http_status, "OK") < 0) {
LOG_ERR("Failed to cancel the update");
}
break;
case HAWKBIT_PROBE_DEPLOYMENT_BASE:
@ -1025,7 +1025,8 @@ enum hawkbit_response hawkbit_probe(void)
goto cleanup;
}
if (hb_context.code_status == HAWKBIT_METADATA_ERROR) {
if (hb_context.code_status == HAWKBIT_METADATA_ERROR ||
hb_context.code_status == HAWKBIT_PERMISSION_ERROR) {
goto cleanup;
}