applay-utils 1.9.0 → 1.9.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "applay-utils",
-  "version": "1.9.0",
+  "version": "1.9.2",
   "description": "Utilitary tools for Applay applications",
   "scripts": {
     "build": "npm version patch",

package/utils/mdb.js

@@ -7,49 +7,87 @@ var MDB = {
   clients: {}
 }
 
-var connect = (alias, url, callback) => {
+var shutdownHandlersRegistered = false;
+
+// Fecha todos os pools de conexão conhecidos (um por URL distinta), não só o
+// da conexão que originou o evento. Antes disso, cada connect() registrava
+// seu próprio listener de SIGTERM/SIGINT/uncaughtException no `process`
+// (um a mais por URL), então uma única uncaughtException não relacionada a
+// Mongo derrubava todos os pools simultaneamente e o processo inteiro, além
+// de acumular listeners duplicados (MaxListenersExceededWarning) a cada
+// nova URL conectada.
+var closeAllPools = (finish) => {
+  for (var url of Object.keys(MDB.clients)) {
+    if (!MDB.status[url]) { continue; }
+    MDB.status[url] = false;
+    try {
+      MDB.clients[url].close();
+    } catch (e) {
+      console.log('Erro ao fechar conexão Mongo', url, e);
+    }
+    console.log('Mongodb', url, 'OFF');
+  }
+
+  if (!finish) { return; }
+
+  if (MDB.stopCustom) {
+    MDB.stop('all', () => {
+      setTimeout(() => { process.exit(); }, 100);
+    });
+  } else {
+    setTimeout(() => { process.exit(); }, 100);
+  }
+}
+
+var registerShutdownHandlersOnce = () => {
+  if (shutdownHandlersRegistered) { return; }
+  shutdownHandlersRegistered = true;
+  process.on('SIGTERM', () => closeAllPools(true));
+  process.on('SIGINT', () => closeAllPools(true));
+  process.on('uncaughtException', (err) => {
+    console.log('ERRO MDB -- uncaughtException -- TRACKING -- ', err);
+    closeAllPools(true);
+  });
+}
+
+var pendingConnections = {};
+
+// `onError` é opcional (mantém compatibilidade com chamadores antigos que só
+// passavam callback de sucesso). Antes, um erro de conexão (ex: ECONNREFUSED
+// durante failover do replica set) só era logado e o callback nunca era
+// chamado — quem chamava connectAsync ficava com a Promise pendurada para
+// sempre. Além disso, várias chamadas concorrentes para a mesma URL durante
+// uma instabilidade abriam um MongoClient.connect() cada uma (thundering
+// herd); agora elas compartilham a mesma tentativa de conexão em andamento.
+var connect = (alias, url, callback, onError) => {
   if (MDB.status[url]) { return callback(MDB.clients[url]); }
+
+  if (pendingConnections[url]) {
+    return pendingConnections[url].then(callback, (err) => { if (onError) { onError(err); } });
+  }
+
   console.log('===============>>>>>>> NOVO POOL DE CONEXÃO MDB ', url, alias);
-  
+
   const separator = url.includes('?') ? '&' : '/?';
   const uriparams = `${separator}appName=${alias}&minPoolSize=2&maxPoolSize=10&maxIdleTimeMS=30000&serverSelectionTimeoutMS=5000`;
-  
-  MongoBase.MongoClient.connect(url + uriparams, (err, client) => {
-    if (err) { return console.log('ERRO MDB -- TRACKING -- ', err); }
-    MDB.status[url] = true;
-    MDB.clients[url] = client;
-    process.on('SIGTERM', internalStop(url, true, 'SIGTERM'));
-    process.on('SIGINT', internalStop(url, true, 'SIGINT'));
-    process.on('uncaughtException', internalStop(url, true, 'uncaughtException'));
-    callback(client);
-  });
-}
 
-var connectAsync = (alias, url) => new Promise(resolve => connect(alias, url, (client) => resolve(client)));
-
-var internalStop = (alias, finish) => {
-  return (code) => {
-    // console.log('Desconnecting...');
-    if (!MDB.status[alias]) { console.log('Mongodb OFF'); return }
-    MDB.status[alias] = false;
-    MDB.clients[alias].close();
-    if (finish && MDB.stopCustom) {
-      MDB.stop(alias, () => {
-        console.log('Mongodb', alias, 'OFF');
-        setTimeout(() => { process.exit(); },100)
-      });
-    }
-    if (finish && !MDB.stopCustom) {
-      console.log('Mongodb', alias, 'OFF');
-      setTimeout(() => { process.exit(); }, 100)
-    }
-    if (!finish) {
-      console.log('Mongodb', alias, 'OFF');
-    }
-  }
+  registerShutdownHandlersOnce();
 
+  pendingConnections[url] = new Promise((resolve, reject) => {
+    MongoBase.MongoClient.connect(url + uriparams, (err, client) => {
+      delete pendingConnections[url];
+      if (err) { console.log('ERRO MDB -- TRACKING -- ', err); return reject(err); }
+      MDB.status[url] = true;
+      MDB.clients[url] = client;
+      resolve(client);
+    });
+  });
+
+  pendingConnections[url].then(callback, (err) => { if (onError) { onError(err); } });
 }
 
+var connectAsync = (alias, url) => new Promise((resolve, reject) => connect(alias, url, (client) => resolve(client), (err) => reject(err)));
+
 
 var stopCustom = (customCallback) => {
   MDB.stopCustom = true;

package/utils/querys.js

@@ -72,7 +72,7 @@ var querys = {
     // console.log("🚀 ~ listaOps", listaOps)
 
     db.collection(collection).bulkWrite(listaOps, (err, result) => {
-      if (err) { console.log('Erro bulkwrite cad: ', err); }
+      if (err) { console.log('Erro bulkwrite cad: ', err); return resolve({ status: false, err }); }
 
       var ids = result.getInsertedIds();
       ids.forEach(e => dataBuild[e.index]._reg.objId = e._id + '');
@@ -94,7 +94,7 @@ var querys = {
     var listaOps = mdb.genOpsUpdate(data, param);
     // console.log(listaOps)
     db.collection(collection).bulkWrite(listaOps, (err, result) => {
-      if (err) { console.log('Erro bulkwrite cad: ', err); }
+      if (err) { console.log('Erro bulkwrite cad: ', err); return resolve({ status: false, err }); }
       var result = result.result;
       resolve({ status: true, result });
     });
@@ -102,7 +102,7 @@ var querys = {
   'updateManyAsync':(db,collection,data) => new Promise(resolve=>{
     var listaOps = mdb.genOpsFull(data, 'save');
     db.collection(collection).bulkWrite(listaOps, (err, result) => {
-      if (err) { console.log('Erro bulkwrite cad: ', err); }
+      if (err) { console.log('Erro bulkwrite cad: ', err); return resolve({status:false,err}); }
       var ids = result.getInsertedIds();
       resolve({status:true,ids:ids});
     });