Skip to content

Commit 8ec2220

Browse files
Merge branch 'bugfix/ota_failed_with_openssl_3.0.2_v5.0' into 'release/v5.0'
fix(esp_http_client): Fix ota failure with openssl server (v5.0) See merge request espressif/esp-idf!35069
2 parents 48ff843 + 7abf64f commit 8ec2220

File tree

1 file changed

+8
-8
lines changed

1 file changed

+8
-8
lines changed

components/esp_http_client/esp_http_client.c

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1114,15 +1114,15 @@ int esp_http_client_read(esp_http_client_handle_t client, char *buffer, int len)
11141114
ESP_LOGD(TAG, "need_read=%d, byte_to_read=%d, rlen=%d, ridx=%d", need_read, byte_to_read, rlen, ridx);
11151115

11161116
if (rlen <= 0) {
1117+
esp_log_level_t sev = ESP_LOG_WARN;
1118+
/* Check for cleanly closed connection */
1119+
if (rlen == ERR_TCP_TRANSPORT_CONNECTION_CLOSED_BY_FIN && client->response->is_chunked) {
1120+
/* Explicit call to parser for invoking `message_complete` callback */
1121+
http_parser_execute(client->parser, client->parser_settings, res_buffer->data, 0);
1122+
/* ...and lowering the message severity, as closed connection from server side is expected in chunked transport */
1123+
sev = ESP_LOG_DEBUG;
1124+
}
11171125
if (errno != 0) {
1118-
esp_log_level_t sev = ESP_LOG_WARN;
1119-
/* Check for cleanly closed connection */
1120-
if (rlen == ERR_TCP_TRANSPORT_CONNECTION_CLOSED_BY_FIN && client->response->is_chunked) {
1121-
/* Explicit call to parser for invoking `message_complete` callback */
1122-
http_parser_execute(client->parser, client->parser_settings, res_buffer->data, 0);
1123-
/* ...and lowering the message severity, as closed connection from server side is expected in chunked transport */
1124-
sev = ESP_LOG_DEBUG;
1125-
}
11261126
ESP_LOG_LEVEL(sev, TAG, "esp_transport_read returned:%d and errno:%d ", rlen, errno);
11271127
}
11281128

0 commit comments

Comments
 (0)