Fix tls.Config copy that breaks synchronization with internal sync types. Closes nsqio#254

Author: iaburton

conn.go

@@ -385,17 +385,17 @@ func (c *Conn) identify() (*IdentifyResponse, error) {
 
 func (c *Conn) upgradeTLS(tlsConf *tls.Config) error {
 	// create a local copy of the config to set ServerName for this connection
-	var conf tls.Config
+	conf := new(tls.Config)
 	if tlsConf != nil {
-		conf = *tlsConf
+		conf = tlsConf.Clone()
 	}
 	host, _, err := net.SplitHostPort(c.addr)
 	if err != nil {
 		return err
 	}
 	conf.ServerName = host
 
-	c.tlsConn = tls.Client(c.conn, &conf)
+	c.tlsConn = tls.Client(c.conn, conf)
 	err = c.tlsConn.Handshake()
 	if err != nil {
 		return err