Fix for espressif#3460 issue

[igolubic]

Fixes: #3460

This code has been run in production for 1 month and it looks stable, no data dropped and it definitely fixes the issue described. I think that this can be merged to avoid using custom package referencing in PlatformIO that has been used in quite a few projects for now.

[Bmooij]

It seems logic to remove the while statement to prevent infinite looping.
I will test this solution the coming days

[bhupiister]

@igolubic If found this as more suitable, as it may take multiple attempts before it gets successful.. i have locked trials to 100

int send_ssl_data(sslclient_context *ssl_client, const uint8_t *data, size_t len){log_v("Writing HTTP request with %d bytes...", len); // for low level debug int ret = -1; int retry = 0; while ((ret = mbedtls_ssl_write(&ssl_client->ssl_ctx, data, len)) <= 0){if ((ret != MBEDTLS_ERR_SSL_WANT_READ && ret != MBEDTLS_ERR_SSL_WANT_WRITE && ret < 0) || retry >= 100){log_v("Trial # %d", retry); // for low level debug // Show number of attempts taken log_v("Handling error %d", ret); // for low level debug return handle_error(ret)} retry++; // wait for space to become available vTaskDelay(2)} log_v("Trial # %d", retry); // for low level debug // Show number of attempts taken return ret}