fix(websocket): skip esp_transport_{tcp|ssl}_set_xxxxx() calls for external transport

Author: huming2207

components/esp_websocket_client/esp_websocket_client.c

@@ -501,6 +501,13 @@ static esp_err_t esp_websocket_client_create_transport(esp_websocket_client_hand
             tcp = esp_transport_tcp_init();
         } else {
             tcp = client->config->ext_transport;
+
+            esp_transport_handle_t ws = esp_transport_ws_init(tcp);
+            ESP_WS_CLIENT_MEM_CHECK(TAG, ws, return ESP_ERR_NO_MEM);
+            esp_transport_set_default_port(ws, WEBSOCKET_TCP_DEFAULT_PORT);
+            esp_transport_list_add(client->transport_list, ws, WS_OVER_TCP_SCHEME);
+            ESP_WS_CLIENT_ERR_OK_CHECK(TAG, set_websocket_transport_optional_settings(client, WS_OVER_TCP_SCHEME), return ESP_FAIL;)
+            return ESP_OK;
         }
 
         ESP_WS_CLIENT_MEM_CHECK(TAG, tcp, return ESP_ERR_NO_MEM);
@@ -526,6 +533,13 @@ static esp_err_t esp_websocket_client_create_transport(esp_websocket_client_hand
             ssl = esp_transport_ssl_init();
         } else {
             ssl = client->config->ext_transport;
+
+            esp_transport_handle_t wss = esp_transport_ws_init(ssl);
+            ESP_WS_CLIENT_MEM_CHECK(TAG, wss, return ESP_ERR_NO_MEM);
+            esp_transport_set_default_port(wss, WEBSOCKET_SSL_DEFAULT_PORT);
+            esp_transport_list_add(client->transport_list, wss, WS_OVER_TLS_SCHEME);
+            ESP_WS_CLIENT_ERR_OK_CHECK(TAG, set_websocket_transport_optional_settings(client, WS_OVER_TLS_SCHEME), return ESP_FAIL;)
+            return ESP_OK;
         }
 
         ESP_WS_CLIENT_MEM_CHECK(TAG, ssl, return ESP_ERR_NO_MEM);

components/esp_websocket_client/include/esp_websocket_client.h

@@ -133,7 +133,7 @@ typedef struct {
     int                         network_timeout_ms;         /*!< Abort network operation if it is not completed after this value, in milliseconds (defaults to 10s) */
     size_t                      ping_interval_sec;          /*!< Websocket ping interval, defaults to 10 seconds if not set */
     struct ifreq                *if_name;                   /*!< The name of interface for data to go through. Use the default interface without setting */
-    esp_transport_handle_t      ext_transport;              /*!< External WebSocket tcp_transport handle to the client; or if null, the client will create its own transport handle. */
+    esp_transport_handle_t      ext_transport;              /*!< External WebSocket tcp_transport handle to the client; or if null, the client will create its own transport handle. When external transport is used, socket options like if_name and keep_alive settings will be ignored, as it should be done when setting up the external transport. */
 } esp_websocket_client_config_t;
 
 /**