@onlineapps/mq-client-core 1.0.3 → 1.0.5

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@onlineapps/mq-client-core",
-  "version": "1.0.3",
+  "version": "1.0.5",
   "description": "Core MQ client library for RabbitMQ - shared by infrastructure services and connectors",
   "main": "src/index.js",
   "scripts": {

package/src/transports/rabbitmqClient.js

@@ -122,22 +122,23 @@ class RabbitMQClient extends EventEmitter {
       // Ensure queue exists if publishing directly to queue and using default exchange
       if (!exchange) {
         // For infrastructure queues, they should already exist with specific arguments
-        // Try to check if queue exists first, if not, create with default options
-        // If it exists with different args (406), use it as-is
-        const queueOptions = options.queueOptions || { durable: this._config.durable };
+        // Use checkQueue instead of assertQueue to avoid 406 PRECONDITION-FAILED
+        // If queue doesn't exist (404), try to create it with default options
         try {
-          await this._channel.assertQueue(queue, queueOptions);
-        } catch (assertErr) {
-          // If queue exists with different arguments (406), use it as-is
-          if (assertErr.code === 406) {
-            console.warn(`[RabbitMQClient] Queue ${queue} exists with different arguments, using as-is:`, assertErr.message);
-            // Don't try to re-assert - just proceed to publish
+          await this._channel.checkQueue(queue);
+          // Queue exists - proceed to publish
+        } catch (checkErr) {
+          // If queue doesn't exist (404), create it with default options
+          if (checkErr.code === 404) {
+            const queueOptions = options.queueOptions || { durable: this._config.durable };
+            await this._channel.assertQueue(queue, queueOptions);
           } else {
             // Other error - rethrow
-            throw assertErr;
+            throw checkErr;
           }
         }
-        this._channel.sendToQueue(queue, buffer, { persistent, headers, routingKey });
+        // Publish to queue (works even if queue has different arguments)
+        this._channel.sendToQueue(queue, buffer, { persistent, headers });
       } else {
         // If exchange is specified, assert exchange and publish to it
         await this._channel.assertExchange(exchange, 'direct', { durable: this._config.durable });