_addedScriptHashesCleanInterval is not cleared if connection is closed by retryStrategy

[samimakicc]

_addedScriptHashesCleanInterval is not cleared If connection to redis is lost and retryStrategy is not a function or retryStrategy returns something else than a number.
This active timeout/interval causes node process unable to terminate when it should.

#1219 fixed that _addedScriptHashesCleanInterval is cleared when connection is manually closed with redis.disconnect() or redis.quit() but I feel it should also be cleared if the connection is closed by other means.

[shaharmor]

@samimakicc would you mind creating a PR?

[taratyn-tc]

I'm also experiencing and I can't figure out how to fix this. I'm happy to contribute a code fix, if I can get some advice how to fix it.

[marcbachmann]

We've fixed that in ioredis v5 by tracking the scripts per redis connection.