diff options
author | James Feist <james.feist@linux.intel.com> | 2019-11-08 21:37:37 +0300 |
---|---|---|
committer | James Feist <james.feist@linux.intel.com> | 2019-11-08 21:37:37 +0300 |
commit | 4380545ee6b3985f30060e4828fe5ac3a1e3475e (patch) | |
tree | a54e098c8702af8d083a48de0aaf576a8cc84b93 | |
parent | 27c10d2ee746b85e9463efb0fc6773c209b2f5ba (diff) | |
download | bmcweb-4380545ee6b3985f30060e4828fe5ac3a1e3475e.tar.xz |
Revert "account_service: Added NoAccess role to Redfish"
This reverts commit 27c10d2ee746b85e9463efb0fc6773c209b2f5ba.
Reason for revert: <Makes the validator fail>
Change-Id: I379d9eda57416476ff1cc17e594c55dedd0bc4eb
Signed-off-by: James Feist <james.feist@linux.intel.com>
-rw-r--r-- | redfish-core/lib/account_service.hpp | 8 | ||||
-rw-r--r-- | redfish-core/lib/roles.hpp | 8 |
2 files changed, 0 insertions, 16 deletions
diff --git a/redfish-core/lib/account_service.hpp b/redfish-core/lib/account_service.hpp index 291a207d02..0658f3ff3b 100644 --- a/redfish-core/lib/account_service.hpp +++ b/redfish-core/lib/account_service.hpp @@ -90,10 +90,6 @@ inline std::string getRoleIdFromPrivilege(std::string_view role) { return "Operator"; } - else if ((role == "") || (role == "priv-noaccess")) - { - return "NoAccess"; - } return ""; } inline std::string getPrivilegeFromRoleId(std::string_view role) @@ -114,10 +110,6 @@ inline std::string getPrivilegeFromRoleId(std::string_view role) { return "priv-operator"; } - else if (role == "NoAccess") - { - return "priv-noaccess"; - } return ""; } diff --git a/redfish-core/lib/roles.hpp b/redfish-core/lib/roles.hpp index 58b45775d7..d170a5c358 100644 --- a/redfish-core/lib/roles.hpp +++ b/redfish-core/lib/roles.hpp @@ -40,10 +40,6 @@ inline std::string getRoleFromPrivileges(std::string_view priv) { return "Operator"; } - else if (priv == "priv-noaccess") - { - return "NoAccess"; - } return ""; } @@ -67,10 +63,6 @@ inline bool getAssignedPrivFromRole(std::string_view role, { privArray = {"Login"}; } - else if (role == "NoAccess") - { - privArray = {}; - } else { return false; |