diff options
author | Ed Tanous <edtanous@google.com> | 2022-06-29 20:05:19 +0300 |
---|---|---|
committer | Ed Tanous <edtanous@google.com> | 2022-12-15 23:03:47 +0300 |
commit | 079360ae6e04d3f2245e00d70f83d15c5cad3630 (patch) | |
tree | fcc89ce4e23a8fc63f18ad7ccf5cd044457861f0 /http/websocket.hpp | |
parent | 0fb5b5051bebfe1330627a02d8f7c83195f71ed3 (diff) | |
download | bmcweb-079360ae6e04d3f2245e00d70f83d15c5cad3630.tar.xz |
Prepare for boost::url upgrade
The new boost URL now interops properly with std::string_view, which is
great, and cleans up a bunch of mediocre code to convert one to another.
It has also been pulled into boost-proper, so we no longer need a
boost-url dependency that's separate.
Unfortunately, boost url makes these improvements by changing
boost::string_view for boost::urls::const_string, which causes us to
have some compile errors on the missing type.
The bulk of these changes fall into a couple categories, and have to be
executed in one commit.
string() is replaced with buffer() on the url and url_view types
boost::string_view is replaced by std::string_view for many times, in
many cases removing a temporary that we had in the code previously.
Tested: Code compiles with boost 1.81.0 beta.
Redfish service validator passes.
Pretty good unit test coverage for URL-specific use cases.
Signed-off-by: Ed Tanous <edtanous@google.com>
Change-Id: I8d3dc89b53d1cc390887fe53605d4867f75f76fd
Diffstat (limited to 'http/websocket.hpp')
-rw-r--r-- | http/websocket.hpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/http/websocket.hpp b/http/websocket.hpp index ecaa1d2f9a..3e4ef0c40c 100644 --- a/http/websocket.hpp +++ b/http/websocket.hpp @@ -218,8 +218,8 @@ class ConnectionImpl : public Connection } if (closeHandler) { - std::string_view reason = ws.reason().reason; - closeHandler(*this, std::string(reason)); + std::string reason{ws.reason().reason.c_str()}; + closeHandler(*this, reason); } return; } |