fixup post reviews

scality · Jan 5, 2025 · a5803ab · a5803ab
1 parent ebc43ba
commit a5803ab
Showing 1 changed file with 12 additions and 13 deletions.
diff --git a/extensions/notification/configManager/MongoConfigManager.js b/extensions/notification/configManager/MongoConfigManager.js
@@ -107,30 +107,29 @@ class MongoConfigManager extends BaseConfigManager {
             this._logger.debug('Connected to MongoDB', {
                 method: 'MongoConfigManager._setupMongoClient',
             });
-
+    
             this._mongoClient = client.db(this._mongoConfig.database, {
                 ignoreUndefined: true,
             });
             this._metastore = this._mongoClient.collection(this._bucketMetastore);
-
-            try {
-                this._mongoVersion = await promisify(getMongoVersion)(this._mongoClient);
-            } catch (err) {
-                this._logger.error('Could not get MongoDB version', {
-                    method: 'MongoConfigManager._setupMongoClient',
-                    error: err.message,
-                });
-                return cb(err);
-            }
-
-            return cb();
         } catch (err) {
             this._logger.error('Could not connect to MongoDB', {
                 method: 'MongoConfigManager._setupMongoClient',
                 error: err.message,
             });
             return cb(err);
         }
+        try {
+            this._mongoVersion = await promisify(getMongoVersion)(this._mongoClient);
+        } catch (err) {
+            this._logger.error('Could not get MongoDB version', {
+                method: 'MongoConfigManager._setupMongoClient',
+                error: err.message,
+            });
+            return cb(err);
+        }
+
+        return cb();
     }
 
     /**