Skip to content
Browse files

Merge pull request #8210 from spycrab/httpreq_error_msg

Common/HttpRequest: Use CURLOPT_ERRORBUFFER for error messages
  • Loading branch information...
Helios747 committed Jun 20, 2019
2 parents f08aa2d + ba4c1c5 commit 84b9b37fef4050897702e449047c8ea9699defbe
Showing with 6 additions and 1 deletion.
  1. +6 −1 Source/Core/Common/HttpRequest.cpp
@@ -42,6 +42,7 @@ class HttpRequest::Impl final
static inline std::once_flag s_curl_was_initialized;
ProgressCallback m_callback;
std::unique_ptr<CURL, decltype(&curl_easy_cleanup)> m_curl{nullptr, curl_easy_cleanup};
std::string m_error_string;

HttpRequest::HttpRequest(std::chrono::milliseconds timeout_ms, ProgressCallback callback)
@@ -119,6 +120,10 @@ HttpRequest::Impl::Impl(std::chrono::milliseconds timeout_ms, ProgressCallback c
curl_easy_setopt(m_curl.get(), CURLOPT_PROGRESSFUNCTION, CurlProgressCallback);

// Set up error buffer
curl_easy_setopt(m_curl.get(), CURLOPT_ERRORBUFFER,;

// libcurl may not have been built with async DNS support, so we disable
// signal handlers to avoid a possible and likely crash if a resolve times out.
curl_easy_setopt(m_curl.get(), CURLOPT_NOSIGNAL, true);
@@ -205,7 +210,7 @@ HttpRequest::Response HttpRequest::Impl::Fetch(const std::string& url, Method me
const CURLcode res = curl_easy_perform(m_curl.get());
if (res != CURLE_OK)
ERROR_LOG(COMMON, "Failed to %s %s: %s", type, url.c_str(), curl_easy_strerror(res));
ERROR_LOG(COMMON, "Failed to %s %s: %s", type, url.c_str(), m_error_string.c_str());
return {};

0 comments on commit 84b9b37

Please sign in to comment.
You can’t perform that action at this time.