Clear connection timeout on error

[adam-nielsen]

If the connection gets unexpectedly dropped during the connection phase (in my case because the database is too busy) then the error handler doesn't cancel the connection timeout. This results in an 'unexpected disconnection' error, followed some seconds later by a 'connection timeout' error.

This prevents any user code from terminating while the second timeout handler is counting down, which in some environments (like AWS Lamba) where you pay by the second, will cost more money while the script sits there waiting for the now superfluous connection timeout to expire.

This PR cancels the connection timeout in the error handler, allowing scripts that wish to exit immediately to do so.

[charmander]

This looks right! Do you think you can add a test? (If not, that’s okay – I’ll be able to at some point.)

[brianc]

ohh nice good catch! @charmander you wanna wait on a test before merging & merge & write a test later?