diff options
author | Jason M. Bills <jason.m.bills@linux.intel.com> | 2019-11-27 03:29:34 +0300 |
---|---|---|
committer | Jason Bills <jason.m.bills@linux.intel.com> | 2019-12-02 19:53:55 +0300 |
commit | e195977836ef5135d9362f40e8755da8c34dc0d7 (patch) | |
tree | e098d05a90fd72df4d0086fcc7264ed2afd08999 /redfish-core | |
parent | d819a420c04b4528c03ba749978338953868fff1 (diff) | |
download | bmcweb-e195977836ef5135d9362f40e8755da8c34dc0d7.tar.xz |
Add "Retry-After" header for temporarily unavailable messages
Whenever the Redfish response is that a service is temporarily
unavailable, the "Retry-After" header is added with the same
value, so just set the header automatically with the response.
Tested:
Confirmed that the "Retry-After" header is set correctly with
the Redfish temporarily unavailable message.
Change-Id: I9c940be94d9d284b9633c5caa2ce71ade76d22d5
Signed-off-by: Jason M. Bills <jason.m.bills@linux.intel.com>
Diffstat (limited to 'redfish-core')
-rw-r--r-- | redfish-core/lib/certificate_service.hpp | 1 | ||||
-rw-r--r-- | redfish-core/lib/log_services.hpp | 1 | ||||
-rw-r--r-- | redfish-core/lib/update_service.hpp | 1 | ||||
-rw-r--r-- | redfish-core/src/error_messages.cpp | 1 |
4 files changed, 1 insertions, 3 deletions
diff --git a/redfish-core/lib/certificate_service.hpp b/redfish-core/lib/certificate_service.hpp index f82363bc3a..e15c308ee3 100644 --- a/redfish-core/lib/certificate_service.hpp +++ b/redfish-core/lib/certificate_service.hpp @@ -421,7 +421,6 @@ class CertificateActionGenerateCSR : public Node static const int TIME_OUT = 10; if (csrMatcher) { - res.addHeader("Retry-After", std::to_string(TIME_OUT)); messages::serviceTemporarilyUnavailable(asyncResp->res, std::to_string(TIME_OUT)); return; diff --git a/redfish-core/lib/log_services.hpp b/redfish-core/lib/log_services.hpp index 2eb00b6832..0f4a90226f 100644 --- a/redfish-core/lib/log_services.hpp +++ b/redfish-core/lib/log_services.hpp @@ -1875,7 +1875,6 @@ class OnDemandCrashdump : public Node // Only allow one OnDemand Log request at a time if (onDemandLogMatcher != nullptr) { - asyncResp->res.addHeader("Retry-After", "30"); messages::serviceTemporarilyUnavailable(asyncResp->res, "30"); return; } diff --git a/redfish-core/lib/update_service.hpp b/redfish-core/lib/update_service.hpp index da88271985..fde05ab4e8 100644 --- a/redfish-core/lib/update_service.hpp +++ b/redfish-core/lib/update_service.hpp @@ -141,7 +141,6 @@ static void monitorForSoftwareAvailable(std::shared_ptr<AsyncResp> asyncResp, { if (asyncResp) { - asyncResp->res.addHeader("Retry-After", "30"); messages::serviceTemporarilyUnavailable(asyncResp->res, "30"); } return; diff --git a/redfish-core/src/error_messages.cpp b/redfish-core/src/error_messages.cpp index b9d72ea15d..bc4cba6eec 100644 --- a/redfish-core/src/error_messages.cpp +++ b/redfish-core/src/error_messages.cpp @@ -348,6 +348,7 @@ void propertyDuplicate(crow::Response& res, const std::string& arg1) */ void serviceTemporarilyUnavailable(crow::Response& res, const std::string& arg1) { + res.addHeader("Retry-After", arg1); res.result(boost::beast::http::status::service_unavailable); addMessageToErrorJson( res.jsonValue, |