From 3c00eeaf16645a7626cd00ce2621155950b896bb Mon Sep 17 00:00:00 2001 From: Simon Pasquier Date: Tue, 2 Oct 2018 12:48:07 +0200 Subject: [PATCH] web/api/v1: fix optional skip_head for snapshot (#4674) Signed-off-by: Simon Pasquier --- web/api/v1/api.go | 12 +++++++++--- 1 file changed, 9 insertions(+), 3 deletions(-) diff --git a/web/api/v1/api.go b/web/api/v1/api.go index 71fd4ac543..3caedb5d2e 100644 --- a/web/api/v1/api.go +++ b/web/api/v1/api.go @@ -892,9 +892,15 @@ func (api *API) snapshot(r *http.Request) (interface{}, *apiError, func()) { if !api.enableAdmin { return nil, &apiError{errorUnavailable, errors.New("Admin APIs disabled")}, nil } - skipHead, err := strconv.ParseBool(r.FormValue("skip_head")) - if err != nil { - return nil, &apiError{errorUnavailable, fmt.Errorf("unable to parse boolean 'skip_head' argument: %v", err)}, nil + var ( + skipHead bool + err error + ) + if r.FormValue("skip_head") != "" { + skipHead, err = strconv.ParseBool(r.FormValue("skip_head")) + if err != nil { + return nil, &apiError{errorUnavailable, fmt.Errorf("unable to parse boolean 'skip_head' argument: %v", err)}, nil + } } db := api.db()