@onlineapps/conn-infra-mq 1.1.43 → 1.1.45

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@onlineapps/conn-infra-mq",
-    "version": "1.1.43",
+    "version": "1.1.45",
     "description": "A promise-based, broker-agnostic client for sending and receiving messages via RabbitMQ",
     "main": "src/index.js",
     "repository": {

package/src/layers/QueueManager.js

@@ -365,24 +365,44 @@ class QueueManager {
         }
       }
       
-      // CRITICAL: Use assertQueue() DIRECTLY - it's idempotent and safer than checkQueue() + assertQueue()
-      // assertQueue() creates queue if it doesn't exist, or does nothing if it exists with same params
-      // If queue exists with different params (406), channel closes but that's OK - we accept the queue
-      let queueCreated = false;
+      // CRITICAL: Ensure channel is usable before assertQueue
+      // Recreate if closed (may have been closed by previous operations)
+      if (!channel || channel.closed) {
+        console.log(`[QueueManager] DEBUG: Channel is closed before assertQueue ${queueName}, recreating...`);
+        channel = await transport._connection.createChannel();
+        channel._createdAt = new Date().toISOString();
+        channel._closeReason = null;
+        channel._lastOperation = `Recreated before assertQueue ${queueName}`;
+        
+        channel.on('error', (err) => {
+          console.error('[RabbitMQClient] Recreated queue channel error:', err.message);
+          channel._closeReason = `Error: ${err.message} (code: ${err.code})`;
+        });
+        channel.on('close', () => {
+          console.error('[RabbitMQClient] Recreated queue channel closed');
+          console.error('[RabbitMQClient] Close reason:', channel._closeReason || 'Unknown');
+        });
+        
+        transport._queueChannel = channel;
+        console.log(`[QueueManager] DEBUG: Channel recreated before assertQueue ${queueName}`);
+      }
       
       // Track operation on channel for debugging
       if (channel && channel._lastOperation !== undefined) {
         channel._lastOperation = `About to assertQueue ${queueName}`;
       }
       
+      let queueCreated = false;
+      
       try {
-        console.log(`[QueueManager] DEBUG: About to assertQueue ${queueName} directly (idempotent)`);
+        console.log(`[QueueManager] DEBUG: About to assertQueue ${queueName} with correct arguments`);
         console.log(`[QueueManager] DEBUG: Channel state: exists=${!!channel}, closed=${channel ? (channel.closed === true ? 'true' : channel.closed === false ? 'false' : 'undefined') : 'N/A'}`);
+        console.log(`[QueueManager] DEBUG: Queue options:`, JSON.stringify(queueOptions));
         
         // CRITICAL: Use assertQueue() DIRECTLY - it's idempotent
         // If queue doesn't exist → creates it
         // If queue exists with same params → does nothing (OK)
-        // If queue exists with different params → 406, channel closes (but we accept the queue)
+        // If queue exists with different params → 406, channel closes (should not happen now, but handle it)
         await channel.assertQueue(queueName, queueOptions);
         
         // Queue created or already exists with same params
@@ -397,37 +417,59 @@ class QueueManager {
         console.log(`[QueueManager] DEBUG: assertQueue failed for ${queueName}:`, assertErr.message);
         console.log(`[QueueManager] DEBUG: Error code: ${assertErr.code}`);
         
-        // If 406 PRECONDITION-FAILED, queue exists with different args - accept it
-        // Channel is closed by server, but that's OK - queue exists and we can use it
+        // If 406 PRECONDITION-FAILED, queue exists with different args
+        // Delete it and recreate with correct args
         if (assertErr.code === 406) {
-          console.warn(`[QueueManager] Queue ${queueName} exists with different arguments (406), accepting as-is:`, assertErr.message);
-          queues[queueInfo.type] = queueName;
-          queueCreated = true;
-          console.info(`[QueueManager] ✓ Business queue exists (different args, accepted): ${queueName}`);
+          console.warn(`[QueueManager] Queue ${queueName} exists with different arguments (406), deleting and recreating...`);
+          console.warn(`[QueueManager] Expected args:`, JSON.stringify(queueOptions.arguments));
           
-          // Channel is closed, but we need to recreate it for next queue
-          // Don't throw error - queue exists, we just need new channel
+          // Delete queue and recreate with correct args
           try {
-            const newChannel = await transport._connection.createChannel();
-            newChannel._createdAt = new Date().toISOString();
-            newChannel._closeReason = null;
-            newChannel._lastOperation = `Recreated after 406 for ${queueName}`;
+            // Recreate channel (closed by 406)
+            channel = await transport._connection.createChannel();
+            channel._createdAt = new Date().toISOString();
+            channel._closeReason = null;
+            channel._lastOperation = `Recreated after 406 for ${queueName}`;
             
-            newChannel.on('error', (err) => {
+            channel.on('error', (err) => {
               console.error('[RabbitMQClient] Recreated queue channel error:', err.message);
-              newChannel._closeReason = `Error: ${err.message} (code: ${err.code})`;
+              channel._closeReason = `Error: ${err.message} (code: ${err.code})`;
             });
-            newChannel.on('close', () => {
+            channel.on('close', () => {
               console.error('[RabbitMQClient] Recreated queue channel closed');
-              console.error('[RabbitMQClient] Close reason:', newChannel._closeReason || 'Unknown');
+              console.error('[RabbitMQClient] Close reason:', channel._closeReason || 'Unknown');
             });
             
-            channel = newChannel;
             transport._queueChannel = channel;
-            console.log(`[QueueManager] DEBUG: Channel recreated after 406 for ${queueName}`);
-          } catch (recreateErr) {
-            // Even if channel recreation fails, queue exists - log warning but continue
-            console.warn(`[QueueManager] Failed to recreate channel after 406 for ${queueName}, but queue exists:`, recreateErr.message);
+            
+            // Delete queue
+            await channel.deleteQueue(queueName);
+            console.log(`[QueueManager] ✓ Deleted queue ${queueName} after 406 error`);
+            
+            // Recreate channel again (deleteQueue may close it)
+            channel = await transport._connection.createChannel();
+            channel._createdAt = new Date().toISOString();
+            channel._closeReason = null;
+            channel._lastOperation = `Recreated after deleteQueue ${queueName}`;
+            
+            channel.on('error', (err) => {
+              console.error('[RabbitMQClient] Recreated queue channel error:', err.message);
+              channel._closeReason = `Error: ${err.message} (code: ${err.code})`;
+            });
+            channel.on('close', () => {
+              console.error('[RabbitMQClient] Recreated queue channel closed');
+              console.error('[RabbitMQClient] Close reason:', channel._closeReason || 'Unknown');
+            });
+            
+            transport._queueChannel = channel;
+            
+            // Try assertQueue again
+            await channel.assertQueue(queueName, queueOptions);
+            console.log(`[QueueManager] ✓ Recreated queue ${queueName} with correct arguments`);
+            queues[queueInfo.type] = queueName;
+            queueCreated = true;
+          } catch (retryErr) {
+            throw new Error(`Failed to recreate queue ${queueName} after 406 error: ${retryErr.message}`);
           }
         } else {
           // Other error - critical, cannot continue