always clear the connectTimeout

[mcollina]

I have seen the close of my processes being severely delayed by this.
I'm not sure how to test this though.

[ronag]

I'm surprised the client-unref test doesn't catch this? Maybe that could be improved?

[codecov-commenter]

Codecov Report

Merging #896 (bf5681c) into main (0391411) will decrease coverage by 0.09%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #896      +/-   ##
==========================================
- Coverage   99.57%   99.48%   -0.10%     
==========================================
  Files          26       26              
  Lines        2131     2132       +1     
==========================================
- Hits         2122     2121       -1     
- Misses          9       11       +2     

Impacted Files	Coverage Δ
lib/core/connect.js	100.00% <100.00%> (ø)
lib/client.js	99.34% <0.00%> (-0.27%)	⬇️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0391411...bf5681c. Read the comment docs.

[mcollina]

@ronag I've updated this.