Connection duplicated after disconnect and connect about tmi.js

[AlcaDesign]

This is because you've added duplicate event handlers. Event handlers are not removed on disconnect.

[samyan]

You will eventually get this error message "(node) warning: possible EventEmitter memory leak detected. [...]" because that code will keep adding a new listener for the same message event every time "reconnect" is called.

async function reconnect() {
    await client.disconnect();
    await client.connect();
    client.on('message', messageHandler);
}
await reconnect();
// client._events.message is a function
await reconnect();
// client._events.message is now an array
console.log(client._events.message.length); // 2
await reconnect();
console.log(client._events.message.length); // 3
await reconnect();
console.log(client._events.message.length); // 4

Instead it should look more like this:

client.on('message', messageHandler);
async function reconnect() {
    await client.disconnect();
    await client.connect();
}

Thanks you for your response. I've tried in this way y seems to work:

// Desconectamos
await this.tmiClient.disconnect();
// Liberamos los triggers
this.tmiClient.removeAllListeners();

What do you think, is it good alternative for prevent duplication?

[AlcaDesign]

I think you're better off just moving when/where you add the listeners instead of removing all of them just to re-add them unnecessarily.

[AlcaDesign]

You will eventually get this error message "(node) warning: possible EventEmitter memory leak detected. [...]" because that code will keep adding a new listener for the same message event every time "reconnect" is called.

async function reconnect() {
    await client.disconnect();
    await client.connect();
    client.on('message', messageHandler);
}
await reconnect();
// client._events.message is a function
await reconnect();
// client._events.message is now an array
console.log(client._events.message.length); // 2
await reconnect();
console.log(client._events.message.length); // 3
await reconnect();
console.log(client._events.message.length); // 4

Instead it should look more like this:

client.on('message', messageHandler);
async function reconnect() {
    await client.disconnect();
    await client.connect();
}

[samyan]

Okey. Thank you very much!

[juliocbgomes]

This works for me! Thanks @samyan!!!

this.tmiClient.removeAllListeners();