@onlineapps/mq-client-core 1.0.6 → 1.0.8

package/package.json

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

package/src/BaseClient.js

@@ -146,7 +146,14 @@ class BaseClient {
     if (typeof noAck === 'boolean') consumeOptions.noAck = noAck;
     if (options.queueOptions) consumeOptions.queueOptions = options.queueOptions;
 
+    // CRITICAL: Log all consume() calls for transparency
+    console.log(`[BaseClient] [CONSUMER] Starting consume() for queue: ${queue} at ${new Date().toISOString()}`);
+    console.log(`[BaseClient] [CONSUMER] Options:`, JSON.stringify(consumeOptions, null, 2));
+    console.log(`[BaseClient] [CONSUMER] Transport type: ${this._transport?.constructor?.name || 'unknown'}`);
+    console.log(`[BaseClient] [CONSUMER] Connected: ${this._connected}`);
+
     try {
+      const consumeStartTime = Date.now();
       await this._transport.consume(
         queue,
         async (msg) => {
@@ -162,7 +169,10 @@ class BaseClient {
         },
         consumeOptions
       );
+      const consumeEndTime = Date.now();
+      console.log(`[BaseClient] [CONSUMER] ✓ consume() completed for queue: ${queue} (took ${consumeEndTime - consumeStartTime}ms)`);
     } catch (err) {
+      console.error(`[BaseClient] [CONSUMER] ✗ consume() failed for queue: ${queue}`, err.message);
       throw new ConsumeError(`Failed to start consumer for queue "${queue}"`, queue, err);
     }
   }

package/src/transports/rabbitmqClient.js

@@ -203,29 +203,103 @@ class RabbitMQClient extends EventEmitter {
       // Skip assertQueue for reply queues (they're already created with specific settings)
       // Reply queues start with 'rpc.reply.' and are created as non-durable
       if (!queue.startsWith('rpc.reply.')) {
-        // Use queueChannel (regular channel) for queue operations to avoid RPC reply queue issues
-        // Simple queue assertion - infrastructure services should ensure queues exist
-        // If queue doesn't exist, assertQueue will create it with default options
-        // If it exists with different args (406), log warning and proceed
-        const queueOptions = options.queueOptions || { durable };
-        try {
-          await this._queueChannel.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 consume
-          } else {
-            // Other error - rethrow
-            throw assertErr;
+      // CRITICAL: Use queueConfig.js to get correct parameters (TTL, max-length, etc.)
+      // This prevents 406 PRECONDITION-FAILED errors from TTL mismatches
+      // Note: mq-client-core is for infrastructure services, but we need queueConfig from conn-infra-mq
+      // Try to load it - if it fails, we'll use defaults
+      let queueConfig = null;
+      try {
+        queueConfig = require('@onlineapps/conn-infra-mq/src/config/queueConfig');
+      } catch (requireErr) {
+        console.warn(`[RabbitMQClient] [mq-client-core] [CONSUMER] Cannot load queueConfig from @onlineapps/conn-infra-mq:`, requireErr.message);
+        console.warn(`[RabbitMQClient] [mq-client-core] [CONSUMER] Using default queue options (this may cause 406 errors if queue exists with different args)`);
+      }
+      const isInfraQueue = queueConfig.isInfrastructureQueue(queue);
+      const isBusinessQueue = queueConfig.isBusinessQueue(queue);
+      
+      let queueOptions = options.queueOptions || { durable };
+      
+      if (queueConfig) {
+        if (isInfraQueue) {
+          // Infrastructure queue - use central config
+          try {
+            const infraConfig = queueConfig.getInfrastructureQueueConfig(queue);
+            queueOptions = {
+              durable: infraConfig.durable !== false,
+              arguments: { ...infraConfig.arguments }
+            };
+            console.log(`[RabbitMQClient] [mq-client-core] [CONSUMER] Asserting infrastructure queue ${queue} with config from queueConfig`);
+          } catch (configErr) {
+            console.warn(`[RabbitMQClient] [mq-client-core] [CONSUMER] Infrastructure queue config not found for ${queue}, using default:`, configErr.message);
+          }
+        } else if (isBusinessQueue) {
+          // Business queue - use central config
+          try {
+            const parsed = queueConfig.parseBusinessQueue(queue);
+            if (parsed) {
+              const businessConfig = queueConfig.getBusinessQueueConfig(parsed.queueType, parsed.serviceName);
+              queueOptions = {
+                durable: businessConfig.durable !== false,
+                arguments: { ...businessConfig.arguments }
+              };
+              console.log(`[RabbitMQClient] [mq-client-core] [CONSUMER] Asserting business queue ${queue} with config from queueConfig`);
+            }
+          } catch (configErr) {
+            console.warn(`[RabbitMQClient] [mq-client-core] [CONSUMER] Business queue config not found for ${queue}, using default:`, configErr.message);
           }
         }
       }
+      
+      console.log(`[RabbitMQClient] [mq-client-core] [CONSUMER] Asserting queue ${queue} before consume() at ${new Date().toISOString()}`);
+      console.log(`[RabbitMQClient] [mq-client-core] [CONSUMER] Queue options:`, JSON.stringify(queueOptions, null, 2));
+      console.log(`[RabbitMQClient] [mq-client-core] [CONSUMER] _queueChannel state: exists=${!!this._queueChannel}, closed=${this._queueChannel ? this._queueChannel.closed : 'N/A'}`);
+      
+      // CRITICAL: Check if queue already exists with different arguments
+      // This can happen if sendToQueue() or amqplib's consume() auto-created it without TTL
+      try {
+        const queueInfo = await this._queueChannel.checkQueue(queue);
+        console.log(`[RabbitMQClient] [mq-client-core] [CONSUMER] Queue ${queue} already exists with arguments:`, JSON.stringify(queueInfo.messageCount !== undefined ? { messageCount: queueInfo.messageCount, consumerCount: queueInfo.consumerCount } : queueInfo));
+        // If queue exists, we need to check if arguments match
+        // Note: checkQueue() doesn't return arguments, so we'll try assertQueue() and catch 406
+      } catch (checkErr) {
+        if (checkErr.code === 404) {
+          console.log(`[RabbitMQClient] [mq-client-core] [CONSUMER] Queue ${queue} does not exist (404), will be created with options`);
+        } else {
+          console.warn(`[RabbitMQClient] [mq-client-core] [CONSUMER] checkQueue() failed for ${queue}:`, checkErr.message);
+        }
+      }
+      
+      try {
+        console.log(`[RabbitMQClient] [mq-client-core] [CONSUMER] About to call assertQueue(${queue}, ${JSON.stringify(queueOptions)})`);
+        await this._queueChannel.assertQueue(queue, queueOptions);
+        console.log(`[RabbitMQClient] [mq-client-core] [CONSUMER] ✓ Queue ${queue} asserted successfully`);
+      } catch (assertErr) {
+        // If queue exists with different arguments (406), this is a CRITICAL ERROR
+        // We should NOT proceed - the root cause must be fixed
+        if (assertErr.code === 406) {
+          console.error(`[RabbitMQClient] [mq-client-core] [CONSUMER] ✗ CRITICAL: Queue ${queue} exists with different arguments!`);
+          console.error(`[RabbitMQClient] [mq-client-core] [CONSUMER] Error:`, assertErr.message);
+          console.error(`[RabbitMQClient] [mq-client-core] [CONSUMER] Expected options:`, JSON.stringify(queueOptions, null, 2));
+          console.error(`[RabbitMQClient] [mq-client-core] [CONSUMER] This means assertQueue() was called without parameters somewhere else. Root cause must be fixed!`);
+          throw new Error(`Cannot assertQueue ${queue}: queue exists with different arguments. Root cause: assertQueue() was called without parameters. Fix the root cause instead of proceeding.`);
+        } else {
+          // Other error - rethrow
+          throw assertErr;
+        }
+      }
+      }
       // Set prefetch if provided
       if (typeof prefetch === 'number') {
         this._channel.prefetch(prefetch);
       }
 
+      // CRITICAL: Log before calling amqplib's consume()
+      // amqplib's consume() may internally call assertQueue() without parameters if queue doesn't exist
+      // This would create queue with default arguments (no TTL), causing 406 errors later
+      console.log(`[RabbitMQClient] [mq-client-core] [CONSUMER] About to call amqplib's channel.consume(${queue})`);
+      console.log(`[RabbitMQClient] [mq-client-core] [CONSUMER] ⚠ WARNING: amqplib's consume() may internally call assertQueue() WITHOUT parameters if queue doesn't exist`);
+      console.log(`[RabbitMQClient] [mq-client-core] [CONSUMER] ⚠ WARNING: We already asserted queue with correct parameters above - this should prevent auto-creation`);
+      
       await this._channel.consume(
         queue,
         async (msg) => {