From 4892822343d099c0ed262fb3ef111b703a5c3991 Mon Sep 17 00:00:00 2001
From: "renovate[bot]" <29139614+renovate[bot]@users.noreply.github.com>
Date: Wed, 21 Aug 2024 12:31:28 +0400
Subject: [PATCH] chore(deps): update dependency golangci/golangci-lint to
 v1.60.2 (#305)

* chore(deps): update dependency golangci/golangci-lint to v1.60.2

* linter fixes

---------

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Paramtamtam <7326800+tarampampam@users.noreply.github.com>
---
 .golangci.yml                             | 2 +-
 Dockerfile                                | 2 +-
 internal/cli/build/command.go             | 4 ++--
 internal/cli/perftest/command.go          | 6 +++---
 internal/cli/serve/command.go             | 4 ++--
 internal/cli/serve/command_test.go        | 2 +-
 internal/http/handlers/error_page/code.go | 4 ++--
 internal/http/server.go                   | 2 +-
 internal/http/server_test.go              | 2 +-
 9 files changed, 14 insertions(+), 14 deletions(-)

diff --git a/.golangci.yml b/.golangci.yml
index 80ed761..069a034 100644
--- a/.golangci.yml
+++ b/.golangci.yml
@@ -61,7 +61,7 @@ linters: # All available linters list: <https://golangci-lint.run/usage/linters/
     - errcheck # Errcheck is a program for checking for unchecked errors in go programs. These unchecked errors can be critical bugs in some cases
     - errorlint # find code that will cause problems with the error wrapping scheme introduced in Go 1.13
     - exhaustive # check exhaustiveness of enum switch statements
-    - exportloopref # checks for pointers to enclosing loop variables
+    - copyloopvar # detects places where loop variables are copied
     - funlen # Tool for detection of long functions
     - gci # Gci control golang package import order and make it always deterministic
     - godot # Check if comments end in a period
diff --git a/Dockerfile b/Dockerfile
index 1f809f6..ca59bfb 100644
--- a/Dockerfile
+++ b/Dockerfile
@@ -8,7 +8,7 @@ ENV GOPATH="/var/tmp/go"
 
 RUN set -x \
     # renovate: source=github-releases name=golangci/golangci-lint
-    && GOLANGCI_LINT_VERSION="1.60.1" \
+    && GOLANGCI_LINT_VERSION="1.60.2" \
     && wget -O- -nv "https://cdn.jsdelivr.net/gh/golangci/golangci-lint@v${GOLANGCI_LINT_VERSION}/install.sh" \
       | sh -s -- -b /bin "v${GOLANGCI_LINT_VERSION}"
 
diff --git a/internal/cli/build/command.go b/internal/cli/build/command.go
index 28ab5dd..64a9c69 100644
--- a/internal/cli/build/command.go
+++ b/internal/cli/build/command.go
@@ -150,7 +150,7 @@ func NewCommand(log *logger.Logger) *cli.Command { //nolint:funlen,gocognit
 }
 
 func (cmd *command) Run( //nolint:funlen,gocognit
-	ctx context.Context,
+	_ context.Context,
 	log *logger.Logger,
 	cfg *config.Config,
 ) error {
@@ -176,7 +176,7 @@ func (cmd *command) Run( //nolint:funlen,gocognit
 			var outFilePath = path.Join(cmd.opt.targetDirAbsPath, templateName, code+".html")
 
 			if content, renderErr := appTemplate.Render(templateContent, appTemplate.Props{ //nolint:nestif
-				Code:               uint16(codeAsUint),
+				Code:               uint16(codeAsUint), //nolint:gosec
 				Message:            codeDescription.Message,
 				Description:        codeDescription.Description,
 				L10nDisabled:       cfg.L10n.Disable,
diff --git a/internal/cli/perftest/command.go b/internal/cli/perftest/command.go
index c6f2abe..d33c349 100644
--- a/internal/cli/perftest/command.go
+++ b/internal/cli/perftest/command.go
@@ -108,10 +108,10 @@ func NewCommand() *cli.Command { //nolint:funlen
 			var runTest = func(scriptContent string) error {
 				if stdOut, stdErr, err := wrkRunTest(ctx,
 					wrkBinPath,
-					uint16(c.Uint(threadsFlag.Name)),
-					uint16(c.Uint(connectionsFlag.Name)),
+					uint16(c.Uint(threadsFlag.Name)),     //nolint:gosec
+					uint16(c.Uint(connectionsFlag.Name)), //nolint:gosec
 					c.Duration(durationFlag.Name),
-					uint16(c.Uint(portFlag.Name)),
+					uint16(c.Uint(portFlag.Name)), //nolint:gosec
 					scriptContent,
 				); err != nil {
 					var errData, _ = io.ReadAll(stdErr)
diff --git a/internal/cli/serve/command.go b/internal/cli/serve/command.go
index 497b26a..0b0b80d 100644
--- a/internal/cli/serve/command.go
+++ b/internal/cli/serve/command.go
@@ -176,10 +176,10 @@ func NewCommand(log *logger.Logger) *cli.Command { //nolint:funlen,gocognit,gocy
 		Suggest: true,
 		Action: func(ctx context.Context, c *cli.Command) error {
 			cmd.opt.http.addr = c.String(addrFlag.Name)
-			cmd.opt.http.port = uint16(c.Uint(portFlag.Name))
+			cmd.opt.http.port = uint16(c.Uint(portFlag.Name)) //nolint:gosec
 			cmd.opt.http.readBufferSize = uint(c.Uint(readBufferSizeFlag.Name))
 			cfg.L10n.Disable = c.Bool(disableL10nFlag.Name)
-			cfg.DefaultCodeToRender = uint16(c.Uint(defaultCodeToRenderFlag.Name))
+			cfg.DefaultCodeToRender = uint16(c.Uint(defaultCodeToRenderFlag.Name)) //nolint:gosec
 			cfg.RespondWithSameHTTPCode = c.Bool(sendSameHTTPCodeFlag.Name)
 			cfg.RotationMode, _ = config.ParseRotationMode(c.String(rotationModeFlag.Name))
 			cfg.ShowDetails = c.Bool(showDetailsFlag.Name)
diff --git a/internal/cli/serve/command_test.go b/internal/cli/serve/command_test.go
index bd21b65..20be40e 100644
--- a/internal/cli/serve/command_test.go
+++ b/internal/cli/serve/command_test.go
@@ -97,5 +97,5 @@ func getFreeTcpPort(t *testing.T) uint16 {
 		<-time.After(5 * time.Millisecond)
 	}
 
-	return uint16(port)
+	return uint16(port) //nolint:gosec
 }
diff --git a/internal/http/handlers/error_page/code.go b/internal/http/handlers/error_page/code.go
index 2a5745d..e317875 100644
--- a/internal/http/handlers/error_page/code.go
+++ b/internal/http/handlers/error_page/code.go
@@ -28,7 +28,7 @@ func extractCodeFromURL(url string) (uint16, bool) {
 	}
 
 	if code, err := strconv.ParseUint(fileName, 10, 16); err == nil && code > 0 && code < 999 {
-		return uint16(code), true
+		return uint16(code), true //nolint:gosec
 	}
 
 	return 0, false
@@ -47,7 +47,7 @@ func extractCodeFromHeaders(headers *fasthttp.RequestHeader) (uint16, bool) {
 	// HTTP status code returned by the request
 	if value := headers.Peek("X-Code"); len(value) > 0 && len(value) <= 3 {
 		if code, err := strconv.ParseUint(string(value), 10, 16); err == nil && code > 0 && code < 999 {
-			return uint16(code), true
+			return uint16(code), true //nolint:gosec
 		}
 	}
 
diff --git a/internal/http/server.go b/internal/http/server.go
index 7edc614..13442e4 100644
--- a/internal/http/server.go
+++ b/internal/http/server.go
@@ -40,7 +40,7 @@ func NewServer(log *logger.Logger, readBufferSize uint) Server {
 		server: &fasthttp.Server{
 			ReadTimeout:                  readTimeout,
 			WriteTimeout:                 writeTimeout,
-			ReadBufferSize:               int(readBufferSize),
+			ReadBufferSize:               int(readBufferSize), //nolint:gosec
 			DisablePreParseMultipartForm: true,
 			NoDefaultServerHeader:        true,
 			CloseOnShutdown:              true,
diff --git a/internal/http/server_test.go b/internal/http/server_test.go
index 1ca8dad..acfe649 100644
--- a/internal/http/server_test.go
+++ b/internal/http/server_test.go
@@ -392,5 +392,5 @@ func getFreeTcpPort(t *testing.T) uint16 {
 		<-time.After(5 * time.Millisecond)
 	}
 
-	return uint16(port)
+	return uint16(port) //nolint:gosec
 }