diff options
author | James Feist <james.feist@linux.intel.com> | 2019-02-21 21:57:24 +0300 |
---|---|---|
committer | Ed Tanous <ed.tanous@intel.com> | 2019-03-01 23:47:48 +0300 |
commit | be12c7291914b8a6dd7d6e78f1a500708b0f20f9 (patch) | |
tree | ed0b2b2d4e6bc9fc835068a3ff653497e9b531ed /redfish-core/src/error_messages.cpp | |
parent | c33a90ec049bc82054106bd4e09381797df72002 (diff) | |
download | bmcweb-be12c7291914b8a6dd7d6e78f1a500708b0f20f9.tar.xz |
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>
Diffstat (limited to 'redfish-core/src/error_messages.cpp')
-rw-r--r-- | redfish-core/src/error_messages.cpp | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/redfish-core/src/error_messages.cpp b/redfish-core/src/error_messages.cpp index 7c2950782a..5b10937609 100644 --- a/redfish-core/src/error_messages.cpp +++ b/redfish-core/src/error_messages.cpp @@ -1013,7 +1013,8 @@ void generalError(crow::Response& res) */ 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{ |