diff options
author | Myung Bae <myungbae@us.ibm.com> | 2024-04-01 17:27:39 +0300 |
---|---|---|
committer | Ed Tanous <ed@tanous.net> | 2024-04-02 02:22:31 +0300 |
commit | 1873a04f43bb414408d1da9a5a775c05474603d2 (patch) | |
tree | 01859da41d4c385874e95fe28f3643049a034609 /http | |
parent | 4393bdb981d97e8db1d2cf7700a507222f14e08b (diff) | |
download | bmcweb-1873a04f43bb414408d1da9a5a775c05474603d2.tar.xz |
Reduce multi-level calls of req.req members
Several places access the members of `req` indirectly like
`req.req.method()`. This can be simplified as `req.method()` .
This would also make the code clearer.
Tested:
- Compiles
- Redfish service validator passes
Change-Id: Ie129564ff907cdea7ac224b1e3d80cc0dedfbd7b
Signed-off-by: Myung Bae <myungbae@us.ibm.com>
Diffstat (limited to 'http')
-rw-r--r-- | http/http2_connection.hpp | 4 | ||||
-rw-r--r-- | http/http_request.hpp | 9 | ||||
-rw-r--r-- | http/websocket.hpp | 3 |
3 files changed, 13 insertions, 3 deletions
diff --git a/http/http2_connection.hpp b/http/http2_connection.hpp index 4b1b2dcb91..8416ff738b 100644 --- a/http/http2_connection.hpp +++ b/http/http2_connection.hpp @@ -434,7 +434,7 @@ class HTTP2Connection : close(); return -1; } - thisReq.req.method(verb); + thisReq.method(verb); } else if (nameSv == ":scheme") { @@ -442,7 +442,7 @@ class HTTP2Connection : } else { - thisReq.req.set(nameSv, valueSv); + thisReq.addHeader(nameSv, valueSv); } return 0; } diff --git a/http/http_request.hpp b/http/http_request.hpp index a3c8fd8d0e..53f51e68fd 100644 --- a/http/http_request.hpp +++ b/http/http_request.hpp @@ -80,6 +80,10 @@ struct Request { return req.method(); } + void method(boost::beast::http::verb verb) + { + return req.method(verb); + } std::string_view getHeaderValue(std::string_view key) const { @@ -91,6 +95,11 @@ struct Request return req[key]; } + void clearHeader(boost::beast::http::field key) + { + req.erase(key); + } + std::string_view methodString() const { return req.method_string(); diff --git a/http/websocket.hpp b/http/websocket.hpp index d232c02330..2ef8412886 100644 --- a/http/websocket.hpp +++ b/http/websocket.hpp @@ -92,7 +92,8 @@ class ConnectionImpl : public Connection BMCWEB_LOG_DEBUG("starting connection {}", logPtr(this)); using bf = boost::beast::http::field; - std::string protocolHeader = req.req[bf::sec_websocket_protocol]; + std::string protocolHeader{ + req.getHeaderValue(bf::sec_websocket_protocol)}; ws.set_option(boost::beast::websocket::stream_base::decorator( [session{session}, |