Break out SSL key handler into a compile unit
This commit allows for no code to have to pull in openssl headers
directly. All openssl code is now included in compile units, or
transitively from boost.
Because http2 is optional, no-unneeded-internal-declaration is needed to
prevent clang from marking the functions as unused. Chromium has
disabled this as well[1]
Tested:
Redfish service validator passes.
[1] https://issues.chromium.org/issues/40340369
Change-Id: I327e8ffa45941c2282db804d0be56cf64155e67d
Signed-off-by: Ed Tanous <ed@tanous.net>
diff --git a/meson.build b/meson.build
index bce2c95..5a4ec87 100644
--- a/meson.build
+++ b/meson.build
@@ -86,6 +86,7 @@
'-Wno-unused-macros',
'-Wno-covered-switch-default',
'-Wno-disabled-macro-expansion',
+ '-Wno-unneeded-internal-declaration',
language: 'cpp',
)
endif
@@ -327,10 +328,8 @@
fs = import('fs')
srcfiles_bmcweb = files(
- # end large files
-
+ 'http/mutual_tls.cpp',
'redfish-core/src/error_messages.cpp',
- # Begin large files, should be at the beginning
'redfish-core/src/filter_expr_executor.cpp',
'redfish-core/src/filter_expr_printer.cpp',
'redfish-core/src/redfish.cpp',
@@ -344,6 +343,7 @@
'src/dbus_singleton.cpp',
'src/json_html_serializer.cpp',
'src/ossl_random.cpp',
+ 'src/ssl_key_handler.cpp',
'src/webserver_run.cpp',
)