diff options
author | bunnei <bunneidev@gmail.com> | 2017-08-27 01:02:03 +0200 |
---|---|---|
committer | bunnei <bunneidev@gmail.com> | 2017-08-27 01:45:41 +0200 |
commit | 7698567fc9b9d0b009264d5d8ab5babc3ea197d8 (patch) | |
tree | d263eaca71ee08c3f9b81441b42bcd0e35977510 | |
parent | web_backend: Fix asynchronous JSON post by spawning new thread. (diff) | |
download | yuzu-7698567fc9b9d0b009264d5d8ab5babc3ea197d8.tar yuzu-7698567fc9b9d0b009264d5d8ab5babc3ea197d8.tar.gz yuzu-7698567fc9b9d0b009264d5d8ab5babc3ea197d8.tar.bz2 yuzu-7698567fc9b9d0b009264d5d8ab5babc3ea197d8.tar.lz yuzu-7698567fc9b9d0b009264d5d8ab5babc3ea197d8.tar.xz yuzu-7698567fc9b9d0b009264d5d8ab5babc3ea197d8.tar.zst yuzu-7698567fc9b9d0b009264d5d8ab5babc3ea197d8.zip |
Diffstat (limited to '')
-rw-r--r-- | src/web_service/web_backend.cpp | 42 |
1 files changed, 27 insertions, 15 deletions
diff --git a/src/web_service/web_backend.cpp b/src/web_service/web_backend.cpp index a6070fc0f..d28a3f757 100644 --- a/src/web_service/web_backend.cpp +++ b/src/web_service/web_backend.cpp @@ -2,6 +2,10 @@ // Licensed under GPLv2 or any later version // Refer to the license.txt file included. +#ifdef _WIN32 +#include <winsock.h> +#endif + #include <cstdlib> #include <thread> #include <cpr/cpr.h> @@ -12,18 +16,7 @@ namespace WebService { static constexpr char API_VERSION[]{"1"}; -static void PostJsonAuthenticated(const std::string& url, const std::string& data, - const std::string& username, const std::string& token) { - cpr::Post(cpr::Url{url}, cpr::Body{data}, cpr::Header{{"Content-Type", "application/json"}, - {"x-username", username}, - {"x-token", token}, - {"api-version", API_VERSION}}); -} - -static void PostJsonAnonymous(const std::string& url, const std::string& data) { - cpr::Post(cpr::Url{url}, cpr::Body{data}, - cpr::Header{{"Content-Type", "application/json"}, {"api-version", API_VERSION}}); -} +static std::unique_ptr<cpr::Session> g_session; void PostJson(const std::string& url, const std::string& data, bool allow_anonymous, const std::string& username, const std::string& token) { @@ -38,14 +31,33 @@ void PostJson(const std::string& url, const std::string& data, bool allow_anonym return; } - // Post JSON asynchronously by spawning a new thread +#ifdef _WIN32 + // On Windows, CPR/libcurl does not properly initialize Winsock. The below code is used to + // initialize Winsock globally, which fixes this problem. Without this, only the first CPR + // session will properly be created, and subsequent ones will fail. + WSADATA wsa_data; + const int wsa_result{WSAStartup(MAKEWORD(2, 2), &wsa_data)}; + if (wsa_result) { + LOG_CRITICAL(WebService, "WSAStartup failed: %d", wsa_result); + } +#endif + + // Built request header + cpr::Header header; if (are_credentials_provided) { // Authenticated request if credentials are provided - std::thread{PostJsonAuthenticated, url, data, username, token}.detach(); + header = {{"Content-Type", "application/json"}, + {"x-username", username.c_str()}, + {"x-token", token.c_str()}, + {"api-version", API_VERSION}}; } else { // Otherwise, anonymous request - std::thread{PostJsonAnonymous, url, data}.detach(); + header = cpr::Header{{"Content-Type", "application/json"}, {"api-version", API_VERSION}}; } + + // Post JSON asynchronously + static cpr::AsyncResponse future; + future = cpr::PostAsync(cpr::Url{url.c_str()}, cpr::Body{data.c_str()}, header); } } // namespace WebService |