@onlineapps/conn-infra-mq 1.1.4 → 1.1.6

package/package.json

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

package/src/transports/rabbitmqClient.js

@@ -163,19 +163,21 @@ class RabbitMQClient extends EventEmitter {
         // Use central config for infrastructure queues
         const queueOptions = this._getQueueOptions(queue, { durable });
         
-        // Try to check if queue exists first (to avoid PRECONDITION-FAILED if it exists with different args)
+        // Try to assert queue with our config
+        // If it fails with 406 (PRECONDITION-FAILED), queue exists with different args - use it as-is
+        // IMPORTANT: Don't try to re-assert after 406, as it will close the channel
         try {
-          await this._channel.checkQueue(queue);
-          // Queue exists - use it as-is (don't try to assert with different args)
-          // This prevents 406 PRECONDITION-FAILED errors
-        } catch (checkErr) {
-          // Queue doesn't exist (404) - create it with our config
-          if (checkErr.code === 404) {
-            await this._channel.assertQueue(queue, queueOptions);
+          await this._channel.assertQueue(queue, queueOptions);
+        } catch (assertErr) {
+          // If queue exists with different arguments (406), use it as-is without re-asserting
+          // Re-asserting would close the channel, so we just log and proceed to consume
+          if (assertErr.code === 406) {
+            console.warn(`[RabbitMQClient] Queue ${queue} exists with different arguments, using as-is (skipping assert to avoid channel close):`, assertErr.message);
+            // Don't try to re-assert - just proceed to consume
+            // The queue exists, we just can't change its arguments
           } else {
-            // Other error (e.g., 406) - queue exists with different args, use it as-is
-            // Log warning but proceed to consume
-            console.warn(`[RabbitMQClient] Queue ${queue} exists with different arguments, using as-is:`, checkErr.message);
+            // Other error (404 shouldn't happen with assertQueue, but handle it)
+            throw assertErr;
           }
         }
       }