Don't set result to success when calling success
res.success is the default, if one async call
sets failure, and another later sets success, we
don't want to overwrite the failure with sucess.
Tested-by: still got success on good patch
Change-Id: Iee76892f7517d508ed98929d9ecd127dd83b1329
Signed-off-by: James Feist <james.feist@linux.intel.com>
diff --git a/redfish-core/src/error_messages.cpp b/redfish-core/src/error_messages.cpp
index 7c29507..5b10937 100644
--- a/redfish-core/src/error_messages.cpp
+++ b/redfish-core/src/error_messages.cpp
@@ -1013,7 +1013,8 @@
*/
void success(crow::Response& res)
{
- res.result(boost::beast::http::status::ok);
+ // don't set res.result here because success is the default and any error
+ // should overwrite the default
addMessageToJsonRoot(
res.jsonValue,
nlohmann::json{