diff options
| author | Ed Tanous <ed@tanous.net> | 2020-10-28 18:23:15 +0300 |
|---|---|---|
| committer | Ed Tanous <ed@tanous.net> | 2020-10-29 02:10:06 +0300 |
| commit | 10f0d240a56e33acf78dae93fa6f71387f2e2af2 (patch) | |
| tree | 5a46bcb9489b7249f9d47bc12c10310175bcedf6 /http | |
| parent | dc3fa6671cd588d5a09416da09030f82aff2e50b (diff) | |
| download | bmcweb-10f0d240a56e33acf78dae93fa6f71387f2e2af2.tar.xz | |
Revert "Redfish Session : Support ClientOriginIPAddress"
This reverts commit e436008377fbcf287be02c9e9e1b59c6627d7673.
Reason for revert:
This breaks several things.
1. Not all login endpoints are handled, which lead to returning blank ip addresses
2. IP addresses are not persisted.
3. This crashes occasionally on remote_endpoint, and ignores ec.
Change-Id: I58c875721cf48bf02db833c9c57a9eead5e249d5
Diffstat (limited to 'http')
| -rw-r--r-- | http/http_connection.hpp | 4 | ||||
| -rw-r--r-- | http/http_request.hpp | 2 |
2 files changed, 0 insertions, 6 deletions
diff --git a/http/http_connection.hpp b/http/http_connection.hpp index 302d19bd97..ec8e9d5233 100644 --- a/http/http_connection.hpp +++ b/http/http_connection.hpp @@ -316,10 +316,6 @@ class Connection : } } - // Copy the client's IP address - req->ipAddress = - boost::beast::get_lowest_layer(adaptor).remote_endpoint().address(); - BMCWEB_LOG_INFO << "Request: " << " " << this << " HTTP/" << req->version() / 10 << "." << req->version() % 10 << ' ' << req->methodString() diff --git a/http/http_request.hpp b/http/http_request.hpp index d9d6cf723c..e5c0c9e982 100644 --- a/http/http_request.hpp +++ b/http/http_request.hpp @@ -4,7 +4,6 @@ #include "sessions.hpp" #include <boost/asio/io_context.hpp> -#include <boost/asio/ip/address.hpp> #include <boost/beast/http/message.hpp> #include <boost/beast/http/string_body.hpp> #include <boost/beast/websocket.hpp> @@ -25,7 +24,6 @@ struct Request const std::string& body; boost::asio::io_context* ioService{}; - boost::asio::ip::address ipAddress; std::shared_ptr<persistent_data::UserSession> session; |