ADD nameOnly to listDatabase command on MongoDB

telefonicaid · Mar 7, 2024 · 17740ee · 17740ee
1 parent d7793bc
commit 17740ee
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 5 deletions.
diff --git a/CHANGES_NEXT_RELEASE b/CHANGES_NEXT_RELEASE
@@ -0,0 +1 @@
+- Fix: lighter operation to get databases list from MongoDB (#4517)
diff --git a/src/lib/mongoBackend/MongoGlobal.cpp b/src/lib/mongoBackend/MongoGlobal.cpp
@@ -232,6 +232,7 @@ bool getOrionDatabases(std::vector<std::string>* dbsP)
 
   orion::BSONObjBuilder bob;
   bob.append("listDatabases", 1);
+  bob.append("nameOnly", true);
 
   if (!orion::runDatabaseCommand("admin", bob.obj(), &result, &err))
   {

diff --git a/src/lib/mongoDriver/mongoConnectionPool.cpp b/src/lib/mongoDriver/mongoConnectionPool.cpp
@@ -112,22 +112,21 @@ static int pingConnection(mongoc_client_t* conn, const char* db, bool mtenat)
   // ping will be listDatabases in the admin DB. But if we run in not mtenant
   // mode listCollections in the default database will suffice
 
-  std::string  cmd;
   std::string  effectiveDb;
 
+  bson_t* ping = bson_new();
   if (mtenat)
   {
     cmd = "listDatabases";
+    BSON_APPEND_INT32(ping, "listDatabases", 1);
+    BSON_APPEND_BOOL(ping, "nameOnly", true);
     effectiveDb = "admin";
   }
   else
   {
-    cmd = "listCollections";
+    BSON_APPEND_INT32(ping, "listCollections", 1);
     effectiveDb = db;
   }
-
-  bson_t* ping = bson_new();
-  BSON_APPEND_INT32(ping, cmd.c_str(), 1);
   mongoc_database_t* database = mongoc_client_get_database(conn, effectiveDb.c_str());
 
   bson_error_t error;