Add TLS support with ConnectTLS function

[linalinn]

Hello,
I added the function ConnectTLS to enable go-zookeeper
to connect to TLS protected zookeepers.
Also I did refactor the Connect func to prevent code redundancy.

[coveralls]

Coverage decreased (-0.4%) to 79.239% when pulling f062bea on linalinn:tls into 2cc03de on samuel:master.

[linalinn]

I don't understand the DeepSource error.
Can someone please help me with that?

[msolo]

It looks like TLS can be implemented as a custom Dialer and injected via WithDialer(). That is a more minimal change that doesn't require any additional refactoring, just a relatively simple wrapper.

Given that TLS configs are often difficult to wire up correctly (particularly with a lot of complex reconnect code in the client here) including tests that cover TLS-enabled connections would improve confidence in the code change.

[linalinn]

So I fixed the DeeSource Problems.
But i can't add a test for Zookeeper with tls because i don't understand how the test is configuring the Zookeeper for a test.
And isn't WithDialer() makred as depracated? so implementing TLS over WithDialer()
would be wrong or not?

[linalinn]

Hello,
I have a problem with fixing this DeepSource fail.
I have tried it with adjusting the code like DeepSource suggested it.
Any suggestions how to solve this problem?