Remove authorization checks in nbd_proxy
nbd proxy should not have its own authorization checks, as these are
now handled in the core as of 7e9093e625961f533250a6c193c1a474e98007c4
Signed-off-by: Ed Tanous <edtanous@google.com>
Change-Id: I8874d8a09278ba21d2acfdf076cb76dee718ecf4
diff --git a/http/websocket.hpp b/http/websocket.hpp
index f17ee5e..216e96f 100644
--- a/http/websocket.hpp
+++ b/http/websocket.hpp
@@ -20,12 +20,7 @@
struct Connection : std::enable_shared_from_this<Connection>
{
public:
- explicit Connection(const crow::Request& reqIn) :
- req(reqIn.req), userdataPtr(nullptr)
- {}
-
- explicit Connection(const crow::Request& reqIn, std::string user) :
- req(reqIn.req), userName{std::move(user)}, userdataPtr(nullptr)
+ explicit Connection(const crow::Request& reqIn) : req(reqIn.req)
{}
Connection(const Connection&) = delete;
@@ -41,26 +36,7 @@
virtual boost::asio::io_context& getIoContext() = 0;
virtual ~Connection() = default;
- void userdata(void* u)
- {
- userdataPtr = u;
- }
- void* userdata()
- {
- return userdataPtr;
- }
-
- const std::string& getUserName() const
- {
- return userName;
- }
-
boost::beast::http::request<boost::beast::http::string_body> req;
- crow::Response res;
-
- private:
- std::string userName{};
- void* userdataPtr;
};
template <typename Adaptor>
@@ -74,8 +50,7 @@
messageHandlerIn,
std::function<void(Connection&, const std::string&)> closeHandlerIn,
std::function<void(Connection&)> errorHandlerIn) :
- Connection(reqIn, reqIn.session == nullptr ? std::string{}
- : reqIn.session->username),
+ Connection(reqIn),
ws(std::move(adaptorIn)), inBuffer(inString, 131088),
openHandler(std::move(openHandlerIn)),
messageHandler(std::move(messageHandlerIn)),