Remove extra variables in websockets

These variables don't need propagated to handlers.  Any usage of them is
incorrect.

This makes Websocket once again a pure virtual class, which is desired.

Signed-off-by: Ed Tanous <edtanous@google.com>
Change-Id: Id1ecc3911fc502d436a3e6aa29024628fc51aff4
diff --git a/http/routing/websocketrule.hpp b/http/routing/websocketrule.hpp
index 5e558f2..bf6daad 100644
--- a/http/routing/websocketrule.hpp
+++ b/http/routing/websocketrule.hpp
@@ -37,9 +37,9 @@
             crow::websocket::ConnectionImpl<boost::asio::ip::tcp::socket>>
             myConnection = std::make_shared<
                 crow::websocket::ConnectionImpl<boost::asio::ip::tcp::socket>>(
-                req, req.url(), std::move(adaptor), openHandler, messageHandler,
-                messageExHandler, closeHandler, errorHandler);
-        myConnection->start();
+                req.url(), req.session, std::move(adaptor), openHandler,
+                messageHandler, messageExHandler, closeHandler, errorHandler);
+        myConnection->start(req);
     }
 #else
     void handleUpgrade(const Request& req,
@@ -52,9 +52,9 @@
             boost::beast::ssl_stream<boost::asio::ip::tcp::socket>>>
             myConnection = std::make_shared<crow::websocket::ConnectionImpl<
                 boost::beast::ssl_stream<boost::asio::ip::tcp::socket>>>(
-                req, req.url(), std::move(adaptor), openHandler, messageHandler,
-                messageExHandler, closeHandler, errorHandler);
-        myConnection->start();
+                req.url(), req.session, std::move(adaptor), openHandler,
+                messageHandler, messageExHandler, closeHandler, errorHandler);
+        myConnection->start(req);
     }
 #endif