Reconnecting triggered with the latest release @4.6.2

[faljabi]

Description

I just upgraded redis ^4.5.1 → ^4.6.2 and I get extra reconnecting event.

const redis = require('redis');
const config = require('config');
const { loggers: { consoleLogger, errorLogger } } = require('../logging');

const client = redis.createClient({
    url: config.get('redis.url'),
    disableOfflineQueue: true,
    legacyMode: true
});

client.on('ready', _ => consoleLogger.info('🔥 redis ready'));
client.on('reconnecting', _ => consoleLogger.info('📞 redis reconnecting'));
client.on('end', _ => consoleLogger.warn('🔌 redis disconnected'));
client.on('error', error => {

    errorLogger.error('❌ redis error', { err: error });
});

client.connect();

module.exports = client;

The below is printed. I did not get this on the earlier versiob v4.5.1
[01:55:11 AM] INFO 🔥 redis ready
[01:55:11 AM] INFO 📞 redis reconnecting

Node.js Version

v19.4

Redis Server Version

No response

Node Redis Version

No response

Platform

macOS

Logs

No response

[leibale]

Tnx for reporting! It's not actually reconnecting, I accidentally moved the emit outside the catch block, so it's emitting reconnecting even when there is no error (see the fix here). Releasing a new version now...

[leibale]

redis@4.6.3/@redis/client@1.5.4 is on npm 🎉