diff options
author | Nan Zhou <nanzhoumails@gmail.com> | 2022-01-28 04:06:51 +0300 |
---|---|---|
committer | Ed Tanous <ed@tanous.net> | 2022-03-01 02:30:04 +0300 |
commit | 72374eb7fe42257e866dd088bc13520b0b28cffa (patch) | |
tree | e3245cf0282b19fc7d2e50300e7dfc41229b6d62 /redfish-core/lib/sensors.hpp | |
parent | a94ac61f69fd3768cf609adc75cfd646c558eb7e (diff) | |
download | bmcweb-72374eb7fe42257e866dd088bc13520b0b28cffa.tar.xz |
Change the completionhandler to accept Res
These modifications are from WIP:Redfish:Query parameters:Only
(https://gerrit.openbmc-project.xyz/c/openbmc/bmcweb/+/47474). It will
be used in future CLs for Query Parameters.
The code changed the completion handle to accept Res to be able to
recall handle with a new Response object.
AsyncResp owns a new res, so there is no need to pass in a res.
Also fixed a self-move assignment bug.
Context:
Originally submitted:
https://gerrit.openbmc-project.xyz/c/openbmc/bmcweb/+/480020
Reveted here:
https://gerrit.openbmc-project.xyz/c/openbmc/bmcweb/+/48880
Because of failures here:
https://gerrit.openbmc-project.xyz/c/openbmc/openbmc/+/48864
Tested:
1. Romulus QEMU + Robot tests; all passed
2. Use scripts/websocket_test.py to test websockets. It is still work correctly.
3. Tested in real hardware; no new validator errors; tested both
authless, session, and basic auth.
4. Hacked codes to return 500 errors on certain resource; response is
expected;
5. Tested Eventing, the push style one (not SSE which is still under
review), worked as expected.
6. Tested 404 errors; response is expected.
Signed-off-by: Nan Zhou <nanzhoumails@gmail.com>
Signed-off-by: John Edward Broadbent <jebr@google.com>
Change-Id: I52adb174476e0f6656335baa6657456752a031be
Diffstat (limited to 'redfish-core/lib/sensors.hpp')
-rw-r--r-- | redfish-core/lib/sensors.hpp | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/redfish-core/lib/sensors.hpp b/redfish-core/lib/sensors.hpp index a53c1124fa..dd2d752ccc 100644 --- a/redfish-core/lib/sensors.hpp +++ b/redfish-core/lib/sensors.hpp @@ -2910,10 +2910,9 @@ inline void retrieveUriToDbusMap(const std::string& chassis, return; } - auto res = std::make_shared<crow::Response>(); - auto asyncResp = std::make_shared<bmcweb::AsyncResp>(*res); + auto asyncResp = std::make_shared<bmcweb::AsyncResp>(); auto callback = - [res, asyncResp, mapCompleteCb{std::move(mapComplete)}]( + [asyncResp, mapCompleteCb{std::move(mapComplete)}]( const boost::beast::http::status status, const boost::container::flat_map<std::string, std::string>& uriToDbus) { mapCompleteCb(status, uriToDbus); }; |