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/http_request.hpp b/http/http_request.hpp
index c2e7d8d..160216b 100644
--- a/http/http_request.hpp
+++ b/http/http_request.hpp
@@ -7,13 +7,20 @@
#include <boost/asio/io_context.hpp>
#include <boost/asio/ip/address.hpp>
+#include <boost/beast/http/field.hpp>
+#include <boost/beast/http/fields.hpp>
#include <boost/beast/http/message.hpp>
-#include <boost/beast/websocket.hpp>
+#include <boost/beast/http/verb.hpp>
+#include <boost/beast/websocket/rfc6455.hpp>
+#include <boost/url/parse.hpp>
#include <boost/url/url.hpp>
+#include <boost/url/url_view.hpp>
+#include <memory>
#include <string>
#include <string_view>
#include <system_error>
+#include <utility>
namespace crow
{
@@ -146,6 +153,7 @@
bool isUpgrade() const
{
+ // NOLINTNEXTLINE(misc-include-cleaner)
return boost::beast::websocket::is_upgrade(req);
}