Elgato_dark/fleet
1
0
Fork 0
mirror of https://github.com/fleetdm/fleet synced 2026-05-23 00:49:03 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 106

Timeout after 30 seconds when posting usage analytics (#1577)

Browse source
This commit is contained in:
Tomas Touceda 2021-08-06 13:20:59 -03:00 committed by GitHub
parent 084fcdfec4
commit e63e690fdb
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 10 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

1
changes/timeout-in-request-analytics Normal file
View file

@ -0,0 +1 @@
* When posting usage analytics, timeout after 30secs.

12
cmd/fleet/serve.go
View file

@ -426,12 +426,18 @@ func trySendStatistics(ds fleet.Datastore, frequency time.Duration, url string)
if err != nil {
return err
}
req, err := http.Post(url, "application/json", bytes.NewBuffer(statsBytes))
client := &http.Client{Timeout: 30 * time.Second}
req, err := http.NewRequest("POST", url, bytes.NewBuffer(statsBytes))
if err != nil {
return err
}
if req.StatusCode != http.StatusOK {
return errors.Errorf("Error posting to %s: %d", url, req.StatusCode)
req.Header.Set("Content-Type", "application/json")
resp, err := client.Do(req)
if err != nil {
return err
}
if resp.StatusCode != http.StatusOK {
return errors.Errorf("Error posting to %s: %d", url, resp.StatusCode)
}
return ds.RecordStatisticsSent()
}