Merge pull request #10 from topfreegames/fix/memory-leak-http-auth

Fix memory leak when using http auth
topfreegames · May 31, 2021 · 0a68760 · 0a68760
2 parents 6390984 + 94e97ac
commit 0a68760
Showing 1 changed file with 8 additions and 0 deletions.
diff --git a/app/helpers.go b/app/helpers.go
@@ -11,6 +11,8 @@ import (
 	"bytes"
 	"context"
 	"encoding/json"
+	"io"
+	"io/ioutil"
 	"net/http"
 	"strings"
 	"time"
@@ -126,6 +128,12 @@ func httpAuthorize(app *App, userID string, topics []string) (bool, []string, er
 		}
 
 		response, err := client.Do(request)
+		// discard response body
+		if response != nil && response.Body != nil {
+			_, _ = io.Copy(ioutil.Discard, response.Body)
+			_ = response.Body.Close()
+		}
+
 		if err != nil {
 			return false, nil, err
 		}