doc comments

Author: kkloberdanz

src/mongocxx/collection.hpp

@@ -722,7 +722,23 @@ class MONGOCXX_API collection {
     void drop(const client_session& session,
               const bsoncxx::stdx::optional<mongocxx::write_concern>& write_concern = {});
 
+    ///
+    /// Drops this collection and all its contained documents from the database.
+    ///
+    /// @param collection_options
+    ///   the options for the dropped collection.
+    ///
+    /// @exception
+    ///   mongocxx::operation_exception if the operation fails.
+    ///
+    /// @see
+    ///   https://docs.mongodb.com/manual/reference/command/drop/
+    ///
+    /// @note
+    ///   Needed for FLE 2 Support.
+    ///
     void drop(bsoncxx::document::view_or_value collection_options);
+
     ///
     /// @}
     ///

src/mongocxx/options/auto_encryption.hpp

@@ -238,8 +238,17 @@ class MONGOCXX_API auto_encryption {
     const stdx::optional<bsoncxx::document::view_or_value>& schema_map() const;
 
     ///
-    /// TODO
+    /// Sets the local encrypted fields map.
+    ///
+    /// Supplying an encryptedFieldsMap provides more security than relying on
+    /// an encryptedFields obtained from the server. It protects against a
+    /// malicious server advertising a false encryptedFields.
+    ///
     /// @param encrypted_fields_map 
+    ///   The mapping of which fields to encrypt.
+    ///
+    /// @see https://docs.mongodb.com/manual/core/security-client-side-encryption/
+    ///
     /// @return 
     ///   A reference to this object to facilitate method chaining.
     ///
@@ -252,6 +261,7 @@ class MONGOCXX_API auto_encryption {
     ///
     /// @return 
     ///   An optional document containing the encrypted fields map
+    ///
     const stdx::optional<bsoncxx::document::view_or_value>& encrypted_fields_map() const;
 
     ///

src/mongocxx/test/spec/operation.cpp

@@ -1283,14 +1283,12 @@ document::value operation_runner::_run_create_collection(document::view operatio
     auto collection_name = arguments["collection"].get_string().value;
     auto session = _lookup_session(arguments);
 
-    std::cerr << "CREATE COLLECTION: " << bsoncxx::to_json(operation) << std::endl;
     if (session) {
         _db->create_collection(*session, collection_name);
     } else if (arguments.find("encryptedFields") != arguments.end()) {
         auto encrypted_fields = arguments["encryptedFields"].get_document().value;
         auto encrypted_fields_map = make_document(kvp("encryptedFields", encrypted_fields));
         _db->create_collection(collection_name, std::move(encrypted_fields_map));
-        std::cerr << "CREATED COLLECTION WITH ENCRYPTED FIELDS" << std::endl;
     } else {
         _db->create_collection(collection_name);
     }
@@ -1305,14 +1303,12 @@ document::value operation_runner::_run_drop_collection(document::view operation)
 
     auto arguments = operation["arguments"].get_document().value;
 
-    std::cerr << "DROP COLLECTION: " << bsoncxx::to_json(operation) << std::endl;
     auto collection_name = operation["arguments"]["collection"].get_string().value;
 
     if (arguments.find("encryptedFields") != arguments.end()) {
         auto encrypted_fields = arguments["encryptedFields"].get_document().value;
         auto encrypted_fields_map = make_document(kvp("encryptedFields", encrypted_fields));
         _db->collection(collection_name).drop(std::move(encrypted_fields_map));
-        std::cerr << "CREATED COLLECTION WITH ENCRYPTED FIELDS" << std::endl;
     } else {
         _db->collection(collection_name).drop();
     }