From 9afe455635f4249f53fcc6b8e0a2ffdade216813 Mon Sep 17 00:00:00 2001 From: "Quentin McGaw (desktop)" Date: Wed, 8 Sep 2021 16:09:32 +0000 Subject: [PATCH] Fix: missing status code check for Windscribe API --- internal/updater/providers/windscribe/api.go | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/internal/updater/providers/windscribe/api.go b/internal/updater/providers/windscribe/api.go index 9b176068..d6f706ac 100644 --- a/internal/updater/providers/windscribe/api.go +++ b/internal/updater/providers/windscribe/api.go @@ -56,6 +56,11 @@ func fetchAPI(ctx context.Context, client *http.Client) ( } 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) if err := decoder.Decode(&data); err != nil { return data, fmt.Errorf("%w: %s", ErrUnmarshalResponseBody, err)