httpclient: a common http.Client with sensible default values

README.md

@@ -72,6 +72,15 @@ type Sessions[I identity.UserIdentity] interface {
 }
 ```
 
+#### package webtools/httpclient
+
+Provides a sharable `http.Client` that sets sensible values for maintaining a
+pool of idle connections.
+
+```go
+client := httpclient.Get()
+```
+
 #### package webtools/middles/identity
 
 Provides a set of generic structs used for marshaling identity. The interfaces

httpclient/client.go

@@ -0,0 +1,36 @@
+package httpclient
+
+import (
+	"net"
+	"net/http"
+	"time"
+)
+
+func pooledTransport() *http.Transport {
+	transport := &http.Transport{
+		Proxy: http.ProxyFromEnvironment,
+		DialContext: (&net.Dialer{
+			Timeout:   30 * time.Second,
+			KeepAlive: 30 * time.Second,
+			DualStack: true,
+		}).DialContext,
+		MaxIdleConns:          32,
+		IdleConnTimeout:       90 * time.Second,
+		TLSHandshakeTimeout:   10 * time.Second,
+		ExpectContinueTimeout: 3 * time.Second,
+		ForceAttemptHTTP2:     true,
+		MaxIdleConnsPerHost:   1,
+	}
+	return transport
+}
+
+var client = &http.Client{
+	Timeout:   1 * time.Minute,
+	Transport: pooledTransport(),
+}
+
+// Get returns an http.Client tuned for shared / pooled connections, with
+// reasonable values for idle connections, timeouts, and http2 usage.
+func Get() *http.Client {
+	return client
+}

httpclient/client_test.go

@@ -0,0 +1,15 @@
+package httpclient
+
+import (
+	"testing"
+	"time"
+
+	"github.com/shoenig/test/must"
+)
+
+func TestGet(t *testing.T) {
+	t.Parallel()
+
+	c := Get()
+	must.Eq(t, 1*time.Minute, c.Timeout)
+}