Fix #483: Unbind hangs in node v10

[jgeurts]

Fixes #483

Seems that when the socket is connected and unbind is called with a callback parameter, the callback is not executed.

[jgeurts]

@joyent/node-ldapjs Is there anything else I can do to get this merged?

[jvanalst]

Hmmm... failed tests. Darned.

[jgeurts]

The failed tests are unrelated to this fix, unless I’m mistaken

[jgeurts]

@melloc Can you please merge this and release an updated npm? This issue is preventing us from moving to node v10

[sunpeinju]

#497 maybe a safer replacement fix for this one. root cause of this issue has discussed with Node.js team (nodejs/node#24577), and #497 has passed ALL UTs on Node V10.

[jgeurts]

ldapts is a viable alternative to this library, that supports node v10+

[jsumners]

@jgeurts I appreciate that you added tests to this PR. But the overall suite still fails on Node 10. Does this PR address a different issue from #497? Or do they both attempt to solve the same issue?

[jsumners]

Thank you for contributing. I have merged #497 as the resolution. If you think that change did not address the same issue, or do so sufficiently, feel free to re-open.

[jsumners]

⚠️ This issue has been locked due to age. If you have encountered a recent
problem that seems to be covered by this issue, please open a new issue.

Please include a minimal reproducible example
when opening a new issue.