solve NewRegistry httpclient 'nil pointer dereference' bug

meiyang1990 · Feb 28, 2019 · d3f447a · d3f447a
1 parent b8f2092
commit d3f447a
Showing 1 changed file with 5 additions and 4 deletions.
diff --git a/registry/consul/consul.go b/registry/consul/consul.go
@@ -105,12 +105,13 @@ func configure(c *consulRegistry, opts ...registry.Option) {
 		}
 	}
 
+	if config.HttpClient == nil {
+		config.HttpClient = new(http.Client)
+	}
+
 	// requires secure connection?
 	if c.opts.Secure || c.opts.TLSConfig != nil {
-		if config.HttpClient == nil {
-			config.HttpClient = new(http.Client)
-		}
-
+
 		config.Scheme = "https"
 		// We're going to support InsecureSkipVerify
 		config.HttpClient.Transport = newTransport(c.opts.TLSConfig)