Merge pull request #72 from cloudflare/bug/close-connections

Bugfix: make sure to close http connections
cloudflare · Jul 7, 2020 · 78f7a6e · 78f7a6e
2 parents 2a50540 + 2d9e8a0
commit 78f7a6e
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/cmd/gortr/gortr.go b/cmd/gortr/gortr.go
@@ -222,7 +222,10 @@ func (s *state) fetchFile(file string) ([]byte, error) {
 		if err != nil {
 			return nil, err
 		}
-
+		if fhttp.Body != nil {
+			defer fhttp.Body.Close()
+		}
+		defer client.CloseIdleConnections()
 		RefreshStatusCode.WithLabelValues(file, fmt.Sprintf("%d", fhttp.StatusCode)).Inc()
 
 		if fhttp.StatusCode == 304 {