@bkmj/node-red-contrib-odbcmj 2.0.1 → 2.0.3

package/odbc.html

@@ -31,15 +31,13 @@
         oneditprepare: function () {
             var node = this;
 
-            // --- Logique pour les sections déroulantes ---
-            // On ajoute un écouteur de clic sur les en-têtes de section
+            // Logique pour les sections déroulantes
             $('.form-section-header').on('click', function() {
                 $(this).toggleClass('expanded');
-                // On fait basculer le conteneur d'options qui suit immédiatement l'en-tête
                 $(this).next('.form-section-content').slideToggle();
             });
 
-            // --- Logique pour le mode de connexion hybride ---
+            // Logique pour le mode de connexion hybride
             function toggleConnectionMode(mode) {
                 if (mode === 'structured') {
                     $(".config-mode-structured").show();
@@ -65,7 +63,7 @@
                 toggleDriverField($(this).val());
             }).trigger("change");
 
-            // --- Logiques pour les cases à cocher ---
+            // Logiques pour les cases à cocher
             $("#node-config-input-syntaxtick").on("change", function () {
                 $(".input-syntax").toggle(this.checked);
             }).trigger("change");
@@ -74,7 +72,7 @@
                 $(".retry-options").toggle(this.checked);
             }).trigger("change");
 
