Remove socket() call from Request
The socket() call in the request object is a pretty bad architecture
break. Requests should have no knowledge of the underlying socket. It
is currently not used.
Tested:
Code compiles
Signed-off-by: Ed Tanous <ed@tanous.net>
Change-Id: I1f72667857c377df370e3238a81d6c46fba1e8f5
diff --git a/http/http_request.h b/http/http_request.h
index fa60f60..367f733 100644
--- a/http/http_request.h
+++ b/http/http_request.h
@@ -15,12 +15,6 @@
struct Request
{
-#ifdef BMCWEB_ENABLE_SSL
- using Adaptor = boost::beast::ssl_stream<boost::asio::ip::tcp::socket>;
-#else
- using Adaptor = boost::asio::ip::tcp::socket;
-#endif
-
boost::beast::http::request<boost::beast::http::string_body>& req;
boost::beast::http::fields& fields;
std::string_view url{};
@@ -35,7 +29,6 @@
std::shared_ptr<persistent_data::UserSession> session;
std::string userRole{};
- std::function<Adaptor&()> socket;
Request(
boost::beast::http::request<boost::beast::http::string_body>& reqIn) :
req(reqIn),