add 'data' argument to http error callback

res/scripts/classes.lua

@@ -273,7 +273,7 @@ network.__process_events = function()
                 _http_response_callbacks[event[3]] = nil
                 _http_error_callbacks[event[3]] = nil
                 if callback then
-                    callback(event[2])
+                    callback(event[2], event[4])
                 end
             end
         end

src/logic/scripting/lua/libs/libnetwork.cpp

@@ -88,27 +88,24 @@ static int perform_get(lua::State* L, network::Network& network, bool binary) {
 
     int currentRequestId = request_id++;
 
-    network.get(url, [currentRequestId, binary](std::vector<char> bytes) {
-        push_event(NetworkEvent(
-            RESPONSE,
-            ResponseEventDto {
-                200,
-                binary,
-                currentRequestId,
-                std::move(bytes)
-            }
-        ));
-    }, [currentRequestId](int code) {
-        push_event(NetworkEvent(
-            RESPONSE,
-            ResponseEventDto {
-                code,
-                false,
-                currentRequestId,
-                {}
-            }
-        ));
-    }, std::move(headers));
+    network.get(
+        url,
+        [currentRequestId, binary](std::vector<char> bytes) {
+            push_event(NetworkEvent(
+                RESPONSE,
+                ResponseEventDto {
+                    200, binary, currentRequestId, std::move(bytes)}
+            ));
+        },
+        [currentRequestId, binary](int code, std::vector<char> bytes) {
+            push_event(NetworkEvent(
+                RESPONSE,
+                ResponseEventDto {
+                    code, binary, currentRequestId, std::move(bytes)}
+            ));
+        },
+        std::move(headers)
+    );
     return lua::pushinteger(L, currentRequestId);
 }
 
@@ -138,21 +135,17 @@ static int l_post(lua::State* L, network::Network& network) {
         url,
         string,
         [currentRequestId](std::vector<char> bytes) {
-            auto buffer = std::make_shared<util::Buffer<ubyte>>(
-                reinterpret_cast<const ubyte*>(bytes.data()), bytes.size()
-            );
             push_event(NetworkEvent(
                 RESPONSE,
                 ResponseEventDto {
-                    200,
-                    false,
-                    currentRequestId,
-                    std::vector<char>(buffer->begin(), buffer->end())}
+                    200, false, currentRequestId, std::move(bytes)}
             ));
         },
-        [currentRequestId](int code) {
+        [currentRequestId](int code, std::vector<char> bytes) {
             push_event(NetworkEvent(
-                RESPONSE, ResponseEventDto {code, false, currentRequestId, {}}
+                RESPONSE,
+                ResponseEventDto {
+                    code, false, currentRequestId, std::move(bytes)}
             ));
         },
         std::move(headers)

src/network/Network.cpp

@@ -173,7 +173,7 @@ public:
             auto message = curl_multi_strerror(res);
             logger.error() << message << " (" << url << ")";
             if (onReject) {
-                onReject(HTTP_BAD_GATEWAY);
+                onReject(HTTP_BAD_GATEWAY, {});
             }
             url = "";
         }
@@ -188,7 +188,7 @@ public:
             auto message = curl_multi_strerror(res);
             logger.error() << message << " (" << url << ")";
             if (onReject) {
-                onReject(HTTP_BAD_GATEWAY);
+                onReject(HTTP_BAD_GATEWAY, {});
             }
             curl_multi_remove_handle(multiHandle, curl);
             url = "";
@@ -213,9 +213,14 @@ public:
                     onResponse(std::move(buffer));
                 }
             } else {
-                logger.error() << "response code " << response << " (" << url << ")";
+                logger.error()
+                    << "response code " << response << " (" << url << ")"
+                    << (buffer.empty()
+                            ? ""
+                            : std::to_string(buffer.size()) + " byte(s)");
+                totalDownload += buffer.size();
                 if (onReject) {
-                    onReject(response);
+                    onReject(response, std::move(buffer));
                 }
             }
             url = "";

src/network/Network.hpp

@@ -11,7 +11,7 @@
 
 namespace network {
     using OnResponse = std::function<void(std::vector<char>)>;
-    using OnReject = std::function<void(int)>;
+    using OnReject = std::function<void(int, std::vector<char>)>;
     using ConnectCallback = std::function<void(u64id_t, u64id_t)>;
     using ServerDatagramCallback = std::function<void(u64id_t sid, const std::string& addr, int port, const char* buffer, size_t length)>;
     using ClientDatagramCallback = std::function<void(u64id_t cid, const char* buffer, size_t length)>;

test/network/curltest.cpp

@@ -16,7 +16,7 @@ TEST(curltest, curltest) {
             auto view = std::string_view(data.data(), data.size());
             auto value = json::parse(view);
             std::cout << value << std::endl;
-        }, [](auto){}
+        }, [](auto, auto){}
     );
     std::cout << "upload: " << network->getTotalUpload() << " B" << std::endl;
     std::cout << "download: " << network->getTotalDownload() << " B" << std::endl;