@onlineapps/conn-infra-mq 1.1.12 → 1.1.14

package/package.json

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

package/src/layers/QueueManager.js

@@ -82,18 +82,40 @@ class QueueManager {
 
     // Use checkQueue first to avoid 406 PRECONDITION-FAILED closing the channel
     // If queue doesn't exist (404), then assertQueue to create it
+    // IMPORTANT: Check if channel is still open before operations
+    if (!channel || channel.closed) {
+      throw new Error('Channel is closed - cannot ensure queue');
+    }
+    
     try {
-      await channel.checkQueue(queueName);
+      const queueInfo = await channel.checkQueue(queueName);
       // Queue exists - return queue info
-      return channel.checkQueue(queueName);
+      return queueInfo;
     } catch (checkErr) {
+      // Check if channel is still open before assertQueue
+      if (!channel || channel.closed) {
+        throw new Error('Channel closed during checkQueue - cannot create queue');
+      }
+      
       // If queue doesn't exist (404), create it with provided options
       if (checkErr.code === 404) {
-        return channel.assertQueue(queueName, queueOptions);
+        try {
+          return await channel.assertQueue(queueName, queueOptions);
+        } catch (assertErr) {
+          // If channel closed during assertQueue, throw descriptive error
+          if (!channel || channel.closed) {
+            throw new Error(`Channel closed during assertQueue for ${queueName} - likely due to RPC reply queue issue`);
+          }
+          throw assertErr;
+        }
       } else {
         // Other error (including 406) - queue exists with different args
         // Log warning and return queue info without asserting
         console.warn(`[QueueManager] Queue ${queueName} exists with different arguments, using as-is:`, checkErr.message);
+        // Check channel again before checkQueue
+        if (!channel || channel.closed) {
+          throw new Error('Channel closed - cannot check queue');
+        }
         return channel.checkQueue(queueName);
       }
     }

package/src/transports/rabbitmqClient.js

@@ -57,13 +57,25 @@ class RabbitMQClient extends EventEmitter {
         this.emit('error', new Error('RabbitMQ connection closed unexpectedly'));
       });
 
-      // Use ConfirmChannel to enable publisher confirms
+      // Use ConfirmChannel to enable publisher confirms for publish operations
+      // BUT: Create a separate regular channel for queue operations to avoid RPC reply queue issues
       this._channel = await this._connection.createConfirmChannel();
       this._channel.on('error', (err) => this.emit('error', err));
       this._channel.on('close', () => {
         // Emit a channel close error
         this.emit('error', new Error('RabbitMQ channel closed unexpectedly'));
       });
+      
+      // Create a separate regular channel for queue operations (assertQueue, checkQueue)
+      // This avoids RPC reply queue issues with ConfirmChannel.assertQueue()
+      this._queueChannel = await this._connection.createChannel();
+      this._queueChannel.on('error', (err) => {
+        // Log but don't emit - queue channel errors are less critical
+        console.warn('[RabbitMQClient] Queue channel error:', err.message);
+      });
+      this._queueChannel.on('close', () => {
+        console.warn('[RabbitMQClient] Queue channel closed');
+      });
     } catch (err) {
       // Cleanup partially created resources
       if (this._connection) {