n8n-nodes-chat2crm 0.1.16 → 0.1.18

package/dist/nodes/Chat2CrmTrigger/Chat2CrmTrigger.node.js

@@ -121,6 +121,10 @@ class Chat2CrmTrigger {
         }
         // Храним последние прочитанные ID для каждого stream
         const lastReadIds = new Map();
+        // ИСПРАВЛЕНИЕ: Добавляем флаг для предотвращения параллельных вызовов
+        let isReading = false;
+        // ИСПРАВЛЕНИЕ: Добавляем Set для отслеживания уже обработанных messageId
+        const processedMessageIds = new Set();
         // Собираем информацию о найденных streams для вывода
         const foundStreams = [];
         // Инициализируем lastReadIds для каждого stream и собираем информацию
@@ -242,87 +246,97 @@ class Chat2CrmTrigger {
         let isClosing = false;
         // Функция для чтения сообщений
         const readMessages = async () => {
-            // Проверяем, не закрывается ли триггер
-            if (isClosing) {
+            // ИСПРАВЛЕНИЕ: Предотвращаем параллельные вызовы
+            if (isReading || isClosing) {
                 return;
             }
-            for (const [db, dbStreams] of streamsByDb.entries()) {
-                // Проверяем снова перед каждой итерацией
-                if (isClosing) {
-                    break;
-                }
-                const redis = redisConnections.get(db);
-                // Читаем каждый stream отдельно через XREAD
-                // XREAD читает напрямую из stream, независимо от consumer groups
-                for (const stream of dbStreams) {
+            isReading = true;
+            try {
+                for (const [db, dbStreams] of streamsByDb.entries()) {
                     if (isClosing) {
                         break;
                     }
-                    try {
-                        const lastId = lastReadIds.get(stream) || '$';
-                        // Используем более короткий block time для возможности быстрого закрытия
-                        const blockTime = isClosing ? 0 : Math.min(block, MAX_BLOCK_TIME_FOR_RESPONSIVENESS_MS);
-                        // Используем XREAD чтобы читать все сообщения напрямую из stream
-                        // Это позволяет читать сообщения даже если они уже обработаны другими consumer'ами
-                        const messages = await redis.xread('COUNT', count, 'BLOCK', blockTime, 'STREAMS', stream, lastId);
-                        // Проверяем снова после блокирующей операции
+                    const redis = redisConnections.get(db);
+                    for (const stream of dbStreams) {
                         if (isClosing) {
-                            return;
+                            break;
                         }
-                        if (messages && Array.isArray(messages) && messages.length > 0) {
-                            // XREAD возвращает массив с одним элементом [streamName, messages]
-                            const streamData = messages[0];
-                            const [streamName, streamMessages] = streamData;
-                            if (Array.isArray(streamMessages) && streamMessages.length > 0) {
-                                // Обновляем lastReadId на последний прочитанный ID
-                                const lastMessageId = streamMessages[streamMessages.length - 1][0];
-                                lastReadIds.set(streamName, lastMessageId);
-                                for (const [messageId, fields] of streamMessages) {
-                                    if (isClosing) {
-                                        break;
-                                    }
-                                    const messageData = {};
-                                    // Parse fields
-                                    for (let j = 0; j < fields.length; j += 2) {
-                                        messageData[fields[j]] = fields[j + 1];
-                                    }
-                                    // Parse commands if it's a string
-                                    if (messageData.commands && typeof messageData.commands === 'string') {
-                                        try {
-                                            messageData.commands = JSON.parse(messageData.commands);
+                        try {
+                            const lastId = lastReadIds.get(stream) || '$';
+                            const blockTime = isClosing ? 0 : Math.min(block, MAX_BLOCK_TIME_FOR_RESPONSIVENESS_MS);
+                            const messages = await redis.xread('COUNT', count, 'BLOCK', blockTime, 'STREAMS', stream, lastId);
+                            if (isClosing) {
+                                return;
+                            }
+                            if (messages && Array.isArray(messages) && messages.length > 0) {
+                                const streamData = messages[0];
+                                const [streamName, streamMessages] = streamData;
+                                if (Array.isArray(streamMessages) && streamMessages.length > 0) {
+                                    // ИСПРАВЛЕНИЕ: Фильтруем уже обработанные сообщения
+                                    const newMessages = streamMessages.filter(([messageId]) => {
+                                        const messageKey = `${streamName}:${messageId}`;
+                                        if (processedMessageIds.has(messageKey)) {
+                                            return false; // Пропускаем уже обработанное сообщение
+                                        }
+                                        processedMessageIds.add(messageKey);
+                                        return true;
+                                    });
+                                    // ИСПРАВЛЕНИЕ: Обновляем lastReadId сразу после чтения, ДО обработки
+                                    const lastMessageId = streamMessages[streamMessages.length - 1][0];
+                                    lastReadIds.set(streamName, lastMessageId);
+                                    // ИСПРАВЛЕНИЕ: Обрабатываем только новые сообщения
+                                    for (const [messageId, fields] of newMessages) {
+                                        if (isClosing) {
+                                            break;
                                         }
-                                        catch (e) {
-                                            // Ignore parse errors
+                                        const messageData = {};
+                                        for (let j = 0; j < fields.length; j += 2) {
+                                            messageData[fields[j]] = fields[j + 1];
                                         }
+                                        if (messageData.commands && typeof messageData.commands === 'string') {
+                                            try {
+                                                messageData.commands = JSON.parse(messageData.commands);
+                                            }
+                                            catch (e) {
+                                                // Ignore parse errors
+                                            }
+                                        }
+                                        const executionData = {
+                                            json: {
+                                                messageId,
+                                                stream: streamName,
+                                                data: messageData,
+                                                commandList: messageData,
+                                            },
+                                        };
+                                        this.emit([[executionData]]);
+                                    }
+                                    // ИСПРАВЛЕНИЕ: Очищаем старые записи из Set для предотвращения утечки памяти
+                                    // Оставляем только последние 1000 обработанных ID
+                                    if (processedMessageIds.size > 1000) {
+                                        const entries = Array.from(processedMessageIds);
+                                        const toRemove = entries.slice(0, entries.length - 1000);
+                                        toRemove.forEach(id => processedMessageIds.delete(id));
                                     }
-                                    // Триггерим workflow с данными сообщения
-                                    const executionData = {
-                                        json: {
-                                            messageId,
-                                            stream: streamName,
-                                            data: messageData,
-                                            commandList: messageData,
-                                        },
-                                    };
-                                    this.emit([[executionData]]);
                                 }
                             }
                         }
-                    }
-                    catch (error) {
-                        // Игнорируем ошибки при закрытии
-                        if (isClosing && (error.message?.includes('Connection is closed') || error.message?.includes('disconnect'))) {
-                            return;
-                        }
-                        // Игнорируем ошибки для несуществующих streams
-                        if (error.message?.includes('Invalid stream ID') || error.message?.includes('no such key')) {
-                            continue;
+                        catch (error) {
+                            if (isClosing && (error.message?.includes('Connection is closed') || error.message?.includes('disconnect'))) {
+                                return;
+                            }
+                            if (error.message?.includes('Invalid stream ID') || error.message?.includes('no such key')) {
+                                continue;
+                            }
+                            console.error(`[Chat2Crm Trigger] Error reading stream ${stream} from Redis DB ${db}:`, error);
                         }
-                        // Логируем только критические ошибки
-                        console.error(`[Chat2Crm Trigger] Error reading stream ${stream} from Redis DB ${db}:`, error);
                     }
                 }
             }
+            finally {
+                // ИСПРАВЛЕНИЕ: Всегда сбрасываем флаг, даже при ошибке
+                isReading = false;
+            }
         };
         // Начинаем polling с более коротким интервалом
         let interval = null;

package/dist/nodes/Chat2CrmTrigger/Infra/StreamConfig.js

@@ -80,6 +80,12 @@ exports.STREAM_DEFINITIONS = [
         description: 'Outgoing messages from N8N (DB 1)',
         db: 1,
     },
+    {
+        name: 'N8N Chat Control',
+        value: 'N8N_CHAT_CONTROL',
+        description: 'Chat control messages for N8N (DB 1)',
+        db: 1,
+    },
 ];
 /**
  * Маппинг stream names к номерам баз данных
@@ -92,6 +98,7 @@ exports.STREAM_DB_MAP = {
     crm_outgoing_status: 1,
     N8N_INCOMING_MESSAGE: 1,
     N8N_OUTGOING_MESSAGE: 1,
+    N8N_CHAT_CONTROL: 1,
     // Поддержка старого формата (строчными) для обратной совместимости
     n8n_incoming_message: 1,
     n8n_outgoing_message: 1,

package/index.js

@@ -1,3 +1,69 @@
+// ========================================
+// PROXY CONFIGURATION - ЛОГИРОВАНИЕ ПРОКСИ
+// ========================================
+const proxyVars = {
+  'all_proxy': process.env.all_proxy,
+  'ALL_PROXY': process.env.ALL_PROXY,
+  'http_proxy': process.env.http_proxy,
+  'HTTP_PROXY': process.env.HTTP_PROXY,
+  'https_proxy': process.env.https_proxy,
+  'HTTPS_PROXY': process.env.HTTPS_PROXY,
+  'no_proxy': process.env.no_proxy,
+  'NO_PROXY': process.env.NO_PROXY,
+};
+
+// Находим активный прокси (lowercase имеет приоритет)
+const activeProxy = process.env.all_proxy || process.env.ALL_PROXY || 
+                    process.env.http_proxy || process.env.HTTP_PROXY || 
+                    process.env.https_proxy || process.env.HTTPS_PROXY;
+
+// Выводим информацию о прокси БОЛЬШИМИ БУКВАМИ, чтобы было видно
+console.log('\n');
+console.log('╔════════════════════════════════════════════════════════════════╗');
+console.log('║                    PROXY CONFIGURATION                         ║');
+console.log('╚════════════════════════════════════════════════════════════════╝');
+console.log('');
+
+if (activeProxy) {
+  // Маскируем пароль в логах
+  const maskedProxy = activeProxy.replace(/:\/\/[^:]+:[^@]+@/, '://***:***@');
+  
+  console.log('✅ PROXY IS CONFIGURED AND ACTIVE');
+  console.log('');
+  console.log('Active Proxy URL:', maskedProxy);
+  console.log('NO_PROXY:', process.env.no_proxy || process.env.NO_PROXY || 'not set');
+  console.log('');
+  console.log('All proxy environment variables:');
+  Object.entries(proxyVars).forEach(([key, value]) => {
+    if (value) {
+      const masked = key.toLowerCase().includes('proxy') && !key.toLowerCase().includes('no') 
+        ? value.replace(/:\/\/[^:]+:[^@]+@/, '://***:***@')
+        : value;
+      console.log(`  ${key.padEnd(15)} = ${masked}`);
+    }
+  });
+  console.log('');
+  console.log('⚠️  All HTTP/HTTPS requests from n8n nodes will use this proxy');
+  console.log('⚠️  Redis connections (TCP) do NOT use HTTP proxy');
+} else {
+  console.log('❌ NO PROXY CONFIGURED');
+  console.log('');
+  console.log('Available proxy environment variables (all empty):');
+  Object.keys(proxyVars).forEach(key => {
+    console.log(`  ${key.padEnd(15)} = ${proxyVars[key] || '(not set)'}`);
+  });
+  console.log('');
+  console.log('To enable proxy, set in .env file:');
+  console.log('  ALL_PROXY=http://user:pass@host:port');
+  console.log('  all_proxy=http://user:pass@host:port');
+}
+
+console.log('');
+console.log('╔════════════════════════════════════════════════════════════════╗');
+console.log('║              END OF PROXY CONFIGURATION                        ║');
+console.log('╚════════════════════════════════════════════════════════════════╝');
+console.log('\n');
+
 module.exports = {
   nodes: [
     require('./dist/nodes/Chat2CrmTrigger/Chat2CrmTrigger.node.js'),

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "n8n-nodes-chat2crm",
-    "version": "0.1.16",
+    "version": "0.1.18",
     "description": "n8n node for Chat2Crm Redis integration",
     "keywords": [
         "n8n-community-node-package",
@@ -11,7 +11,7 @@
     "main": "index.js",
     "scripts": {
         "build": "rm -rf dist && tsc && gulp build:icons",
-        "dev": "npm run build && npx n8n",
+        "dev": "npm run build && node -r dotenv/config -r ./proxy-logger.js node_modules/.bin/n8n",
         "dev:watch": "tsc --watch",
         "format": "prettier nodes credentials --write",
         "lint": "eslint \"nodes/**/*.ts\" \"credentials/**/*.ts\" \"package.json\"",