Fix: check github http status code for version
This commit is contained in:
@@ -3,6 +3,8 @@ package version
|
|||||||
import (
|
import (
|
||||||
"context"
|
"context"
|
||||||
"encoding/json"
|
"encoding/json"
|
||||||
|
"errors"
|
||||||
|
"fmt"
|
||||||
"net/http"
|
"net/http"
|
||||||
"time"
|
"time"
|
||||||
)
|
)
|
||||||
@@ -23,17 +25,25 @@ type githubCommit struct {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
var errHTTPStatusCode = errors.New("bad response HTTP status code")
|
||||||
|
|
||||||
func getGithubReleases(ctx context.Context, client *http.Client) (releases []githubRelease, err error) {
|
func getGithubReleases(ctx context.Context, client *http.Client) (releases []githubRelease, err error) {
|
||||||
const url = "https://api.github.com/repos/qdm12/gluetun/releases"
|
const url = "https://api.github.com/repos/qdm12/gluetun/releases"
|
||||||
request, err := http.NewRequestWithContext(ctx, http.MethodGet, url, nil)
|
request, err := http.NewRequestWithContext(ctx, http.MethodGet, url, nil)
|
||||||
if err != nil {
|
if err != nil {
|
||||||
return nil, err
|
return nil, err
|
||||||
}
|
}
|
||||||
|
|
||||||
response, err := client.Do(request)
|
response, err := client.Do(request)
|
||||||
if err != nil {
|
if err != nil {
|
||||||
return nil, err
|
return nil, err
|
||||||
}
|
}
|
||||||
defer response.Body.Close()
|
defer response.Body.Close()
|
||||||
|
|
||||||
|
if response.StatusCode != http.StatusOK {
|
||||||
|
return nil, fmt.Errorf("%w: %s", errHTTPStatusCode, response.Status)
|
||||||
|
}
|
||||||
|
|
||||||
decoder := json.NewDecoder(response.Body)
|
decoder := json.NewDecoder(response.Body)
|
||||||
if err := decoder.Decode(&releases); err != nil {
|
if err := decoder.Decode(&releases); err != nil {
|
||||||
return nil, err
|
return nil, err
|
||||||
|
|||||||
Reference in New Issue
Block a user