diff options
author | Abhishek Patel <Abhishek.Patel@ibm.com> | 2021-08-10 16:50:09 +0300 |
---|---|---|
committer | Ed Tanous <ed@tanous.net> | 2021-08-20 00:15:55 +0300 |
commit | 1759dc7a351b665cce9cf260e0c911f489a344ff (patch) | |
tree | 6000fe05773e24bf0acafa197e1501aed7a996a5 /redfish-core/lib/bios.hpp | |
parent | 97128e926c8253f02cf25d2ca9881cadc426df2c (diff) | |
download | bmcweb-1759dc7a351b665cce9cf260e0c911f489a344ff.tar.xz |
Fix BIOS privileges
Post method:
1) /redfish/v1/Systems/system/Bios/Actions/Bios.ResetBios/
ConfigureManager -> ConfigureComponents
This change allows Admin and operator users to Reset bios.
Before this change, the only admin user has that privileges.
Tested: Ran curl Post requests with Admin and Operator privileged users
Get output as aspected.
Email sent to openbmc list:
https://lists.ozlabs.org/pipermail/openbmc/2021-August/027232.html
Signed-off-by: Abhishek Patel <Abhishek.Patel@ibm.com>
Change-Id: Ia8638c822f0f84d657ece1a8e1b1aa38019b24d7
Diffstat (limited to 'redfish-core/lib/bios.hpp')
-rw-r--r-- | redfish-core/lib/bios.hpp | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/redfish-core/lib/bios.hpp b/redfish-core/lib/bios.hpp index 604b8e7edd..3cf6077c4b 100644 --- a/redfish-core/lib/bios.hpp +++ b/redfish-core/lib/bios.hpp @@ -42,9 +42,7 @@ inline void requestRoutesBiosService(App& app) inline void requestRoutesBiosReset(App& app) { BMCWEB_ROUTE(app, "/redfish/v1/Systems/system/Bios/Actions/Bios.ResetBios/") - // Incorrect Privilege; Should be ConfigureComponents - //.privileges(redfish::privileges::postBios) - .privileges({{"ConfigureManager"}}) + .privileges(redfish::privileges::postBios) .methods(boost::beast::http::verb::post)( [](const crow::Request&, const std::shared_ptr<bmcweb::AsyncResp>& asyncResp) { |