[]HttpServer -> []*HTTPServer

command/agent/command.go

@@ -51,7 +51,7 @@ type Command struct {
 
 	args           []string
 	agent          *Agent
-	httpServers    []HTTPServer
+	httpServers    []*HTTPServer
 	logFilter      *logutils.LevelFilter
 	logOutput      io.Writer
 	retryJoinErrCh chan struct{}

command/agent/http.go

@@ -78,8 +78,8 @@ type HTTPServer struct {
 
 // NewHTTPServers starts an HTTP server for every address.http configured in
 // the agent.
-func NewHTTPServers(agent *Agent, config *Config) ([]HTTPServer, error) {
-	var srvs []HTTPServer
+func NewHTTPServers(agent *Agent, config *Config) ([]*HTTPServer, error) {
+	var srvs []*HTTPServer
 	var serverInitializationErrors error
 
 	// Handle requests with gzip compression
@@ -166,7 +166,7 @@ func NewHTTPServers(agent *Agent, config *Config) ([]HTTPServer, error) {
 			httpServer.Serve(ln)
 		}()
 
-		srvs = append(srvs, *srv)
+		srvs = append(srvs, srv)
 	}
 
 	if serverInitializationErrors != nil {

command/agent/testagent.go

@@ -68,7 +68,7 @@ type TestAgent struct {
 
 	// All HTTP servers started. Used to prevent server leaks and preserve
 	// backwards compability.
-	Servers []HTTPServer
+	Servers []*HTTPServer
 
 	// Server is a reference to the primary, started HTTP endpoint.
 	// It is valid after Start().
@@ -267,7 +267,7 @@ func (a *TestAgent) start() (*Agent, error) {
 	// TODO: investigate if there is a way to remove the requirement by updating test.
 	// Initial pass at implementing this is https://github.com/kevinschoonover/nomad/tree/tests.
 	a.Servers = httpServers
-	a.Server = &httpServers[0]
+	a.Server = httpServers[0]
 	return agent, nil
 }