ESP port: Use esp_websocket_client as an alternate implementation

 - Added esp_websocket_client over libwebsockets for ESP-IDF
 - Using this as default for ESP port as it provides better flexibility for ESP-IDF

Author: vikramdattu

esp_port/components/kvs_webrtc/CMakeLists.txt

@@ -1,5 +1,31 @@
 set(KVS_WEBRTC_ROOT "${CMAKE_CURRENT_SOURCE_DIR}/../../../")
 
+# Define source files
+set(KVS_SIGNALING_SOURCES
+    "${KVS_WEBRTC_ROOT}/src/source/Signaling/ChannelInfo.c"
+    "${KVS_WEBRTC_ROOT}/src/source/Signaling/FileCache.c"
+    "${KVS_WEBRTC_ROOT}/src/source/Signaling/Client.c"
+    "${KVS_WEBRTC_ROOT}/src/source/Signaling/StateMachine.c"
+)
+
+# Conditionally add either ESP implementation or original implementation
+if(CONFIG_USE_ESP_WEBSOCKET_CLIENT)
+    set(KVS_SIGNALING_SOURCES
+        ${KVS_SIGNALING_SOURCES}
+        "${CMAKE_CURRENT_SOURCE_DIR}/esp_signaling/SignalingESP.c"
+        "${CMAKE_CURRENT_SOURCE_DIR}/esp_signaling/LwsApiCallsESP.c"
+        "${CMAKE_CURRENT_SOURCE_DIR}/src/DataBuffer.c"
+    )
+    message(STATUS "Using ESP WebSocket client for signaling")
+else()
+    set(KVS_SIGNALING_SOURCES
+        ${KVS_SIGNALING_SOURCES}
+        "${KVS_WEBRTC_ROOT}/src/source/Signaling/Signaling.c"
+        "${KVS_WEBRTC_ROOT}/src/source/Signaling/LwsApiCalls.c"
+    )
+    message(STATUS "Using libwebsockets for signaling")
+endif()
+
 idf_component_register(
     SRCS "${KVS_WEBRTC_ROOT}/src/source/Crypto/Crypto.c"
         "${KVS_WEBRTC_ROOT}/src/source/Crypto/Dtls_mbedtls.c"
@@ -37,15 +63,10 @@ idf_component_register(
         "${KVS_WEBRTC_ROOT}/src/source/Sctp/Sctp.c"
         "${KVS_WEBRTC_ROOT}/src/source/Sdp/Deserialize.c"
         "${KVS_WEBRTC_ROOT}/src/source/Sdp/Serialize.c"
-        "${KVS_WEBRTC_ROOT}/src/source/Signaling/ChannelInfo.c"
-        "${KVS_WEBRTC_ROOT}/src/source/Signaling/Client.c"
-        "${KVS_WEBRTC_ROOT}/src/source/Signaling/FileCache.c"
-        "${KVS_WEBRTC_ROOT}/src/source/Signaling/LwsApiCalls.c"
-        "${KVS_WEBRTC_ROOT}/src/source/Signaling/Signaling.c"
-        "${KVS_WEBRTC_ROOT}/src/source/Signaling/StateMachine.c"
         "${KVS_WEBRTC_ROOT}/src/source/Srtp/SrtpSession.c"
         "${KVS_WEBRTC_ROOT}/src/source/Stun/Stun.c"
         "${KVS_WEBRTC_ROOT}/src/source/Threadpool/ThreadPoolContext.c"
+        ${KVS_SIGNALING_SOURCES}
 
     INCLUDE_DIRS "${KVS_WEBRTC_ROOT}/src/include"
         "${KVS_WEBRTC_ROOT}/src/source/Crypto"
@@ -60,7 +81,7 @@ idf_component_register(
         "${KVS_WEBRTC_ROOT}/src/source/Sctp"
         "${KVS_WEBRTC_ROOT}/src/source/Stun"
         "${KVS_WEBRTC_ROOT}/src/source/Threadpool"
-        # "${CMAKE_CURRENT_SOURCE_DIR}/include"
+        "${CMAKE_CURRENT_SOURCE_DIR}/include"
 
     PRIV_INCLUDE_DIRS "${KVS_WEBRTC_ROOT}/src/source"
 
@@ -72,10 +93,15 @@ idf_component_register(
              "freertos"
              "esp_system"
              "heap"
-             "driver" # libwebsockets.h has this include
-             "spi_flash" # libwebsockets.h has this include
+             "driver"
+             "spi_flash"
+             "esp_http_client"
 )
 
+if(CONFIG_USE_ESP_WEBSOCKET_CLIENT)
+    target_compile_definitions(${COMPONENT_LIB} PUBLIC -DUSE_ESP_WEBSOCKET -DESP_PLATFORM)
+endif()
+
 option(ENABLE_KVS_THREADPOOL "Enable support for KVS thread pool in signaling" ON)
 set(KVS_PLAT_ESP_FREERTOS ON CACHE BOOL "Build for ESP FreeRTOS")
 add_definitions(-DKVS_PLAT_ESP_FREERTOS)

esp_port/components/kvs_webrtc/Kconfig.projbuild

@@ -1,10 +1,18 @@
 menu "KVS WebRTC Configuration"
+
     config ENABLE_DATA_CHANNEL
         bool "Enable Data channel support using usrsctp"
         default true
         help
             Enables WebRTC Data channel support
 
+    config USE_ESP_WEBSOCKET_CLIENT
+        bool "Use esp_websocket_client instead of libwebscoekts"
+        default true
+        help
+            Kinesis Video Stream SDK uses libwebsockets for the signaling API requests.
+            By using esp_websocket_client, you get better control over memory usage as the complete signaling part is re-implemented with ESP-IDF constraints in mind
+
     config PREFER_DYNAMIC_ALLOCS
         bool "Prefer dynamic memory allocations"
         default false
@@ -25,4 +33,4 @@ menu "KVS WebRTC Configuration"
         range 1 10
         help
             Maximum number of concurrent WebRTC streams.
-endmenu
+endmenu