From fcad2c6584cf74af9b7ccaa13daedabba248e9b5 Mon Sep 17 00:00:00 2001 From: Sebastiaan van Stijn Date: Mon, 23 Oct 2023 15:01:31 +0200 Subject: [PATCH] image/build: use "nolint" comment to work around gosec regression Latest gosec linter has a regression in parsing "nosec" comments; see https://github.com/securego/gosec/issues/1046 Signed-off-by: Sebastiaan van Stijn (cherry picked from commit 9e1f8d646e15b2626e56960e9505ad281937dc30) Signed-off-by: Cory Snider --- cli/command/image/build/context.go | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/cli/command/image/build/context.go b/cli/command/image/build/context.go index c4deef7e08..141344a407 100644 --- a/cli/command/image/build/context.go +++ b/cli/command/image/build/context.go @@ -231,8 +231,7 @@ func GetContextFromURL(out io.Writer, remoteURL, dockerfileName string) (io.Read // getWithStatusError does an http.Get() and returns an error if the // status code is 4xx or 5xx. func getWithStatusError(url string) (resp *http.Response, err error) { - // #nosec G107 - if resp, err = http.Get(url); err != nil { + if resp, err = http.Get(url); err != nil { //nolint:gosec // Ignore G107: Potential HTTP request made with variable url return nil, err } if resp.StatusCode < http.StatusBadRequest {