kopia lustrzana https://github.com/espressif/esp-idf
esp_https_ota: Add check for 303 and 308 status code
Closes https://github.com/espressif/esp-idf/issues/8581pull/10827/head
rodzic
983547290b
commit
342f566002
|
@ -45,12 +45,29 @@ struct esp_https_ota_handle {
|
|||
|
||||
typedef struct esp_https_ota_handle esp_https_ota_t;
|
||||
|
||||
static bool redirection_required(int status_code)
|
||||
{
|
||||
switch (status_code) {
|
||||
case HttpStatus_MovedPermanently:
|
||||
case HttpStatus_Found:
|
||||
case HttpStatus_SeeOther:
|
||||
case HttpStatus_TemporaryRedirect:
|
||||
case HttpStatus_PermanentRedirect:
|
||||
return true;
|
||||
default:
|
||||
return false;
|
||||
}
|
||||
return false;
|
||||
}
|
||||
|
||||
static bool process_again(int status_code)
|
||||
{
|
||||
switch (status_code) {
|
||||
case HttpStatus_MovedPermanently:
|
||||
case HttpStatus_Found:
|
||||
case HttpStatus_SeeOther:
|
||||
case HttpStatus_TemporaryRedirect:
|
||||
case HttpStatus_PermanentRedirect:
|
||||
case HttpStatus_Unauthorized:
|
||||
return true;
|
||||
default:
|
||||
|
@ -62,7 +79,7 @@ static bool process_again(int status_code)
|
|||
static esp_err_t _http_handle_response_code(esp_http_client_handle_t http_client, int status_code)
|
||||
{
|
||||
esp_err_t err;
|
||||
if (status_code == HttpStatus_MovedPermanently || status_code == HttpStatus_Found || status_code == HttpStatus_TemporaryRedirect) {
|
||||
if (redirection_required(status_code)) {
|
||||
err = esp_http_client_set_redirection(http_client);
|
||||
if (err != ESP_OK) {
|
||||
ESP_LOGE(TAG, "URL redirection Failed");
|
||||
|
|
Ładowanie…
Reference in New Issue