]> granicus.if.org Git - icinga2/commitdiff
Fix API action status codes
authorNoah Hilverling <noah.hilverling@icinga.com>
Fri, 6 Apr 2018 08:03:09 +0000 (10:03 +0200)
committerNoah Hilverling <noah.hilverling@icinga.com>
Fri, 6 Apr 2018 08:06:15 +0000 (10:06 +0200)
lib/remote/actionshandler.cpp

index 9358f7b32b682711036fb13679eba54d5ad522fa..614ca6f8713c12f7e5b9f82e7745272270ce7523 100644 (file)
@@ -91,15 +91,19 @@ bool ActionsHandler::HandleRequest(const ApiUser::Ptr& user, HttpRequest& reques
                }
        }
 
-       response.SetStatus(500, "No action executed successfully");
+       String statusCode = 500;
+       String statusMessage = "No action executed successfully";
 
        for (const Dictionary::Ptr& res : results) {
-               if (res->Contains("result") && res->Get("result") == 200) {
-                       response.SetStatus(200, "OK");
+               if (res->Contains("code") && res->Get("code") == 200) {
+                       statusCode = 200;
+                       statusMessage = "OK";
                        break;
                }
        }
 
+       response.SetStatus(statusCode, statusMessage);
+
        Dictionary::Ptr result = new Dictionary({
                { "results", new Array(std::move(results)) }
        });