Merge pull request #68 from SokoloffA/master

Rucknar · web-flow · commit fc2e58a5128f · 2021-07-23T13:26:56.000+01:00
Fix: Socket leaks if server returns RateLimitExceededStatus status.
diff --git a/exporter/http.go b/exporter/http.go
@@ -152,6 +152,7 @@ func getHTTPResponse(url string, token string) (*http.Response, error) {
 
 	// check rate limit exceeded.
 	if resp.Status == RateLimitExceededStatus {
+		resp.Body.Close()
 		return nil, fmt.Errorf("%s", resp.Status)
 	}
 

Original file line number	Diff line number	Diff line change
`@@ -152,6 +152,7 @@ func getHTTPResponse(url string, token string) (*http.Response, error) {`
`152`	`152`
`153`	`153`	`// check rate limit exceeded.`
`154`	`154`	`if resp.Status == RateLimitExceededStatus {`
	`155`	`+ resp.Body.Close()`
`155`	`156`	`return nil, fmt.Errorf("%s", resp.Status)`
`156`	`157`	`}`
`157`	`158`