audio: rework HTTP timeout behaviour

Something between ESP-IDF v5.1.1 and 5.3.3 changed that causes the 10s
timeout we set in the HTTP_STREAM_POST_REQUEST callback to be ignored
when using HTTPS. Due to this, people with slower WIS cannot use Willow
0.4 anymore. Set the timeout to 10s in HTTP_STREAM_PRE_REQUEST to not
use the ESP-ADF http_stream hardcoded default of 30s.

If WIS is not running, or the user configured a wrong IP or port, the
transport layer detects the problem early anyway.

Author: stintel

main/audio.c

@@ -59,8 +59,7 @@
 #define DEFAULT_WIS_TTS_URL         "https://infer.tovera.io/api/tts"
 #define DEFAULT_WIS_URL             "https://infer.tovera.io/api/willow"
 
-#define HTTP_STREAM_TIMEOUT_MS              2 * 1000
-#define HTTP_STREAM_TIMEOUT_MS_POST_REQUEST 10 * 1000
+#define HTTP_STREAM_TIMEOUT_MS      10 * 1000
 
 #define MULTINET_TWDT   30
 #define STR_WAKE_LEN    25
@@ -225,9 +224,6 @@ static esp_err_t hdl_ev_hs_esp_audio(http_stream_event_msg_t *msg)
             esp_http_client_set_authtype(http, HTTP_AUTH_TYPE_BASIC);
             esp_http_client_set_timeout_ms(http, HTTP_STREAM_TIMEOUT_MS);
             break;
-        case HTTP_STREAM_POST_REQUEST:
-            esp_http_client_set_timeout_ms(http, HTTP_STREAM_TIMEOUT_MS_POST_REQUEST);
-            break;
         default:
             break;
     }
@@ -572,7 +568,6 @@ static esp_err_t hdl_ev_hs_to_api(http_stream_event_msg_t *msg)
 
         case HTTP_STREAM_POST_REQUEST:
             ESP_LOGI(TAG, "WIS HTTP client HTTP_STREAM_POST_REQUEST, write end chunked marker");
-            esp_http_client_set_timeout_ms(http, HTTP_STREAM_TIMEOUT_MS_POST_REQUEST);
             if (esp_http_client_write(http, "0\r\n\r\n", 5) <= 0) {
                 return ESP_FAIL;
             }
@@ -589,7 +584,6 @@ static esp_err_t hdl_ev_hs_to_api(http_stream_event_msg_t *msg)
             if (http_status != 200) {
                 // when ESP HTTP Client terminates connection due to timeout we get -1
                 if (http_status == -1) {
-                    ESP_LOGE(TAG, "WIS response took longer than %dms, connection aborted", HTTP_STREAM_TIMEOUT_MS);
                     ui_pr_err("WIS timeout", "Check server performance");
                 } else if (http_status == 401) {
                     ESP_LOGE(TAG, "WIS returned Unauthorized Access (HTTP 401)");