@bkmj/node-red-contrib-odbcmj 2.6.0 → 2.6.1

package/odbc.js

@@ -399,6 +399,11 @@ module.exports = function (RED) {
         // LOGIQUE PRINCIPALE (SAFE PATTERN & NON-BLOCKING UI)
         // =====================================================================
         this.on("input", async (msg, send, done) => {
+            // Initialisation du compteur de retry sur le message s'il n'existe pas
+            if (!msg.hasOwnProperty('_odbc_retry_attempt')) {
+                msg._odbc_retry_attempt = 0;
+            }
+
             // Nettoyage état précédent
             if (this.retryTimer) { clearTimeout(this.retryTimer); this.retryTimer = null; }
             this.isAwaitingRetry = false;
@@ -469,7 +474,9 @@ module.exports = function (RED) {
                     this.poolNode.resetPool(); 
                 }
 
-                done(); // Succès
+                // Succès : On peut retirer le compteur de l'objet msg (optionnel) ou le laisser
+                delete msg._odbc_retry_attempt;
+                done(); 
 
             } catch (err) {
                 errorOccurred = err;
@@ -479,7 +486,11 @@ module.exports = function (RED) {
                     || (err.odbcErrors && err.odbcErrors.length > 0)
                 );
 
-                if (isConnectionError || this.poolNode.config.retryFreshConnection) {
+                // CORRECTION BUG BOUCLE INFINIE :
+                // On planifie un retry SEULEMENT si :
+                // 1. C'est une erreur de connexion OU l'option retryFresh est active
+                // 2. ET on n'a pas encore fait de retry pour ce message (compteur < 1)
+                if ((isConnectionError || this.poolNode.config.retryFreshConnection) && msg._odbc_retry_attempt < 1) {
                     shouldScheduleRetry = true;
                 }
             } finally {
@@ -502,13 +513,17 @@ module.exports = function (RED) {
                 const retryDelay = parseInt(this.poolNode.config.retryDelay, 10);
 
                 if (shouldScheduleRetry && retryDelay > 0) {
-                    this.warn(`Query failed (${errorOccurred.message}). Retrying in ${retryDelay}s...`);
+                    this.warn(`Query failed (${errorOccurred.message}). Retrying once in ${retryDelay}s...`);
                     this.status({ fill: "red", shape: "ring", text: `Retry in ${retryDelay}s` });
                     
                     this.isAwaitingRetry = true;
                     this.retryTimer = setTimeout(() => {
                         this.isAwaitingRetry = false;
-                        this.log("Retry timer expired. Re-processing message.");
+                        
+                        // Incrémenter le compteur de tentatives AVANT de réémettre
+                        msg._odbc_retry_attempt++;
+                        
+                        this.log(`Retry timer expired. Re-processing message (Attempt ${msg._odbc_retry_attempt + 1}).`);
                         this.emit("input", msg, send, done); 
                     }, retryDelay * 1000);
                     

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bkmj/node-red-contrib-odbcmj",
-  "version": "2.6.0",
+  "version": "2.6.1",
   "description": "A powerful Node-RED node to connect to any ODBC data source, with connection pooling, advanced retry logic, and result streaming.",
   "keywords": [
     "node-red",