diff options
author | Ed Tanous <edtanous@google.com> | 2021-09-23 05:09:11 +0300 |
---|---|---|
committer | Ed Tanous <ed@tanous.net> | 2021-10-18 23:37:20 +0300 |
commit | efee36b96e75475f778a7887d9000458437d0285 (patch) | |
tree | 2fb77436652cbbd55cae80778e8667f68e58f50a | |
parent | f79b7a502db253f6a862aa2cdf5308a7bf2b9070 (diff) | |
download | bmcweb-efee36b96e75475f778a7887d9000458437d0285.tar.xz |
Rename method to isOnAllowlist
While we're here testing things anyway, it seems like a good time to do
a simple method rename following the projects new naming guidelines.
This seems like an easy thing to do while we're here anyway and might
short circuit a harder discussion later.
Tested:
Code compiles.
Signed-off-by: Ed Tanous <edtanous@google.com>
Change-Id: Ie5ed1c38d7dec13f875efae4acc22f4fd582f9b9
-rw-r--r-- | http/http_connection.hpp | 2 | ||||
-rw-r--r-- | include/authorization.hpp | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/http/http_connection.hpp b/http/http_connection.hpp index c296782b59..b1693092d0 100644 --- a/http/http_connection.hpp +++ b/http/http_connection.hpp @@ -355,7 +355,7 @@ class Connection : return; } - if (!crow::authorization::isOnWhitelist(req->url, req->method()) && + if (!crow::authorization::isOnAllowlist(req->url, req->method()) && thisReq.session == nullptr) { BMCWEB_LOG_WARNING << "[AuthMiddleware] authorization failed"; diff --git a/include/authorization.hpp b/include/authorization.hpp index 1c51414638..4869554d13 100644 --- a/include/authorization.hpp +++ b/include/authorization.hpp @@ -223,7 +223,7 @@ static std::shared_ptr<persistent_data::UserSession> #endif // checks if request can be forwarded without authentication -static bool isOnWhitelist(std::string_view url, boost::beast::http::verb method) +static bool isOnAllowlist(std::string_view url, boost::beast::http::verb method) { if (boost::beast::http::verb::get == method) { |