admin/gluetun
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix: missing status code check for Windscribe API

Browse Source
This commit is contained in:
Quentin McGaw (desktop)
2021-09-08 16:09:32 +00:00
parent 45ce422a89
commit 9afe455635
1 changed files with 5 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
internal/updater/providers/windscribe/api.go
View File

@@ -56,6 +56,11 @@ func fetchAPI(ctx context.Context, client *http.Client) (
} }
defer response.Body.Close() defer response.Body.Close()
if response.StatusCode != http.StatusOK {
return data, fmt.Errorf("%w: %d %s", ErrHTTPStatusCodeNotOK,
response.StatusCode, response.Status)
}
decoder := json.NewDecoder(response.Body) decoder := json.NewDecoder(response.Body)
if err := decoder.Decode(&data); err != nil { if err := decoder.Decode(&data); err != nil {
return data, fmt.Errorf("%w: %s", ErrUnmarshalResponseBody, err) return data, fmt.Errorf("%w: %s", ErrUnmarshalResponseBody, err)