bennetgallein/cachet-monitor
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge pull request #92 from srabouin/master

Browse Source 
Close HTTP requests
This commit is contained in:
Matej Kramny
2019-01-08 10:04:19 +08:00
committed by GitHub
parent ae3a18591f 581b1465e6
commit c4f7544640
1 changed files with 6 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
api.go
View File

@@ -33,6 +33,8 @@ func (api CachetAPI) Ping() error {
return errors.New("API Responded with non-200 status code") return errors.New("API Responded with non-200 status code")
} }
defer resp.Body.Close()
return nil return nil
} }
@@ -49,6 +51,8 @@ func (api CachetAPI) SendMetric(id int, lag int64) {
if err != nil || resp.StatusCode != 200 { if err != nil || resp.StatusCode != 200 {
logrus.Warnf("Could not log metric! ID: %d, err: %v", id, err) logrus.Warnf("Could not log metric! ID: %d, err: %v", id, err)
} }
defer resp.Body.Close()
} }
// TODO: test // TODO: test
@@ -75,5 +79,7 @@ func (api CachetAPI) NewRequest(requestType, url string, reqBody []byte) (*http.
} }
err = json.NewDecoder(res.Body).Decode(&body) err = json.NewDecoder(res.Body).Decode(&body)
defer req.Body.Close()
return res, body, err return res, body, err
} }