Continue with the peer connection even if getIceServerConfig failed

 - TURN servers are not `MUST` for the peer connection
 - The device can still continue with the STUN server
 - In cases where, server is configured to not allow IceServers, we can continue

Author: vikramdattu

esp_port/components/kvs_webrtc/src/LwsApiCallsESP.c

@@ -1581,6 +1581,16 @@ STATUS getIceConfigEsp(PSignalingClient pSignalingClient, UINT64 time)
     retStatus = performEspHttpRequest(pSignalingClient, url, HTTP_METHOD_POST,
                                      paramsJson, &pResponseStr, &resultLen);
 
+
+    // It should be okay if this failed
+    if (STATUS_FAILED(retStatus)) {
+        ESP_LOGW(TAG, "Failed to get ice config, proceeding anyway...");
+        // Store successful result
+        ATOMIC_STORE(&pSignalingClient->result, (SIZE_T) SERVICE_CALL_RESULT_OK);
+        retStatus = STATUS_SUCCESS;
+        goto CleanUp;
+    }
+
     // Set the service call result
     ATOMIC_STORE(&pSignalingClient->result, (SIZE_T) retStatus);
 
@@ -2117,6 +2127,9 @@ STATUS performEspHttpRequest(PSignalingClient pSignalingClient, PCHAR url,
     int statusCode = esp_http_client_get_status_code(client);
     if (statusCode < 200 || statusCode >= 300) {
         ESP_LOGE(TAG, "HTTP request failed with status code %d", statusCode);
+        if (response != NULL && responseLen > 0) {
+            ESP_LOGE(TAG, "Response data: %.*s", (int) responseLen, response);
+        }
         CHK(FALSE, STATUS_INTERNAL_ERROR);
     }
 

esp_port/components/kvs_webrtc/src/app_webrtc.c

@@ -732,12 +732,22 @@ STATUS initializePeerConnection(PSampleConfiguration pSampleConfiguration, PRtcP
 
     if (pSampleConfiguration->useTurn && !isStreamingOnly) {
         // Set the URIs from the configuration
-        CHK_STATUS(signalingClientGetIceConfigInfoCount(pSampleConfiguration->signalingClientHandle, &iceConfigCount));
+        retStatus = signalingClientGetIceConfigInfoCount(pSampleConfiguration->signalingClientHandle, &iceConfigCount);
+        if (STATUS_FAILED(retStatus)) {
+            ESP_LOGW(TAG, "Failed to get ice config count, proceeding anyway...");
+            retStatus = STATUS_SUCCESS;
+            iceConfigCount = 0;
+        }
 
         /* signalingClientGetIceConfigInfoCount can return more than one turn server. Use only one to optimize
          * candidate gathering latency. But user can also choose to use more than 1 turn server. */
-        for (uriCount = 0, i = 0; i < maxTurnServer; i++) {
-            CHK_STATUS(signalingClientGetIceConfigInfo(pSampleConfiguration->signalingClientHandle, i, &pIceConfigInfo));
+        for (uriCount = 0, i = 0; i < maxTurnServer && i < iceConfigCount; i++) {
+            retStatus = signalingClientGetIceConfigInfo(pSampleConfiguration->signalingClientHandle, i, &pIceConfigInfo);
+            if (STATUS_FAILED(retStatus)) {
+                ESP_LOGW(TAG, "Failed to get ice config, proceeding anyway...");
+                retStatus = STATUS_SUCCESS;
+                break;
+            }
             for (j = 0; j < pIceConfigInfo->uriCount; j++) {
                 CHECK(uriCount < MAX_ICE_SERVERS_COUNT);
                 /*