@autofleet/rabbit 4.0.2-beta.8 → 4.1.0-beta.0

package/dist/index.js

@@ -1,18 +1,18 @@
 "use strict";
+/* eslint-disable no-empty,consistent-return,no-async-promise-executor,@typescript-eslint/no-unused-vars */
 var __importDefault = (this && this.__importDefault) || function (mod) {
     return (mod && mod.__esModule) ? mod : { "default": mod };
 };
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.sendCeleryTaskViaHttp = void 0;
-/* eslint-disable no-empty,consistent-return,no-async-promise-executor,@typescript-eslint/no-unused-vars,no-param-reassign */
 const events_1 = require("events");
 const util_1 = require("util");
 const moment_1 = __importDefault(require("moment"));
 const redis_lock_1 = __importDefault(require("redis-lock"));
 const amqp_connection_manager_1 = require("amqp-connection-manager");
-const exponential_backoff_1 = require("exponential-backoff");
 const zehut_1 = require("@autofleet/zehut");
 const node_crypto_1 = require("node:crypto");
+const axios_1 = __importDefault(require("axios"));
 const logger_1 = __importDefault(require("./logger"));
 const rabbitError_1 = __importDefault(require("./lib/rabbitError"));
 const redis_1 = __importDefault(require("./lib/redis"));
@@ -22,8 +22,6 @@ const types_1 = require("./lib/types");
 // const debug = nodeDebug('af-rabbitmq')
 const debug = logger_1.default.debug.bind(logger_1.default);
 const PUBLISH_TIMEOUT = 1000 * 10;
