Fix includes
Clang-tidy misc-include-cleaner appears to now be enforcing
significantly more headers than previously. That is overall a good
thing, but forces us to fix some issues. This commit is largely just
taking the clang-recommended fixes and checking them in. Subsequent
patches will fix the more unique issues.
Note, that a number of new ignores are added into the .clang-tidy file.
These can be cleaned up over time as they're understood. The majority
are places where boost includes a impl/x.hpp and x.hpp, but expects you
to use the later. include-cleaner opts for the impl, but it isn't clear
why.
Change-Id: Id3fdd7ee6df6c33b2fd35626898523048dd51bfb
Signed-off-by: Ed Tanous <etanous@nvidia.com>
Signed-off-by: Gunnar Mills <gmills@us.ibm.com>
diff --git a/http/routing.hpp b/http/routing.hpp
index 010ee2d..6c39939 100644
--- a/http/routing.hpp
+++ b/http/routing.hpp
@@ -4,33 +4,30 @@
#include "async_resp.hpp"
#include "dbus_privileges.hpp"
-#include "dbus_utility.hpp"
-#include "error_messages.hpp"
#include "http_request.hpp"
#include "http_response.hpp"
#include "logging.hpp"
-#include "privileges.hpp"
#include "routing/baserule.hpp"
#include "routing/dynamicrule.hpp"
-#include "routing/sserule.hpp"
#include "routing/taggedrule.hpp"
-#include "routing/websocketrule.hpp"
-#include "sessions.hpp"
-#include "utility.hpp"
-#include "utils/dbus_utils.hpp"
#include "verb.hpp"
-#include "websocket.hpp"
+#include <boost/beast/http/field.hpp>
+#include <boost/beast/http/status.hpp>
#include <boost/container/flat_map.hpp>
#include <boost/container/small_vector.hpp>
#include <algorithm>
+#include <array>
#include <cerrno>
#include <cstdint>
#include <cstdlib>
-#include <limits>
+#include <format>
+#include <functional>
#include <memory>
#include <optional>
+#include <stdexcept>
+#include <string>
#include <string_view>
#include <tuple>
#include <utility>