feedc0de/espasynchttpreq
1
0
Fork 0
Code Issues Pull Requests 2 Actions Packages Projects Releases Wiki Activity

Hopefully finally fixed close errors

Browse Source
This commit is contained in:
0xFEEDC0DE64
2022-01-11 13:59:47 +01:00
parent 50487cf564
commit 650f5ed465
1 changed files with 6 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

14
src/asynchttprequest.cpp
View File

@@ -212,9 +212,8 @@ tl::expected<void, std::string> AsyncHttpRequest::start(std::string_view url,
if (m_client) if (m_client)
{ {
constexpr auto msg = "http client not null"; ESP_LOGW(TAG, "old http client still constructed, destructing now");
ESP_LOGW(TAG, "%s", msg); m_client = {};
return tl::make_unexpected(msg);
} }
if (auto result = createClient(url, method, timeout_ms); !result) if (auto result = createClient(url, method, timeout_ms); !result)
@@ -523,10 +522,9 @@ void AsyncHttpRequest::requestTask()
m_statusCode = m_client.get_status_code(); m_statusCode = m_client.get_status_code();
} }
// workaround for esp-idf bug, every request after the first one fails with ESP_ERR_HTTP_FETCH_HEADER {
const auto result = m_client.close(); const auto result = m_client.close();
ESP_LOG_LEVEL_LOCAL((result == ESP_OK ? ESP_LOG_VERBOSE : ESP_LOG_ERROR), TAG, "m_client.close() returned: %s", esp_err_to_name(result)); ESP_LOGD(TAG, "m_client.close() returned: %s", esp_err_to_name(result));
if (result == ESP_OK) }
m_client = {};
} }
} }