Fix a boatload of #includes

Most of these missing includes were found by running clang-tidy on all
files, including headers.  The existing scripts just run clang-tidy on
source files, which doesn't catch most of these.

Tested: Code compiles

Signed-off-by: Ed Tanous <edtanous@google.com>
Change-Id: Ic741fbb2cc9e5e92955fd5a1b778a482830e80e8

7 files changed, 12 insertions, 11 deletions

diff --git a/http/http_client.hpp b/http/http_client.hpp
index 117584570a..48e8cfdfac 100644
--- a/http/http_client.hpp
+++ b/http/http_client.hpp
@@ -17,6 +17,8 @@
 
 #include "async_resolve.hpp"
 #include "http_response.hpp"
+#include "logging.hpp"
+#include "ssl_key_handler.hpp"
 
 #include <boost/asio/connect.hpp>
 #include <boost/asio/io_context.hpp>
@@ -37,8 +39,6 @@
 #include <boost/beast/version.hpp>
 #include <boost/container/devector.hpp>
 #include <boost/system/error_code.hpp>
-#include <logging.hpp>
-#include <ssl_key_handler.hpp>
 
 #include <cstdlib>
 #include <functional>
diff --git a/http/http_connection.hpp b/http/http_connection.hpp
index 45581fdf26..aa8b8c9510 100644
--- a/http/http_connection.hpp
+++ b/http/http_connection.hpp
@@ -4,7 +4,11 @@
 #include "authentication.hpp"
 #include "http_response.hpp"
 #include "http_utility.hpp"
+#include "json_html_serializer.hpp"
 #include "logging.hpp"
+#include "mutual_tls.hpp"
+#include "security_headers.hpp"
+#include "ssl_key_handler.hpp"
 #include "utility.hpp"
 
 #include <boost/algorithm/string/predicate.hpp>
@@ -21,10 +25,6 @@
 #include <boost/beast/ssl/ssl_stream.hpp>
 #include <boost/beast/websocket.hpp>
 #include <boost/url/url_view.hpp>
-#include <json_html_serializer.hpp>
-#include <mutual_tls.hpp>
-#include <security_headers.hpp>
-#include <ssl_key_handler.hpp>
 
 #include <atomic>
 #include <chrono>
diff --git a/http/http_response.hpp b/http/http_response.hpp
index b1b15fb362..f9204e4a4e 100644
--- a/http/http_response.hpp
+++ b/http/http_response.hpp
@@ -1,10 +1,10 @@
 #pragma once
 #include "logging.hpp"
 #include "nlohmann/json.hpp"
+#include "utils/hex_utils.hpp"
 
 #include <boost/beast/http/message.hpp>
 #include <boost/beast/http/string_body.hpp>
-#include <utils/hex_utils.hpp>
 
 #include <optional>
 #include <string>
diff --git a/http/http_server.hpp b/http/http_server.hpp
index 0d224a10d4..cc4fc2294d 100644
--- a/http/http_server.hpp
+++ b/http/http_server.hpp
@@ -2,6 +2,7 @@
 
 #include "http_connection.hpp"
 #include "logging.hpp"
+#include "ssl_key_handler.hpp"
 
 #include <boost/asio/ip/address.hpp>
 #include <boost/asio/ip/tcp.hpp>
@@ -9,7 +10,6 @@
 #include <boost/asio/ssl/context.hpp>
 #include <boost/asio/steady_timer.hpp>
 #include <boost/beast/ssl/ssl_stream.hpp>
-#include <ssl_key_handler.hpp>
 
 #include <atomic>
 #include <chrono>
diff --git a/http/routing.hpp b/http/routing.hpp
index 8d5def670e..5b3ddfe429 100644
--- a/http/routing.hpp
+++ b/http/routing.hpp
@@ -1,5 +1,6 @@
 #pragma once
 
+#include "async_resp.hpp"
 #include "common.hpp"
 #include "dbus_utility.hpp"
 #include "error_messages.hpp"
@@ -12,7 +13,6 @@
 #include "verb.hpp"
 #include "websocket.hpp"
 
-#include <async_resp.hpp>
 #include <boost/beast/ssl/ssl_stream.hpp>
 #include <boost/container/flat_map.hpp>
 
diff --git a/http/utility.hpp b/http/utility.hpp
index f49aa9b859..4ea44d5863 100644
--- a/http/utility.hpp
+++ b/http/utility.hpp
@@ -1,6 +1,7 @@
 #pragma once
 
-#include <bmcweb_config.h>
+#include "bmcweb_config.h"
+
 #include <openssl/crypto.h>
 
 #include <boost/callable_traits.hpp>
diff --git a/http/websocket.hpp b/http/websocket.hpp
index 3e4ef0c40c..0ab13f2cdd 100644
--- a/http/websocket.hpp
+++ b/http/websocket.hpp
@@ -1,7 +1,7 @@
 #pragma once
+#include "async_resp.hpp"
 #include "http_request.hpp"
 
-#include <async_resp.hpp>
 #include <boost/asio/buffer.hpp>
 #include <boost/beast/websocket.hpp>