ssh "connect" event is not fired when connected after close() about ssh2-promise

[jeffrson]

There is a problem when closing the connection - the array deregister is not cleared (or otherwise modified) upon close.

Then, in getSSHConnection, the newly created sshConnection is not registered at all, because isRegistered thinks, the connection had already been registered, because the "unique id" has not changed.

@sanketbajoria
Would it be possible to fix it on short-term? I see three alternative solutions:

1. clear the array in SSH2Promise.close()
2. remove the element in disconnectCb
3. forcibly re-register in getSSHConnection, as soon as a new sshConnection is created.

This doesn't fix the strange sequence of "beforedisconnect", though.

[jeffrson]

BTW, I believe this is somehow related to 'ssh.close()' not really closing when called after connected the second time.

That's why I hesitate to report this fact as a new bug. What do you think?

[sanketbajoria]

@jeffrson
Thanks for reporting this. I will look into this. If you do find a solution. Please feel free to raise a PR

[oscarrobinson]

Is a fix for this coming? I've observed lots of weird behaviour around reconnects that seems to be related to the issues described here.

[dobby-tgc]

Any updates yet?