feat: Implement http/https requests. (#27)

Author: kinyoklion

libs/client-sdk/src/CMakeLists.txt

@@ -9,7 +9,10 @@ file(GLOB HEADER_LIST CONFIGURE_DEPENDS
         )
 
 # Automatic library: static or dynamic based on user config.
-add_library(${LIBNAME} api.cpp ${HEADER_LIST}
+
+add_library(${LIBNAME}
+        api.cpp
+        ${HEADER_LIST}
         data_sources/streaming_data_source.cpp
         data_sources/base_64.cpp
         data_sources/streaming_data_handler.cpp

libs/common/CMakeLists.txt

@@ -24,13 +24,20 @@ endif ()
 # Needed to fetch external dependencies. 
 include(FetchContent)
 
+set(OPENSSL_USE_STATIC_LIBS OFF)
+set(OPENSSL_ROOT_DIR "/opt/homebrew/opt/[email protected]")
+find_package(OpenSSL REQUIRED)
+message(STATUS "LaunchDarkly: using OpenSSL v${OPENSSL_VERSION}")
+
 set(Boost_USE_STATIC_LIBS OFF)
 set(Boost_USE_MULTITHREADED ON)
 set(Boost_USE_STATIC_RUNTIME OFF)
-find_package(Boost 1.80 REQUIRED)
+find_package(Boost 1.80 REQUIRED COMPONENTS json url)
 message(STATUS "LaunchDarkly: using Boost v${Boost_VERSION}")
 
+
 include(${CMAKE_FILES}/expected.cmake)
+include(${CMAKE_FILES}/certify.cmake)
 
 # Add main SDK sources.
 add_subdirectory(src)

libs/common/include/config/detail/builders/http_properties_builder.hpp

@@ -10,31 +10,116 @@
 
 namespace launchdarkly::config::detail::builders {
 
+/**
+ * Class used for building a set of HttpProperties.
+ * @tparam SDK The SDK type to build properties for. This affects the default
+ * values of the built properties.
+ */
 template <typename SDK>
 class HttpPropertiesBuilder {
    public:
-    HttpPropertiesBuilder() = default;
+    /**
+     * Construct a new HttpPropertiesBuilder. The builder will use the default
+     * properties based on the SDK type. Setting a property will override
+     * the default value.
+     */
+    HttpPropertiesBuilder();
 
+    /**
+     * Create a properties builder from an initial set of properties.
+     * This can be useful when extending a set of properties for a request.
+     * For instance to add extra headers.
+     *
+     * ```
+     * HttpPropertiesBuilder(my_properties)
+     *   .Header("authorization", "my-key")
+     *   .Build();
+     * ```
+     *
+     * @param properties The properties to start with.
+     */
+    HttpPropertiesBuilder(built::HttpProperties properties);
+
+    /**
+     * The network connection timeout.
+     *
+     * @param connect_timeout The connect timeout.
+     * @return A reference to this builder.
+     */
     HttpPropertiesBuilder& ConnectTimeout(
         std::chrono::milliseconds connect_timeout);
 
+    /**
+     * Set a read timeout. This is the time after the first byte
+     * has been received that a read has to complete.
+     *
+     * @param read_timeout The read timeout.
+     * @return A reference to this builder.
+     */
     HttpPropertiesBuilder& ReadTimeout(std::chrono::milliseconds read_timeout);
 
+    /**
+     * The time for the first byte to be received during a read. If a byte
+     * is not received within this time, then the request will be cancelled.
+     *
+     * @param response_timeout The response timeout.
+     * @return A reference to this builder.
+     */
+    HttpPropertiesBuilder& ResponseTimeout(
+        std::chrono::milliseconds response_timeout);
+
+    /**
+     * This should be used for wrapper SDKs to set the wrapper name.
+     *
+     * Wrapper information will be included in request headers.
+     * @param wrapper_name The name of the wrapper.
+     * @return A reference to this builder.
+     */
     HttpPropertiesBuilder& WrapperName(std::string wrapper_name);
 
+    /**
+     * This should be used for wrapper SDKs to set the wrapper version.
+     *
+     * Wrapper information will be included in request headers.
+     * @param wrapper_version The version of the wrapper.
+     * @return A reference to this builder.
+     */
     HttpPropertiesBuilder& WrapperVersion(std::string wrapper_version);
 
+    /**
+     * Set all custom headers. This will replace any other customer headers
+     * that were set with the Header method, or any previously set
+     * headers using the CustomHeaders method.
+     * @param base_headers The custom headers.
+     * @return A reference to this builder.
+     */
     HttpPropertiesBuilder& CustomHeaders(
         std::map<std::string, std::string> base_headers);
 
+    /**
+     * Set a custom header value.
+     *
+     * Calling CustomHeaders will replace any previously set values.
+     * @param key The key for the header.
+     * @param value The header value.
+     * @return A reference to this builder.
+     */
+    HttpPropertiesBuilder& Header(std::string key, std::string value);
+
+    /**
+     * Build a set of HttpProperties.
+     * @return The built properties.
+     */
     [[nodiscard]] built::HttpProperties Build() const;
 
    private:
-    std::chrono::milliseconds connect_timeout_{};
-    std::chrono::milliseconds read_timeout_{};
+    std::chrono::milliseconds connect_timeout_;
+    std::chrono::milliseconds read_timeout_;
+    std::chrono::milliseconds response_timeout_;
     std::string wrapper_name_;
     std::string wrapper_version_;
     std::map<std::string, std::string> base_headers_;
+    std::string user_agent_;
 };
 
 }  // namespace launchdarkly::config::detail::builders

libs/common/include/config/detail/built/http_properties.hpp

@@ -11,17 +11,20 @@ class HttpProperties final {
    public:
     HttpProperties(std::chrono::milliseconds connect_timeout,
                    std::chrono::milliseconds read_timeout,
+                   std::chrono::milliseconds response_timeout,
                    std::string user_agent,
                    std::map<std::string, std::string> base_headers);
 
     [[nodiscard]] std::chrono::milliseconds ConnectTimeout() const;
     [[nodiscard]] std::chrono::milliseconds ReadTimeout() const;
+    [[nodiscard]] std::chrono::milliseconds ResponseTimeout() const;
     [[nodiscard]] std::string const& UserAgent() const;
     [[nodiscard]] std::map<std::string, std::string> const& BaseHeaders() const;
 
    private:
     std::chrono::milliseconds connect_timeout_;
     std::chrono::milliseconds read_timeout_;
+    std::chrono::milliseconds response_timeout_;
     std::string user_agent_;
     std::map<std::string, std::string> base_headers_;
 

libs/common/include/config/detail/defaults.hpp

@@ -49,10 +49,10 @@ struct Defaults<ClientSDK> {
     }
 
     static auto HttpProperties() -> built::HttpProperties {
-        return {std::chrono::milliseconds{10000},
-                std::chrono::milliseconds{10000},
-                SdkName() + "/" + SdkVersion(),
-                std::map<std::string, std::string>()};
+        return {
+            std::chrono::milliseconds{10000}, std::chrono::milliseconds{10000},
+            std::chrono::milliseconds{10000}, SdkName() + "/" + SdkVersion(),
+            std::map<std::string, std::string>()};
     }
 
     static auto DataSourceConfig() -> built::DataSourceConfig<ClientSDK> {
@@ -78,10 +78,10 @@ struct Defaults<ServerSDK> {
     }
 
     static auto HttpProperties() -> built::HttpProperties {
-        return {std::chrono::milliseconds{2000},
-                std::chrono::milliseconds{10000},
-                SdkName() + "/" + SdkVersion(),
-                std::map<std::string, std::string>()};
+        return {
+            std::chrono::milliseconds{2000}, std::chrono::milliseconds{10000},
+            std::chrono::milliseconds{10000}, SdkName() + "/" + SdkVersion(),
+            std::map<std::string, std::string>()};
     }
 
     static auto DataSourceConfig() -> built::DataSourceConfig<ServerSDK> {