-// TODO: [QUORUM-PHASE-3] Delete this env var
-const { DISABLE_QUORUM_QUEUES_CONSUME, DISABLE_QUORUM_QUEUES_PUBLISH, } = process.env;
 const HEARTBEAT = '60';
 class RabbitMq {
     static parseMsg(msg) {
@@ -62,52 +60,49 @@ class RabbitMq {
     constructor(options = {}, redisConfig) {
         this.DISCONNECT_MSG = 'rabbit: connection disconnect';
         this.RECONNECT_MSG = 'rabbit: connection disconnect - reconnecting';
-        this.consumersToRegister = [];
-        this.doesVHostExist = false;
-        this.vhost = 'quorum-vhost';
-        this.gracefulShutdownStarted = false;
-        this.oldConsumersToRegister = [];
+        this.consumers = [];
+        this.isVHostExist = false;
+        this.oldConsumers = [];
         this.assertVHost = async () => {
-            if (this.doesVHostExist) {
+            if (this.isVHostExist) {
                 return;
             }
-            const username = process.env.RABBITMQ_USERNAME || 'guest';
-            const password = process.env.RABBITMQ_PASSWORD || 'guest';
-            const credentials = Buffer.from(`${username}:${password}`).toString('base64');
-            const headers = {
-                Authorization: `Basic ${credentials}`,
-                'Content-Type': 'application/json',
+            if (!this.options?.vhost) {
+                throw new Error('vhost is not defined');
+            }
+            const { vhost } = this.options;
+            const auth = {
+                auth: {
+                    username: process.env.RABBITMQ_USERNAME || 'guest',
+                    password: process.env.RABBITMQ_PASSWORD || 'guest',
+                },
             };
-            const rabbitHost = `http://${(this.options?.rabbitHost || process.env.RABBITMQ_SERVICE_HOST || 'localhost').split(':')[0]}:15672`;
-            const url = `${rabbitHost}/api/vhosts/${encodeURIComponent(this.vhost)}`;
+            const rabbitHost = `http://${(this.options.rabbitHost || process.env.RABBITMQ_SERVICE_HOST || 'localhost').split(':')[0]}:15672`;
             try {
-                const response = await fetch(url, {
-                    method: 'GET',
-                    headers,
-                });
-                if (response.status === 200) {
-                    this.doesVHostExist = true;
-                    logger_1.default.info('Vhost exists', { vhost: this.vhost });
-                    return;
-                }
-                if (response.status !== 404) {
-                    logger_1.default.error('Failed to check vhost', { response });
-                    throw new rabbitError_1.default('Failed to check vhost');
+                const response = await axios_1.default.get(`${rabbitHost}/api/vhosts/${encodeURIComponent(vhost)}`, auth);
+                if (response.status !== 200) {
+                    throw new Error('Failed to check vhost');
                 }
-                const createResponse = await fetch(url, {
-                    method: 'PUT',
-                    headers,
-                    body: JSON.stringify({ default_queue_type: 'quorum' }),
-                });
-                if (!createResponse.ok) {
-                    logger_1.default.error('Failed to create vhost', { response: createResponse });
-                    throw new rabbitError_1.default('Failed to create vhost');
-                }
-                this.doesVHostExist = true;
-                logger_1.default.info('Vhost created', { vhost: this.vhost });
+                this.isVHostExist = true;
+                logger_1.default.info('Vhost exists', { vhost });
             }
             catch (error) {
-                logger_1.default.error('Failed to check or create vhost', { error });
+                // If the vhost does not exist, create it
+                if (error instanceof axios_1.default.AxiosError && error.response?.status === 404) {
+                    try {
+                        await axios_1.default.put(`${rabbitHost}/api/vhosts/${encodeURIComponent(vhost)}`, {
+                            default_queue_type: 'quorum',
+                        }, auth);
+                        this.isVHostExist = true;
+                        logger_1.default.info('Vhost created', { vhost: this.options?.vhost });
+                        return;
+                    }
+                    catch (createError) {
+                        logger_1.default.error('Failed to create vhost', { error: createError });
+                        throw createError;
+                    }
+                }
+                logger_1.default.error('Failed to check vhost', { error });
                 throw error;
             }
         };
@@ -140,11 +135,11 @@ class RabbitMq {
             await this.unlockRedisIfNeeded(releaseLock);
             if (channel && msg) {
                 if (!skipRetry
-                    && (!msg.properties.headers?.[consts_1.RETRY_HEADER]
+                    && (!msg.properties.headers[consts_1.RETRY_HEADER]
                         || parseInt(msg.properties.headers[consts_1.RETRY_HEADER], 10) < options.retries)) {
                     await this.sendToQueue(queue, RabbitMq.parseMsg(msg).content, options, {
                         ...msg.properties.headers,
-                        [consts_1.RETRY_HEADER]: msg.properties.headers?.[consts_1.RETRY_HEADER]
+                        [consts_1.RETRY_HEADER]: msg.properties.headers[consts_1.RETRY_HEADER]
                             ? msg.properties.headers[consts_1.RETRY_HEADER] + 1
                             : 1,
                     });
@@ -153,7 +148,7 @@ class RabbitMq {
                     const deadQueue = `${queue}-dead`;
                     await this.sendToQueue(deadQueue, RabbitMq.parseMsg(msg).content, deadQueueOptions, {
                         ...msg.properties.headers,
-                        [consts_1.RETRY_HEADER]: msg.properties.headers?.[consts_1.RETRY_HEADER]
+                        [consts_1.RETRY_HEADER]: msg.properties.headers[consts_1.RETRY_HEADER]
                             ? msg.properties.headers[consts_1.RETRY_HEADER] + 1
                             : 1,
                     });
@@ -167,46 +162,25 @@ class RabbitMq {
                 });
             }
         };
-        this.maskURL = (url) => {
-            try {
-                const urlObj = new URL(url);
-                urlObj.username = '***';
-                urlObj.password = '***';
-                return urlObj.toString();
-            }
-            catch {
-                return url;
-            }
-        };
         this.em = new events_1.EventEmitter();
-        this.publishChannel = null;
+        this.channel = null;
         this.publishChannelSetupPromise = null;
-        this.publishConnection = {
-            amqpConnection: null,
-            creatingConnection: false,
-            connectionCreatedEventName: 'publishConnectionCreated',
-            connectionFailedEventName: 'publishConnectionFailed',
-            blockReconnect: false,
-        };
-        this.consumeConnection = {
-            amqpConnection: null,
-            creatingConnection: false,
-            connectionCreatedEventName: 'consumeConnectionCreated',
-            connectionFailedEventName: 'consumeConnectionFailed',
-            blockReconnect: false,
-        };
+        this.connection = null;
         this.creatingConnection = false;
         this.exchanges = {};
         this.queues = {};
         this.queueSetupPromises = {};
         this.assertExchangePromises = {};
-        this.consumersToRegister = [];
+        this.consumers = [];
         this.options = options;
+        if (!options?.vhost) {
+            this.options.vhost = process.env.VHOST_NAME || 'quorum-vhost';
+        }
         this.redisClient = redisConfig && (0, redis_1.default)(redisConfig);
         if (this.redisClient) {
             this.redisLock = (0, util_1.promisify)((0, redis_lock_1.default)(this.redisClient));
         }
-        this.consumersTags = new Map();
+        this.consumersTags = [];
         logger_1.default.info(`rabbit: [gracefully-shutdown] adding gracefully shutdown for process.pid ${process.pid}`);
         if (!this.options?.dontGracefulShutdown) {
             process.on('SIGTERM', async () => {
@@ -216,57 +190,43 @@ class RabbitMq {
                 await this.gracefulShutdown('SIGINT');
             });
         }
-        // TODO: [QUORUM-PHASE-3] Delete the old properties that we use for the old consumers and publishers
+        // TODO: DELETE OLD PROPERTIES
         this.oldEm = new events_1.EventEmitter();
-        this.oldPublishChannel = null;
+        this.oldChannel = null;
         this.oldPublishChannelSetupPromise = null;
-        this.oldPublishConnection = {
-            amqpConnection: null,
-            creatingConnection: false,
-            connectionCreatedEventName: 'oldPublishConnectionCreated',
-            connectionFailedEventName: 'oldPublishConnectionFailed',
-            blockReconnect: false,
-        };
-        this.oldConsumeConnection = {
-            amqpConnection: null,
-            creatingConnection: false,
-            connectionCreatedEventName: 'oldConsumeConnectionCreated',
-            connectionFailedEventName: 'oldConsumeConnectionFailed',
-            blockReconnect: false,
-        };
+        this.oldConnection = null;
         this.oldCreatingConnection = false;
         this.oldExchanges = {};
         this.oldQueues = {};
         this.oldQueueSetupPromises = {};
         this.oldAssertExchangePromises = {};
-        this.oldConsumersToRegister = [];
-        this.oldConsumersTags = new Map();
+        this.oldConsumers = [];
+        this.oldConsumersTags = [];
     }
-    async getConnection(connection) {
+    async getConnection() {
         return new Promise(async (resolve, reject) => {
-            const { amqpConnection: connectionLocal, creatingConnection, connectionCreatedEventName, connectionFailedEventName, blockReconnect, } = connection;
-            if (blockReconnect) {
+            if (this.blockReconnect) {
                 debug('rabbit: block reconnect');
                 // eslint-disable-next-line @typescript-eslint/ban-ts-comment
                 // @ts-ignore
                 return resolve();
             }
-            if (connectionLocal !== null) {
-                if (this.options?.disableReconnect || connectionLocal?.isConnected()) {
+            if (this.connection !== null) {
+                if (this.options?.disableReconnect || this.connection?.isConnected()) {
                     debug('rabbit: connection - is connected');
                     // eslint-disable-next-line @typescript-eslint/ban-ts-comment
                     // @ts-ignore
-                    return resolve(connectionLocal);
+                    return resolve(this.connection);
                 }
                 debug('rabbit: connection - reconnecting');
             }
-            if (creatingConnection) {
+            if (this.creatingConnection) {
                 debug('rabbit: creating connection emi');
-                this.em.once(connectionCreatedEventName, resolve);
-                this.em.once(connectionFailedEventName, reject);
+                this.em.once(consts_1.CONNECTION_CREATED_CONST, resolve);
+                this.em.once(consts_1.CONNECTION_FAILED_CONST, reject);
                 return;
             }
-            connection.creatingConnection = true;
+            this.creatingConnection = true;
             let isResolved = false;
             // It is import to use it as a function and not as a variable
             // because of k8s changes the env variables
@@ -276,63 +236,61 @@ class RabbitMq {
                 const password = process.env.RABBITMQ_PASSWORD || 'guest';
                 const host = this.options?.rabbitHost || process.env.RABBITMQ_SERVICE_HOST || 'localhost';
                 debug('rabbit: creating connection', { host, userName, HEARTBEAT });
-                return [`amqp://${userName}:${password}@${host}/${this.vhost}?heartbeat=${HEARTBEAT}`];
+                return [`amqp://${userName}:${password}@${host}/${this.options?.vhost}?heartbeat=${HEARTBEAT}`];
             };
             const defaultUrls = findServers();
-            const newConnection = await (0, amqp_connection_manager_1.connect)(defaultUrls, {
+            const connection = await (0, amqp_connection_manager_1.connect)(defaultUrls, {
                 findServers,
             });
-            connection.amqpConnection = newConnection;
-            newConnection.on('error', (err) => {
+            this.connection = connection;
+            this.connection.on('error', (err) => {
                 logger_1.default.error('rabbit: connection error', { err });
                 if (!isResolved) {
                     isResolved = true;
                     reject(err);
-                    this.em.emit(connectionFailedEventName, err);
+                    this.em.emit(consts_1.CONNECTION_FAILED_CONST, err);
                 }
             });
-            newConnection.on('connectFailed', (err) => {
-                this.consumersTags.clear();
-                if (typeof err.url === 'string') {
-                    err.url = this.maskURL(err.url);
-                }
-                logger_1.default.error('rabbit: connection connectFailed', { err, advice: 'Check if the vhost exist', vhost: this.vhost });
+            this.connection.on('connectFailed', (err) => {
+                this.consumersTags = [];
+                logger_1.default.error('rabbit: connection connectFailed', { err, advice: 'Check if the vhost exist', vhost: this.options?.vhost });
                 if (!isResolved) {
                     isResolved = true;
                     reject(err);
-                    this.em.emit(connectionFailedEventName, err);
+                    this.em.emit(consts_1.CONNECTION_FAILED_CONST, err);
                 }
             });
-            newConnection.on('disconnect', ({ err }) => {
-                this.consumersTags.clear();
+            this.connection.on('disconnect', ({ err }) => {
+                // this.channel = null;
+                this.consumersTags = [];
                 debug('rabbit: connection closed');
                 if (this.options?.disableReconnect) {
                     logger_1.default.error(`${this.DISCONNECT_MSG}${err && ` - ${err}`}`);
-                    connection.blockReconnect = true;
+                    this.blockReconnect = true;
                 }
                 else {
                     logger_1.default.error(`${this.RECONNECT_MSG}${err && ` - ${err}`}`);
                 }
             });
-            newConnection.once('connect', async () => {
+            this.connection.once('connect', async () => {
                 debug('rabbit: connection established');
-                connection.creatingConnection = false;
-                this.em.emit(connectionCreatedEventName, newConnection);
+                this.creatingConnection = false;
+                this.em.emit(consts_1.CONNECTION_CREATED_CONST, connection);
                 isResolved = true;
-                resolve(newConnection);
+                resolve(connection);
             });
         });
     }
-    async getNewChannel({ name = (0, utils_1.rand)().toString(), onClose = null, options = {}, connection, }) {
-        let localConnection;
+    async getNewChannel({ name = (0, utils_1.rand)().toString(), onClose = null, options = {} } = {}) {
+        let connection;
         try {
-            localConnection = await this.getConnection(connection);
+            connection = await this.getConnection();
         }
         catch (e) {
             logger_1.default.error(`rabbit: error on get connection for new channel ${name} `, { e });
             throw e;
         }
-        const channel = localConnection?.createChannel({ ...options });
+        const channel = connection.createChannel({ ...options });
         (0, events_1.once)(channel, 'close').then((args) => {
             logger_1.default.error(`rabbit: channel ${name} closed`);
             onClose?.(args);
@@ -347,20 +305,18 @@ class RabbitMq {
             throw err;
         }
     }
-    async assertChannel({ force = false, connection }) {
+    async assertChannel({ force = false } = {}) {
         if (!this.publishChannelSetupPromise) {
             this.publishChannelSetupPromise = new Promise(async (resolve, reject) => {
-                if (this.publishChannel && !force) {
-                    return resolve(this.publishChannel);
+                if (this.channel && !force) {
+                    return resolve(this.channel);
                 }
                 try {
-                    const channel = await this.getNewChannel({ connection });
+                    const channel = await this.getNewChannel({});
                     channel.on('error', (err) => {
                         logger_1.default.error('rabbit: channel error', { err });
                     });
-                    if (this.publishConnection === connection) {
-                        this.publishChannel = channel;
-                    }
+                    this.channel = channel;
                     resolve(channel);
                 }
                 catch (e) {
@@ -370,8 +326,8 @@ class RabbitMq {
         }
         return this.publishChannelSetupPromise;
     }
-    async assertExchange(exchangeName, connection) {
-        const channel = await this.assertChannel({ connection });
+    async assertExchange(exchangeName, options) {
+        const channel = await this.assertChannel();
         if (this.exchanges[exchangeName]) {
             delete this.assertExchangePromises[exchangeName];
             return this.exchanges[exchangeName];
@@ -385,29 +341,28 @@ class RabbitMq {
     }
     async getQueueLength(queue) {
         RabbitMq.validateName('queue', queue);
-        const { publishChannel: channel } = this;
+        const { channel } = this;
         if (!channel) {
-            throw new rabbitError_1.default('channel is not defined');
+            throw new Error('channel is not defined');
         }
-        debug('rabbit: getting queue length', { queue, connected: this.publishConnection.amqpConnection?.isConnected() });
+        debug('rabbit: getting queue length', { queue, connected: this.connection?.isConnected() });
         return channel?.checkQueue(queue);
     }
-    async deleteQueue(queue, connection) {
+    async deleteQueue(queue) {
         RabbitMq.validateName('queue', queue);
-        const channel = await this.assertChannel({ connection });
+        const channel = await this.assertChannel();
         logger_1.default.info('rabbit: deleting queue', { queue });
         const deleteQueueRes = await channel.deleteQueue(queue);
         debug('queue deleted', deleteQueueRes);
         return deleteQueueRes;
     }
     async bindQueue(queue, exchange) {
-        const channel = await this.assertChannel({ connection: this.publishConnection });
+        const channel = await this.assertChannel();
         await channel.addSetup((setupChannel) => setupChannel.bindQueue(queue, exchange, ''));
         return channel.bindQueue(queue, exchange, '');
     }
     async setupQueue(queueName, options) {
         let queue;
-        const connection = this.publishConnection;
         const localeOptions = {
             ...options,
             durable: true,
@@ -418,7 +373,7 @@ class RabbitMq {
             },
         };
         try {
-            const channel = await this.assertChannel({ connection });
+            const channel = await this.assertChannel();
             debug('assertQueue->channel.addSetup', { queueName });
             await channel.addSetup(async (setupChannel) => {
                 await setupChannel.assertQueue(queueName, localeOptions);
@@ -430,8 +385,8 @@ class RabbitMq {
             logger_1.default.error('rabbit: assertQueue error', { queueName, options, error: e });
             if (!this.options?.dontRetryAssert) {
                 debug('retrying assertQueue', { queueName });
-                const channel = await this.assertChannel({ force: true, connection });
-                await this.deleteQueue(queueName, connection);
+                const channel = await this.assertChannel({ force: true });
+                await this.deleteQueue(queueName);
                 debug('retrying assertQueue->channel.addSetup', { queueName });
                 await channel.addSetup((setupChannel) => setupChannel.assertQueue(queueName, localeOptions));
                 debug('retrying assertQueue->channel.assertQueue', { queueName });
@@ -444,7 +399,7 @@ class RabbitMq {
         this.queues[queueName] = queue;
         return queue;
     }
-    // TODO: [QUORUM-PHASE-3] Can be deleted after deleting the old consumers and publishers because all the queues are created us quorum queues
+    // TODO: Can be deleted after deleting the old consumers
     static shouldUseQuorum(queueName) {
         const envQuorumQueuesWhitelist = process.env.QUORUM_QUEUES_WHITELIST;
         if (envQuorumQueuesWhitelist === '*') {
@@ -469,10 +424,10 @@ class RabbitMq {
         return this.queueSetupPromises[queueName];
     }
     saveConsumer(queue, callback, options) {
-        const isConsumerExist = this.consumersToRegister.some((consumer) => consumer.queue === queue);
+        const isConsumerExist = this.consumers.some((consumer) => consumer.queue === queue);
         if (!isConsumerExist) {
             logger_1.default.info(`rabbit: consumer: ${queue} saved in consumer array`);
-            this.consumersToRegister.push({
+            this.consumers.push({
                 queue,
                 callback,
                 options,
@@ -481,21 +436,18 @@ class RabbitMq {
     }
     // Used by the microservices to consume messages from the queue
     async consume(queue, callback, options) {
-        this.saveConsumerOld(queue, callback, options);
-        // TODO: [QUORUM-PHASE-3] Use only the implementation of consumeNew and delete consumeNew and consumeOld
-        if (options?.isQuorumQueue !== false && DISABLE_QUORUM_QUEUES_CONSUME !== 'true') {
-            this.saveConsumer(queue, callback, options);
-            const backoffConfig = (0, utils_1.getAssertVhostExponentialBackoffConfig)();
-            await (0, exponential_backoff_1.backOff)(() => this.assertVHost(), backoffConfig);
+        // TODO: USE THE IMPLEMENTATION OF THE NEW CONSUME AND DELETE THE NEW AND OLD
+        if (options?.isQuorumQueue !== false) {
+            await this.assertVHost();
             await this.consumeNew(queue, callback, options);
         }
         await this.consumeOld(queue, callback, options);
     }
-    // TODO: [QUORUM-PHASE-3] Delete consumeNew we do not use it anymore
+    // TODO: DELETE
     async consumeNew(queue, callback, options) {
         await this.consumeFromRabbit(queue, callback, options);
     }
-    // TODO: [QUORUM-PHASE-3] Delete consumeOld we do not use it anymore
+    // TODO: DELETE
     async consumeOld(queue, callback, options) {
         await this.consumeFromRabbitOld(queue, callback, options);
     }
@@ -516,15 +468,16 @@ class RabbitMq {
     async consumeFromRabbit(queue, callback, options) {
         const optionsWithDefaults = { ...consts_1.DEFAULT_OPTIONS, ...options };
         RabbitMq.validateName('queue', queue);
+        this.saveConsumer(queue, callback, options);
         const uniqueId = (0, node_crypto_1.randomUUID)();
         const { limit, deadMessageTtl, useConsumeWithLock, lockTimeout, auditContext, enableRabbitTrace, } = optionsWithDefaults;
         if (useConsumeWithLock) {
             if (!this.redisLock) {
-                throw new rabbitError_1.default('Usage of consumeWithLock requires RedisInstance');
+                throw new Error('Usage of consumeWithLock requires RedisInstance');
             }
             logger_1.default.info(`rabbit: Consuming with lock from queue ${queue} with lockTimeout: ${lockTimeout}ms`);
         }
-        const channel = await this.getNewChannel({ connection: this.consumeConnection });
+        const channel = await this.getNewChannel({});
         return channel.addSetup(async (confirmChannel) => {
             const q = await this.assertQueue(queue, optionsWithDefaults);
             await confirmChannel.prefetch(limit, false);
@@ -598,7 +551,7 @@ class RabbitMq {
             }
             else {
                 logger_1.default.info(`rabbit: adding tag ${consumerTag} to the array.`);
-                this.consumersTags.set(queue, { channel: confirmChannel, consumerTag });
+                this.consumersTags.push([confirmChannel, consumerTag]);
             }
         });
     }
@@ -608,15 +561,10 @@ class RabbitMq {
         RabbitMq.validateName('exchange', exchange);
         RabbitMq.validateName('queue', queue);
         const { limit, deadMessageTtl } = optionsWithDefaults;
-        // TODO: [QUORUM-PHASE-3] Delete the if statement after all the queues are created as quorum queues
-        if (options?.isQuorumQueue !== false && DISABLE_QUORUM_QUEUES_CONSUME !== 'true') {
+        if (options?.isQuorumQueue !== false) {
+            await this.assertVHost();
             await this.saveConsumer(queue, callback, options);
-            const backoffConfig = (0, utils_1.getAssertVhostExponentialBackoffConfig)();
-            await (0, exponential_backoff_1.backOff)(() => this.assertVHost(), backoffConfig);
-            const channel = await this.getNewChannel({
-                name: `consume-exchange-${exchange}-queue-${queue}`,
-                connection: this.consumeConnection,
-            });
+            const channel = await this.getNewChannel({ name: `consume-exchange-${exchange}-queue-${queue}` });
             await channel.addSetup(async (c) => {
                 const assertExchange = await (0, utils_1.assertExchangeFanout)(c, exchange);
                 await c.assertQueue(queue);
@@ -628,12 +576,9 @@ class RabbitMq {
                 ]);
             });
         }
-        // TODO: [QUORUM-PHASE-3] Delete the old implementation
+        // TODO: DELETE OLD
         await this.saveConsumerOld(queue, callback, options);
-        const channelOld = await this.getNewChannelOld({
-            name: `consume-exchange-${exchange}-queue-${queue}-old`,
-            connection: this.oldConsumeConnection,
-        });
+        const channelOld = await this.getNewChannelOld({ name: `consume-exchange-${exchange}-queue-${queue}-old` });
         await channelOld.addSetup(async (c) => {
             const assertExchange = await (0, utils_1.assertExchangeFanout)(c, exchange);
             await c.assertQueue(queue);
@@ -646,32 +591,32 @@ class RabbitMq {
         });
     }
     // Used by the microservices to publish messages to the exchange
-    // TODO: [QUORUM-PHASE-3] isQuorumQueue flag is not needed anymore because all the queues are created as quorum queues
     async publish(exchange, content, customHeaders, isQuorumQueue = true) {
-        if (isQuorumQueue && DISABLE_QUORUM_QUEUES_PUBLISH !== 'true') {
+        // TODO: DELETE THE IF
+        if (isQuorumQueue) {
             return (0, utils_1.wrapSetImmediate)(async () => {
                 await this.assertVHost();
                 RabbitMq.validateName('exchange', exchange);
-                const channel = await this.assertChannel({ connection: this.publishConnection });
-                await this.assertExchange(exchange, this.publishConnection);
+                const channel = await this.assertChannel();
+                await this.assertExchange(exchange);
                 await channel.publish(exchange, '', Buffer.from(JSON.stringify(content)), RabbitMq.getPublishOptions(customHeaders));
             });
         }
-        // TODO: [QUORUM-PHASE-3] Delete the old implementation
+        // TODO: DELETE THE OLD
         return (0, utils_1.wrapSetImmediate)(async () => {
             RabbitMq.validateName('exchange', exchange);
-            const channel = await this.assertChannelOld({ connection: this.oldPublishConnection });
-            await this.assertExchangeOld(exchange, this.oldPublishConnection);
+            const channel = await this.assertChannelOld();
+            await this.assertExchangeOld(exchange);
             await channel.publish(exchange, '', Buffer.from(JSON.stringify(content)), RabbitMq.getPublishOptions(customHeaders));
         });
     }
     // Used by the microservices to send messages to the queue
-    // TODO: [QUORUM-PHASE-3] isQuorumQueue flag is not needed anymore because all the queues are created as quorum queues
     async sendToQueue(queue, content, options, customHeaders, isQuorumQueue = true) {
-        if (isQuorumQueue && DISABLE_QUORUM_QUEUES_PUBLISH !== 'true') {
+        // TODO: DELETE THE IF AND THE OLD
+        if (isQuorumQueue) {
             try {
                 await this.assertVHost();
-                await this.assertChannel({ connection: this.publishConnection });
+                await this.assertChannel();
             }
             catch (e) {
                 logger_1.default.error(`rabbit sendToQueue: failed to send assert channel when sending to queue ${queue}`, { e });
@@ -686,7 +631,7 @@ class RabbitMq {
                 throw e;
             }
             try {
-                const res = await this.publishChannel?.sendToQueue(queue, Buffer.from(JSON.stringify(content)), RabbitMq.getPublishOptions(customHeaders));
+                const res = await this.channel?.sendToQueue(queue, Buffer.from(JSON.stringify(content)), RabbitMq.getPublishOptions(customHeaders));
                 debug(`rabbit: sending to queue ${queue}`, { res });
                 return res;
             }
@@ -698,85 +643,62 @@ class RabbitMq {
         }
         else {
             try {
-                await this.assertChannelOld({ connection: this.oldPublishConnection });
+                await this.assertChannelOld();
             }
             catch (e) {
                 logger_1.default.error(`rabbit sendToQueue: failed to send assert channel when sending to queue ${queue}`, { e });
                 throw e;
             }
-            try {
-                RabbitMq.validateName('queue', queue);
-                await this.assertQueueOld(queue, options);
-            }
-            catch (e) {
-                logger_1.default.error(`rabbit sendToQueue: failed to assert queue ${queue}`, { e });
-                throw e;
-            }
-            try {
-                const res = await this.oldPublishChannel?.sendToQueue(queue, Buffer.from(JSON.stringify(content)), RabbitMq.getPublishOptions(customHeaders));
-                debug(`rabbit: sending to queue ${queue}`, { res });
-                return res;
-            }
-            catch (e) {
-                const isConnected = await this.isConnectedOld();
-                logger_1.default.error(`rabbit sendToQueue: failed to send to queue ${queue}, isConnected: ${isConnected}`, { e });
-                throw e;
-            }
+        }
+        try {
+            RabbitMq.validateName('queue', queue);
+            await this.assertQueueOld(queue, options);
+        }
+        catch (e) {
+            logger_1.default.error(`rabbit sendToQueue: failed to assert queue ${queue}`, { e });
+            throw e;
+        }
+        try {
+            const res = await this.oldChannel?.sendToQueue(queue, Buffer.from(JSON.stringify(content)), RabbitMq.getPublishOptions(customHeaders));
+            debug(`rabbit: sending to queue ${queue}`, { res });
+            return res;
+        }
+        catch (e) {
+            const isConnected = await this.isConnected();
+            logger_1.default.error(`rabbit sendToQueue: failed to send to queue ${queue}, isConnected: ${isConnected}`, { e });
+            throw e;
         }
     }
     async isConnected() {
-        let isConnected = true;
-        // TODO:[QUORUM-PHASE-3] Remove the condition
-        if (!this.gracefulShutdownStarted) {
-            if (DISABLE_QUORUM_QUEUES_CONSUME !== 'true' || DISABLE_QUORUM_QUEUES_PUBLISH !== 'true') {
-                debug('rabbit: start is connected');
-                const [consumeConnection, publishConnection] = await Promise.all([
-                    this.getConnection(this.consumeConnection),
-                    this.getConnection(this.publishConnection),
-                ]);
-                isConnected = consumeConnection.isConnected() && publishConnection.isConnected();
-                if (!isConnected) {
-                    logger_1.default.error('rabbit: isConnected - false');
-                    return false;
-                }
-                try {
-                    const unRegisteredConsumers = this.consumersToRegister.filter((c) => {
-                        const doesConsumerExist = this.consumersTags.get(c.queue);
-                        return !doesConsumerExist;
-                    });
-                    if (unRegisteredConsumers.length > 0) {
-                        const queueNames = unRegisteredConsumers.map((c) => c.queue);
-                        logger_1.default.error('rabbit: found unregistered consumers for queues', {
-                            count: queueNames.length,
-                            queues: queueNames,
-                        });
-                        throw new rabbitError_1.default('Found unregistered consumers');
-                    }
-                    const channel = await this.assertChannel({ connection: this.publishConnection });
-                    await channel.waitForConnect();
-                    await Promise.all(this.consumersToRegister.map((c) => channel.checkQueue(c.queue)));
-                }
-                catch (e) {
-                    logger_1.default.error('rabbit: isConnected - false', { msg: e.message });
-                    return false;
-                }
-            }
-            // TODO:[QUORUM-PHASE-3] Delete the old is connected and adjust the return value to return true
-            isConnected = await this.isConnectedOld();
+        const connection = await this.getConnection();
+        const isConnected = connection.isConnected();
+        if (!isConnected) {
+            logger_1.default.error('rabbit: isConnected - false');
+            return false;
+        }
+        const channel = await this.assertChannel();
+        try {
+            await Promise.all([
+                channel.waitForConnect(),
+                ...this.consumers.map((c) => channel.checkQueue(c.queue)),
+            ]);
+        }
+        catch (e) {
+            logger_1.default.error('rabbit: isConnected - false');
+            return false;
         }
-        logger_1.default.debug(`rabbit: isConnected - ${isConnected}`);
-        return isConnected;
+        logger_1.default.info('rabbit: isConnected - true');
+        return true;
     }
     async gracefulShutdown(signal) {
-        // TODO: [QUORUM-PHASE-3] Delete the old implementation they are not needed anymore
-        this.gracefulShutdownStarted = true;
-        const tagsNumber = this.consumersTags.size + this.oldConsumersTags.size;
+        // TODO: DELETE OLD
+        const tagsNumber = this.consumersTags.length + this.oldConsumersTags.length;
         logger_1.default.info(`rabbit: [gracefully-shutdown] received ${signal}! canceling #${tagsNumber} tags...`);
-        const cancelTagPromises = [...this.consumersTags].map(([_, { channel, consumerTag }]) => channel.cancel(consumerTag));
-        const cancelTagPromisesOld = [...this.oldConsumersTags].map(([_, { channel, consumerTag }]) => channel.cancel(consumerTag));
+        const cancelTagPromises = this.consumersTags.map(([channel, tag]) => channel.cancel(tag));
+        const cancelTagPromisesOld = this.oldConsumersTags.map(([channel, tag]) => channel.cancel(tag));
         // Clean the array to avoid race
-        this.consumersTags.clear();
-        this.oldConsumersTags.clear();
+        this.consumersTags = [];
+        this.oldConsumersTags = [];
         const results = await Promise.allSettled([...cancelTagPromises, ...cancelTagPromisesOld]);
         const rejected = results.filter((p) => p.status === 'rejected');
         if (rejected.length > 0) {
@@ -789,15 +711,16 @@ class RabbitMq {
     async consumeFromRabbitOld(queue, callback, options) {
         const optionsWithDefaults = { ...consts_1.DEFAULT_OPTIONS, ...options };
         RabbitMq.validateName('queue', queue);
+        this.saveConsumerOld(queue, callback, options);
         const uniqueId = (0, node_crypto_1.randomUUID)();
         const { limit, deadMessageTtl, useConsumeWithLock, lockTimeout, auditContext, enableRabbitTrace, } = optionsWithDefaults;
         if (useConsumeWithLock) {
             if (!this.redisLock) {
-                throw new rabbitError_1.default('Usage of consumeWithLock requires RedisInstance');
+                throw new Error('Usage of consumeWithLock requires RedisInstance');
             }
             logger_1.default.info(`rabbit: Consuming with lock from queue ${queue} with lockTimeout: ${lockTimeout}ms`);
         }
-        const channel = await this.getNewChannelOld({ connection: this.oldConsumeConnection });
+        const channel = await this.getNewChannelOld({});
         return channel.addSetup(async (confirmChannel) => {
             const q = await this.assertQueueOld(queue, optionsWithDefaults);
             await confirmChannel.prefetch(limit, false);
@@ -867,28 +790,25 @@ class RabbitMq {
                 }
             }, types_1.CONSUMER_DEFAULT_OPTIONS);
             if (!consumerTag) {
-                logger_1.default.error(`rabbit: failed to consume from queue ${queue} old`);
+                logger_1.default.error(`rabbit: failed to consume from queue ${queue}`);
             }
             else {
-                logger_1.default.info(`rabbit: adding tag ${consumerTag} to the array old`);
-                this.oldConsumersTags.set(queue, { channel: confirmChannel, consumerTag });
+                logger_1.default.info(`rabbit: adding tag ${consumerTag} to the array.`);
+                this.oldConsumersTags.push([confirmChannel, consumerTag]);
             }
         });
     }
-    // TODO: [QUORUM-PHASE-3] Delete all the function under this line.
-    async getNewChannelOld({ name = (0, utils_1.rand)().toString(), onClose = null, options = {}, connection, }) {
-        let localConnection;
+    // TODO: DELETE OLD PROPERTIES UNDER THIS LINE
+    async getNewChannelOld({ name = (0, utils_1.rand)().toString(), onClose = null, options = {} } = {}) {
+        let connection;
         try {
-            localConnection = await this.getConnectionOld(connection);
+            connection = await this.getConnectionOld();
         }
         catch (e) {
             logger_1.default.error(`rabbit: error on get connection for new channel ${name} `, { e });
             throw e;
         }
-        if (!localConnection) {
-            throw new Error(`rabbit: couldnt get connection for new channel ${name}`);
-        }
-        const channel = localConnection?.createChannel({ ...options });
+        const channel = connection.createChannel({ ...options });
         (0, events_1.once)(channel, 'close').then((args) => {
             logger_1.default.error(`rabbit: channel ${name} closed`);
             onClose?.(args);
@@ -903,31 +823,30 @@ class RabbitMq {
             throw err;
         }
     }
-    async getConnectionOld(connection) {
+    async getConnectionOld() {
         return new Promise(async (resolve, reject) => {
-            const { amqpConnection: connectionLocal, creatingConnection, connectionCreatedEventName, connectionFailedEventName, blockReconnect, } = connection;
-            if (blockReconnect) {
+            if (this.oldBlockReconnect) {
                 debug('rabbit: block reconnect');
                 // eslint-disable-next-line @typescript-eslint/ban-ts-comment
                 // @ts-ignore
                 return resolve();
             }
-            if (connectionLocal !== null) {
-                if (this.options?.disableReconnect || connectionLocal?.isConnected()) {
+            if (this.oldConnection !== null) {
+                if (this.options?.disableReconnect || this.oldConnection?.isConnected()) {
                     debug('rabbit: connection - is connected');
                     // eslint-disable-next-line @typescript-eslint/ban-ts-comment
                     // @ts-ignore
-                    return resolve(connectionLocal);
+                    return resolve(this.oldConnection);
                 }
                 debug('rabbit: connection - reconnecting');
             }
-            if (creatingConnection) {
+            if (this.oldCreatingConnection) {
                 debug('rabbit: creating connection emi');
-                this.oldEm.once(connectionCreatedEventName, resolve);
-                this.oldEm.once(connectionFailedEventName, reject);
+                this.oldEm.once(consts_1.CONNECTION_CREATED_CONST, resolve);
+                this.oldEm.once(consts_1.CONNECTION_FAILED_CONST, reject);
                 return;
             }
-            connection.creatingConnection = true;
+            this.oldCreatingConnection = true;
             let isResolved = false;
             // It is import to use it as a function and not as a variable
             // because of k8s changes the env variables
@@ -940,55 +859,52 @@ class RabbitMq {
                 return [`amqp://${userName}:${password}@${host}?heartbeat=${HEARTBEAT}`];
             };
             const defaultUrls = findServers();
-            const newConnection = await (0, amqp_connection_manager_1.connect)(defaultUrls, {
+            const connection = await (0, amqp_connection_manager_1.connect)(defaultUrls, {
                 findServers,
             });
-            connection.amqpConnection = newConnection;
-            newConnection.on('error', (err) => {
+            this.oldConnection = connection;
+            this.oldConnection.on('error', (err) => {
                 logger_1.default.error('rabbit: connection error', { err });
                 if (!isResolved) {
                     isResolved = true;
                     reject(err);
-                    this.oldEm.emit(connectionFailedEventName, err);
+                    this.oldEm.emit(consts_1.CONNECTION_FAILED_CONST, err);
                 }
             });
-            newConnection.on('connectFailed', (err) => {
-                this.oldConsumersTags.clear();
-                if (typeof err.url === 'string') {
-                    err.url = this.maskURL(err.url);
-                }
+            this.oldConnection.on('connectFailed', (err) => {
+                this.oldConsumersTags = [];
                 logger_1.default.error('rabbit: connection connectFailed', { err });
                 if (!isResolved) {
                     isResolved = true;
                     reject(err);
-                    this.oldEm.emit(connectionFailedEventName, err);
+                    this.oldEm.emit(consts_1.CONNECTION_FAILED_CONST, err);
                 }
             });
-            newConnection.on('disconnect', ({ err }) => {
-                this.oldConsumersTags.clear();
+            this.oldConnection.on('disconnect', ({ err }) => {
+                this.oldConsumersTags = [];
                 debug('rabbit: connection closed');
                 if (this.options?.disableReconnect) {
                     logger_1.default.error(`${this.DISCONNECT_MSG}${err && ` - ${err}`}`);
-                    connection.blockReconnect = true;
+                    this.oldBlockReconnect = true;
                 }
                 else {
                     logger_1.default.error(`${this.RECONNECT_MSG}${err && ` - ${err}`}`);
                 }
             });
-            newConnection.once('connect', async () => {
+            this.oldConnection.once('connect', async () => {
                 debug('rabbit: connection established');
-                connection.creatingConnection = false;
-                this.oldEm.emit(connectionCreatedEventName, newConnection);
+                this.oldCreatingConnection = false;
+                this.oldEm.emit(consts_1.CONNECTION_CREATED_CONST, connection);
                 isResolved = true;
-                resolve(newConnection);
+                resolve(connection);
             });
         });
     }
     saveConsumerOld(queue, callback, options) {
-        const isConsumerExist = this.oldConsumersToRegister.some((consumer) => consumer.queue === queue);
+        const isConsumerExist = this.oldConsumers.some((consumer) => consumer.queue === queue);
         if (!isConsumerExist) {
             logger_1.default.info(`rabbit: consumer: ${queue} saved in consumer array`);
-            this.oldConsumersToRegister.push({
+            this.oldConsumers.push({
                 queue,
                 callback,
                 options,
@@ -1009,7 +925,6 @@ class RabbitMq {
     }
     async setupQueueOld(queueName, options) {
         let queue;
-        const connection = this.oldPublishConnection;
         const shouldUseQuorum = RabbitMq.shouldUseQuorum(queueName);
         const localeOptions = {
             ...options,
@@ -1021,7 +936,7 @@ class RabbitMq {
             },
         };
         try {
-            const channel = await this.assertChannelOld({ connection });
+            const channel = await this.assertChannelOld();
             debug('assertQueue->channel.addSetup', { queueName });
             await channel.addSetup((setupChannel) => setupChannel.assertQueue(queueName, localeOptions));
             debug('assertQueue->channel.assertQueue', { queueName });
@@ -1031,8 +946,8 @@ class RabbitMq {
             logger_1.default.error('rabbit: assertQueue error', { queueName, options, error: e });
             if (!this.options?.dontRetryAssert) {
                 debug('retrying assertQueue', { queueName });
-                const channel = await this.assertChannelOld({ force: true, connection });
-                await this.deleteQueueOld(queueName, connection);
+                const channel = await this.assertChannelOld({ force: true });
+                await this.deleteQueueOld(queueName);
                 debug('retrying assertQueue->channel.addSetup', { queueName });
                 await channel.addSetup((setupChannel) => setupChannel.assertQueue(queueName, localeOptions));
                 debug('retrying assertQueue->channel.assertQueue', { queueName });
@@ -1045,20 +960,18 @@ class RabbitMq {
         this.oldQueues[queueName] = queue;
         return queue;
     }
-    async assertChannelOld({ force = false, connection }) {
+    async assertChannelOld({ force = false } = {}) {
         if (!this.oldPublishChannelSetupPromise) {
             this.oldPublishChannelSetupPromise = new Promise(async (resolve, reject) => {
-                if (this.oldPublishChannel && !force) {
-                    return resolve(this.oldPublishChannel);
+                if (this.oldChannel && !force) {
+                    return resolve(this.oldChannel);
                 }
                 try {
-                    const channel = await this.getNewChannelOld({ connection });
+                    const channel = await this.getNewChannelOld({});
                     channel.on('error', (err) => {
                         logger_1.default.error('rabbit: channel error', { err });
                     });
-                    if (this.oldPublishConnection === connection) {
-                        this.oldPublishChannel = channel;
-                    }
+                    this.oldChannel = channel;
                     resolve(channel);
                 }
                 catch (e) {
@@ -1068,16 +981,16 @@ class RabbitMq {
         }
         return this.oldPublishChannelSetupPromise;
     }
-    async deleteQueueOld(queue, connection) {
+    async deleteQueueOld(queue) {
         RabbitMq.validateName('queue', queue);
-        const channel = await this.assertChannelOld({ connection });
+        const channel = await this.assertChannelOld();
         logger_1.default.info('rabbit: deleting queue', { queue });
         const deleteQueueRes = await channel.deleteQueue(queue);
         debug('queue deleted', deleteQueueRes);
         return deleteQueueRes;
     }
-    async assertExchangeOld(exchangeName, connection) {
-        const channel = await this.assertChannelOld({ connection });
+    async assertExchangeOld(exchangeName, options) {
+        const channel = await this.assertChannelOld();
         if (this.oldExchanges[exchangeName]) {
             delete this.oldAssertExchangePromises[exchangeName];
             return this.oldExchanges[exchangeName];
@@ -1089,44 +1002,7 @@ class RabbitMq {
         this.oldExchanges[exchangeName] = await this.oldAssertExchangePromises[exchangeName];
         return this.oldExchanges[exchangeName];
     }
-    async isConnectedOld() {
-        if (!this.gracefulShutdownStarted) {
-            debug('rabbit: start is connected old');
-            const [oldConsumeConnection, oldPublishConnection] = await Promise.all([
-                this.getConnectionOld(this.oldConsumeConnection),
-                this.getConnectionOld(this.oldPublishConnection),
-            ]);
-            const isConnected = oldConsumeConnection.isConnected() && oldPublishConnection.isConnected();
-            if (!isConnected) {
-                logger_1.default.error('rabbit: isConnected old - false');
-                return false;
-            }
-            try {
-                const unRegisteredConsumersOld = this.consumersToRegister.filter((c) => {
-                    const doesConsumerExist = this.consumersTags.get(c.queue);
-                    return !doesConsumerExist;
-                });
-                if (unRegisteredConsumersOld.length > 0) {
-                    const queueNames = unRegisteredConsumersOld.map((c) => c.queue);
-                    logger_1.default.error('rabbit: found unregistered consumers for queues old', {
-                        count: queueNames.length,
-                        queues: queueNames,
-                    });
-                    throw new rabbitError_1.default('Found unregistered consumers old');
-                }
-                const channelOld = await this.assertChannelOld({ connection: this.oldPublishConnection });
-                await channelOld.waitForConnect();
-                await Promise.all(this.oldConsumersToRegister.map((c) => channelOld.checkQueue(c.queue)));
-            }
-            catch (e) {
-                logger_1.default.error('rabbit: isConnected - false old', { msg: e.message });
-                return false;
-            }
-        }
-        return true;
-    }
 }
 exports.default = RabbitMq;
 var celery_1 = require("./lib/celery");
 Object.defineProperty(exports, "sendCeleryTaskViaHttp", { enumerable: true, get: function () { return celery_1.sendCeleryTaskViaHttp; } });
-//# sourceMappingURL=index.js.map