Minimal c interface

Summary

See https://gitlab.indiscale.com/caosdb/customers/lfpb/management/-/issues/404

• Minimal C Interface
• Configuration via json files
• Renaming of many classes and functions (In order to comply with best-practices as formulated by https://google.github.io/styleguide/cppguide.html)

Integration tests are in caosdb-cppinttest!1 (merged)

Focus

• Please take a look at the c-interface functions:
  • I the naming consistent?
  • I see one problem regarding the destructors: When a client uses caosdb_connection_connection_manager_get_default_connection to get the default connection, the destructor caosdb_connection_delete_connection should not be called, because the connection object is actually owned by the ConnectionManager. Maybe, we should prevent the deletion of the wrapped connection somehow, when the connection is owned by the ConnectionManager?
• ConfigurationManager and ConnectionManager:
  • These two singletons are tightly coupled. When the ConfigurationManager is being reset, also the ConnectionManager resets. Is this a good behavior?
• There is currently no error/exception handling in the c-interface. I will add this in https://gitlab.indiscale.com/caosdb/customers/lfpb/management/-/issues/390

Test Environment

Please try and setup a client (e.g. the ccaosdbcli) with a json configuration file.

Check List for the Author

Please, prepare your MR for a review. Be sure to write a summary and a focus and create gitlab comments for the reviewer. They should guide the reviewer through the changes, explain your changes and also point out open questions. For further good practices have a look at our review guidelines

• All automated tests pass
• Reference related Issues
• Up-to-date CHANGELOG.md
• Annotations in code (Gitlab comments)
  • Intent of new code
  • Problems with old code
  • Why this implementation?

Check List for the Reviewer

• I understand the intent of this MR
• All automated tests pass
• Up-to-date CHANGELOG.md
• The test environment setup works and the intended behavior is reproducible in the test environment
• In-code documentation and comments are up-to-date.
• Check: Are there spezifications? Are they satisfied?

For further good practices have a look at our review guidelines.

include/caosdb/configuration.h

@@ -29,9 +29,9 @@
 #include "boost/json/value.hpp"            // for value
 #include "boost/json/value_ref.hpp"        // for array, object
 #include "caosdb/authentication.h"         // for Authenticator, PlainPassw...
-#include "caosdb/connection.h"             // for ConnectionConfig, Certifi...
-#include "caosdb/exceptions.h"             // for ConfigurationError
-#include "caosdb/utility.h"                // for load_json_file
+#include "caosdb/connection.h" // for ConnectionConfiguration, Certifi...
+#include "caosdb/exceptions.h" // for ConfigurationError
+#include "caosdb/utility.h"    // for load_json_file
 
 namespace caosdb::configuration {
 using boost::filesystem::exists;
@@ -42,11 +42,11 @@ using boost::json::value;
 using caosdb::authentication::Authenticator;
 using caosdb::authentication::PlainPasswordAuthenticator;
 using caosdb::connection::CertificateProvider;
-using caosdb::connection::ConnectionConfig;
+using caosdb::connection::ConnectionConfiguration;
 using caosdb::connection::ConnectionManager;
-using caosdb::connection::InsecureConnectionConfig;
+using caosdb::connection::InsecureConnectionConfiguration;
 using caosdb::connection::PemFileCertificateProvider;
-using caosdb::connection::TlsConnectionConfig;
+using caosdb::connection::TlsConnectionConfiguration;
 using caosdb::exceptions::ConfigurationError;
 using caosdb::utility::load_json_file;
 
@@ -80,7 +80,7 @@ private:
                                 const int port,
                                 const CertificateProvider *certificate_provider,
                                 const Authenticator *authenticator) const
-    -> std::unique_ptr<ConnectionConfig>;
+    -> std::unique_ptr<ConnectionConfiguration>;
 
   /**
    * @param from - a single connection configuration.
@@ -91,7 +91,7 @@ private:
    * @param from - a single connection configuration.
    */
   auto CreateConnectionConfiguration(const object &from) const
-    -> std::unique_ptr<ConnectionConfig>;
+    -> std::unique_ptr<ConnectionConfiguration>;
 };
 
 /**
@@ -129,15 +129,15 @@ public:
    * See mGetConnectionConfiguration.
    */
   inline static auto GetConnectionConfiguration(const std::string &name)
-    -> std::unique_ptr<ConnectionConfig> {
+    -> std::unique_ptr<ConnectionConfiguration> {
     return GetInstance().mGetConnectionConfiguration(name);
   }
 
   /**
-   * Return the ConnectionConfig for the default connection.
+   * Return the ConnectionConfiguration for the default connection.
    */
   inline static auto GetDefaultConnectionConfiguration()
-    -> std::unique_ptr<ConnectionConfig> {
+    -> std::unique_ptr<ConnectionConfiguration> {
     return GetInstance().mGetConnectionConfiguration(
       GetInstance().mGetDefaultConnectionName());
   }
@@ -206,13 +206,13 @@ private:
   auto mLoadSingleJSONConfiguration(const path &json_file) -> void;
 
   /**
-   * Return the ConnectionConfig for the connection of the given name.
+   * Return the ConnectionConfiguration for the connection of the given name.
    */
   auto mGetConnectionConfiguration(const std::string &name) const
-    -> std::unique_ptr<ConnectionConfig>;
+    -> std::unique_ptr<ConnectionConfiguration>;
 
   /**
-   * Return the ConnectionConfig for the default connection.
+   * Return the ConnectionConfiguration for the default connection.
    */
   auto mGetDefaultConnectionName() const -> std::string;
 };