From 10f0d240a56e33acf78dae93fa6f71387f2e2af2 Mon Sep 17 00:00:00 2001
From: Ed Tanous <ed@tanous.net>
Date: Wed, 28 Oct 2020 15:23:15 +0000
Subject: 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
---
 http/http_connection.hpp | 4 ----
 http/http_request.hpp    | 2 --
 2 files changed, 6 deletions(-)

(limited to 'http')

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;
 
-- 
cgit v1.2.3