Refactor MongoConfigManager to use Promises for MongoDB connection ha…

…ndling and improve error logging
scality · Nov 21, 2024 · 9b9db18 · 9b9db18
1 parent c0fbb85
commit 9b9db18
Showing 1 changed file with 7 additions and 9 deletions.
diff --git a/extensions/notification/configManager/MongoConfigManager.js b/extensions/notification/configManager/MongoConfigManager.js
@@ -100,15 +100,7 @@ class MongoConfigManager extends BaseConfigManager {
         MongoClient.connect(mongoUrl, {
             replicaSet: this._mongoConfig.replicaSet,
             useNewUrlParser: true,
-        },
-        (err, client) => {
-            if (err) {
-                this._logger.error('Could not connect to MongoDB', {
-                    method: 'MongoConfigManager._setupMongoClient',
-                    error: err.message,
-                });
-                return cb(err);
-            }
+        }).then((client) => {
             this._logger.debug('Connected to MongoDB', {
                 method: 'MongoConfigManager._setupMongoClient',
             });
@@ -133,6 +125,12 @@ class MongoConfigManager extends BaseConfigManager {
             } catch (error) {
                 return cb(error);
             }
+        }).catch(err => {
+            this._logger.error('Could not connect to MongoDB', {
+                method: 'MongoConfigManager._setupMongoClient',
+                error: err.message,
+            });
+            return cb(err);
         });
     }