From a4705cee230a18aa4d2e9ef6bdef89e30e24770c Mon Sep 17 00:00:00 2001
From: Mike Kabischev <kabischev@gmail.com>
Date: Fri, 10 Feb 2017 22:09:07 +0300
Subject: [PATCH] remove mutex from HTTP endpoint (#148)

* fix leaking http connections

* remove unused fields

* remove mutex from HTTPEndpointConn
---
 controller/endpoint/http.go | 21 +++++++--------------
 1 file changed, 7 insertions(+), 14 deletions(-)

diff --git a/controller/endpoint/http.go b/controller/endpoint/http.go
index 5364efad..7326aee1 100644
--- a/controller/endpoint/http.go
+++ b/controller/endpoint/http.go
@@ -6,7 +6,6 @@ import (
 	"io"
 	"io/ioutil"
 	"net/http"
-	"sync"
 	"time"
 )
 
@@ -17,7 +16,6 @@ const (
 )
 
 type HTTPEndpointConn struct {
-	mu     sync.Mutex
 	ep     Endpoint
 	client *http.Client
 }
@@ -25,6 +23,13 @@ type HTTPEndpointConn struct {
 func newHTTPEndpointConn(ep Endpoint) *HTTPEndpointConn {
 	return &HTTPEndpointConn{
 		ep: ep,
+		client: &http.Client{
+			Transport: &http.Transport{
+				MaxIdleConnsPerHost: httpMaxIdleConnections,
+				IdleConnTimeout:     httpExpiresAfter,
+			},
+			Timeout: httpRequestTimeout,
+		},
 	}
 }
 
@@ -33,18 +38,6 @@ func (conn *HTTPEndpointConn) Expired() bool {
 }
 
 func (conn *HTTPEndpointConn) Send(msg string) error {
-	conn.mu.Lock()
-	defer conn.mu.Unlock()
-
-	if conn.client == nil {
-		conn.client = &http.Client{
-			Transport: &http.Transport{
-				MaxIdleConnsPerHost: httpMaxIdleConnections,
-				IdleConnTimeout:     httpExpiresAfter,
-			},
-			Timeout: httpRequestTimeout,
-		}
-	}
 	req, err := http.NewRequest("POST", conn.ep.Original, bytes.NewBufferString(msg))
 	if err != nil {
 		return err