-            // --- Logique pour le bouton de test ---
+            // Logique pour le bouton de test
             $('#node-config-test-connection').on('click', function() {
                 var button = $(this);
                 var originalText = "Test Connection";
@@ -84,11 +82,6 @@
                 
                 var connectionMode = $("#node-config-input-connectionMode").val();
                 
-                // --- CORRECTION : Récupérer le bon mot de passe selon le mode ---
-                var password = (connectionMode === 'structured') 
-                    ? $("#node-config-input-password-structured").val() 
-                    : $("#node-config-input-password-string").val();
-
                 var configData = {
                     connectionMode: connectionMode,
                     dbType: $("#node-config-input-dbType").val(),
@@ -96,8 +89,9 @@
                     server: $("#node-config-input-server").val(),
                     database: $("#node-config-input-database").val(),
                     user: $("#node-config-input-user").val(),
-                    password: password,
-                    connectionString: $("#node-config-input-connectionString").val()
+                    connectionString: $("#node-config-input-connectionString").val(),
+                    // Le mot de passe n'est envoyé que si on est en mode structuré
+                    password: (connectionMode === 'structured') ? $("#node-config-input-password").val() : null
                 };
 
                 $.ajax({
@@ -128,6 +122,7 @@
         padding: 5px;
         border-bottom: 1px solid #ddd;
         margin-bottom: 10px;
+        user-select: none;
     }
     .form-section-header i.fa-caret-right {
         transition: transform 0.2s ease-in-out;
@@ -181,20 +176,15 @@
             <input type="text" id="node-config-input-user">
         </div>
         <div class="form-row">
-            <label for="node-config-input-password-structured"><i class="fa fa-lock"></i> Password</label>
-            <input type="password" id="node-config-input-password-structured">
+            <label for="node-config-input-password"><i class="fa fa-lock"></i> Password</label>
+            <input type="password" id="node-config-input-password">
         </div>
     </div>
     
     <div class="config-mode-string">
         <div class="form-row">
             <label for="node-config-input-connectionString"><i class="fa fa-font"></i> Connection String</label>
-            <input type="text" id="node-config-input-connectionString" placeholder="DRIVER={...};PWD={{{password}}};">
-        </div>
-        <div class="form-row">
-            <label for="node-config-input-password-string"><i class="fa fa-lock"></i> Password</label>
-            <input type="password" id="node-config-input-password-string">
-            <span class="form-tips">Use <code>{{{password}}}</code> placeholder in the string above.</span>
+            <input type="text" id="node-config-input-connectionString" placeholder="DRIVER={...};SERVER=...;UID=...;PWD=...;">
         </div>
     </div>
 
@@ -205,8 +195,8 @@
 
     <hr/>
     
-    <div class="form-section-header expanded"><h4><i class="fa fa-caret-right"></i> <i class="fa fa-sitemap"></i> Pool Options</h4></div>
-    <div class="form-section-content">
+    <div class="form-section-header"><h4><i class="fa fa-caret-right"></i> <i class="fa fa-sitemap"></i> Pool Options</h4></div>
+    <div class="form-section-content" style="display: none;">
         <div class="form-row">
             <label for="node-config-input-initialSize"><i class="fa fa-play"></i> Initial Size</label>
             <input type="number" id="node-config-input-initialSize" placeholder="5" />
@@ -235,8 +225,8 @@
         </div>
     </div>
     
-    <div class="form-section-header expanded"><h4><i class="fa fa-caret-right"></i> <i class="fa fa-exclamation-triangle"></i> Error Handling & Retry</h4></div>
-    <div class="form-section-content">
+    <div class="form-section-header"><h4><i class="fa fa-caret-right"></i> <i class="fa fa-exclamation-triangle"></i> Error Handling & Retry</h4></div>
+    <div class="form-section-content" style="display: none;">
         <div class="form-row">
             <label for="node-config-input-retryFreshConnection" style="width: auto;"><i class="fa fa-refresh"></i> Retry with fresh connection</label>
             <input type="checkbox" id="node-config-input-retryFreshConnection" style="display: inline-block; width: auto; vertical-align: top;" />
@@ -277,10 +267,9 @@
     </div>
 </script>
 
-
 <script type="text/javascript">
     RED.nodes.registerType("odbc", {
-        category: "storage-input",
+        category: "storage",
         color: "#89A5C0",
         defaults: {
             name: { value: "" },

package/odbc.js

@@ -120,14 +120,18 @@ module.exports = function (RED) {
         }
     });
 
+    // --- API ENDPOINT POUR LE BOUTON DE TEST (VERSION CORRIGÉE) ---
     RED.httpAdmin.post("/odbc_config/:id/test", RED.auth.needsPermission("odbc.write"), async function(req, res) {
         const tempConfig = req.body;
-        const tempCredentials = { password: tempConfig.password };
-        delete tempConfig.password;
 
+        // Cette fonction interne ne doit PAS interagir avec `res`.
+        // Elle retourne une chaîne ou lance une erreur.
         const buildTestConnectionString = () => {
-             if (tempConfig.connectionMode === 'structured') {
-                if (!tempConfig.dbType || !tempConfig.server) return res.status(400).send("Mode structuré : le type de BD et le serveur sont requis.");
+            if (tempConfig.connectionMode === 'structured') {
+                if (!tempConfig.dbType || !tempConfig.server) {
+                    // On lance une erreur au lieu d'envoyer une réponse.
+                    throw new Error("En mode structuré, le type de base de données et le serveur sont requis.");
+                }
                 let driver;
                 let parts = [];
                 switch (tempConfig.dbType) {
@@ -137,15 +141,19 @@ module.exports = function (RED) {
                     default: driver = tempConfig.driver || ''; break;
                 }
                 if(driver) parts.unshift(`DRIVER={${driver}}`);
-                parts.push(`SERVER=${tempConfig.server}`);
+                if (tempConfig.server) parts.push(`SERVER=${tempConfig.server}`);
                 if (tempConfig.database) parts.push(`DATABASE=${tempConfig.database}`);
                 if (tempConfig.user) parts.push(`UID=${tempConfig.user}`);
-                if (tempCredentials.password) parts.push(`PWD=${tempCredentials.password}`);
+                // Le mot de passe est géré dans le bloc try/catch principal
+                if (tempConfig.password) parts.push(`PWD=${tempConfig.password}`);
+
                 return parts.join(';');
-            } else {
+
+            } else { // 'string' mode
                 let connStr = tempConfig.connectionString || "";
-                if (tempCredentials.password && connStr.includes('{{{password}}}')) {
-                    connStr = connStr.replace('{{{password}}}', tempCredentials.password);
+                // Le mot de passe est supposé être dans la chaîne ici
+                if (!connStr) {
+                    throw new Error("La chaîne de connexion ne peut pas être vide.");
                 }
                 return connStr;
             }
@@ -153,14 +161,19 @@ module.exports = function (RED) {
 
         let connection;
         try {
+            // Le bloc try/catch gère maintenant TOUTES les erreurs.
             const testConnectionString = buildTestConnectionString();
-            if (!testConnectionString) return res.status(400).send("La chaîne de connexion est vide.");
+            
             connection = await odbcModule.connect(testConnectionString);
-            res.sendStatus(200);
+            res.sendStatus(200); // Succès, on envoie la seule et unique réponse.
+
         } catch (err) {
+            // Qu'il s'agisse d'une erreur de validation ou de connexion, on l'attrape ici.
             res.status(500).send(err.message || "Erreur inconnue durant le test.");
         } finally {
-            if (connection) await connection.close();
+            if (connection) {
+                await connection.close();
+            }
         }
     });
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bkmj/node-red-contrib-odbcmj",
-  "version": "2.0.1",
+  "version": "2.0.3",
   "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",