From b1d39009c6fe7250d8983d7c74b7abe4970c9ee2 Mon Sep 17 00:00:00 2001
From: Valery Piashchynski <piashchynski.valery@gmail.com>
Date: Fri, 6 Sep 2024 11:18:54 +0200
Subject: [PATCH] chore: fix linters warnings

Signed-off-by: Valery Piashchynski <piashchynski.valery@gmail.com>
---
 .github/workflows/linters.yml |  4 ++--
 .github/workflows/linux.yml   |  2 +-
 .golangci.yml                 |  2 +-
 pkg/rpc/client.go             |  4 ++--
 pkg/rpc/codec.go              | 18 +++++++++---------
 pkg/socket/socket_test.go     |  4 ++--
 6 files changed, 17 insertions(+), 17 deletions(-)

diff --git a/.github/workflows/linters.yml b/.github/workflows/linters.yml
index 9a03e58..307fe76 100644
--- a/.github/workflows/linters.yml
+++ b/.github/workflows/linters.yml
@@ -3,7 +3,7 @@ name: Linters
 on: [push, pull_request]
 
 jobs:
-  golangci-lint:
+  linters:
     name: Golang-CI (lint)
     runs-on: ubuntu-latest
     steps:
@@ -18,5 +18,5 @@ jobs:
       - name: Run linter
         uses: golangci/golangci-lint-action@v6.1.0 # Action page: <https://github.com/golangci/golangci-lint-action>
         with:
-          version: v1.60 # without patch version
+          version: v1 # without patch version
           only-new-issues: false # show only new issues if it's a pull request
diff --git a/.github/workflows/linux.yml b/.github/workflows/linux.yml
index c805952..1beebc4 100644
--- a/.github/workflows/linux.yml
+++ b/.github/workflows/linux.yml
@@ -3,7 +3,7 @@ name: Linux
 on: [push, pull_request]
 
 jobs:
-  build:
+  testing:
     name: Tests [Go ${{ matrix.go }} OS ${{ matrix.os }}]
     runs-on: ${{ matrix.os }}
     timeout-minutes: 20
diff --git a/.golangci.yml b/.golangci.yml
index fd856a9..1b553da 100644
--- a/.golangci.yml
+++ b/.golangci.yml
@@ -39,7 +39,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 # checks for pointers to enclosing loop variables
     - gochecknoinits # Checks that no init functions are present in Go code
     - goconst # Finds repeated strings that could be replaced by a constant
     - gocritic # The most opinionated Go source code linter
diff --git a/pkg/rpc/client.go b/pkg/rpc/client.go
index a436fca..6c93b54 100644
--- a/pkg/rpc/client.go
+++ b/pkg/rpc/client.go
@@ -99,10 +99,10 @@ func (c *ClientCodec) WriteRequest(r *rpc.Request, body any) error {
 	}
 
 	// SEQ_ID + METHOD_NAME_LEN
-	fr.WriteOptions(fr.HeaderPtr(), uint32(r.Seq), uint32(len(r.ServiceMethod)))
+	fr.WriteOptions(fr.HeaderPtr(), uint32(r.Seq), uint32(len(r.ServiceMethod))) //nolint:gosec
 	fr.WriteVersion(fr.Header(), frame.Version1)
 
-	fr.WritePayloadLen(fr.Header(), uint32(buf.Len()))
+	fr.WritePayloadLen(fr.Header(), uint32(buf.Len())) //nolint:gosec
 	fr.WritePayload(buf.Bytes())
 	fr.WriteCRC(fr.Header())
 
diff --git a/pkg/rpc/codec.go b/pkg/rpc/codec.go
index 664cf9a..1ade16f 100644
--- a/pkg/rpc/codec.go
+++ b/pkg/rpc/codec.go
@@ -67,14 +67,14 @@ func (c *Codec) putFrame(f *frame.Frame) {
 	c.fPool.Put(f)
 }
 
-// WriteResponse marshals response, byte slice or error to remote party.
+// WriteResponse marshals response, byte slice or error to remote.
 func (c *Codec) WriteResponse(r *rpc.Response, body any) error { //nolint:funlen
 	const op = errors.Op("goridge_write_response")
 	fr := c.getFrame()
 	defer c.putFrame(fr)
 
 	// SEQ_ID + METHOD_NAME_LEN
-	fr.WriteOptions(fr.HeaderPtr(), uint32(r.Seq), uint32(len(r.ServiceMethod)))
+	fr.WriteOptions(fr.HeaderPtr(), uint32(r.Seq), uint32(len(r.ServiceMethod))) //nolint:gosec
 	// Write protocol version
 	fr.WriteVersion(fr.Header(), frame.Version1)
 
@@ -110,7 +110,7 @@ func (c *Codec) WriteResponse(r *rpc.Response, body any) error { //nolint:funlen
 		buf.WriteString(r.ServiceMethod)
 		buf.Write(d)
 
-		fr.WritePayloadLen(fr.Header(), uint32(buf.Len()))
+		fr.WritePayloadLen(fr.Header(), uint32(buf.Len())) //nolint:gosec
 		// copy inside
 		fr.WritePayload(buf.Bytes())
 		fr.WriteCRC(fr.Header())
@@ -128,7 +128,7 @@ func (c *Codec) WriteResponse(r *rpc.Response, body any) error { //nolint:funlen
 			buf.WriteString(r.ServiceMethod)
 			buf.Write(data)
 
-			fr.WritePayloadLen(fr.Header(), uint32(buf.Len()))
+			fr.WritePayloadLen(fr.Header(), uint32(buf.Len())) //nolint:gosec
 			fr.WritePayload(buf.Bytes())
 		case *[]byte:
 			buf.Grow(len(*data) + len(r.ServiceMethod))
@@ -136,7 +136,7 @@ func (c *Codec) WriteResponse(r *rpc.Response, body any) error { //nolint:funlen
 			buf.WriteString(r.ServiceMethod)
 			buf.Write(*data)
 
-			fr.WritePayloadLen(fr.Header(), uint32(buf.Len()))
+			fr.WritePayloadLen(fr.Header(), uint32(buf.Len())) //nolint:gosec
 			fr.WritePayload(buf.Bytes())
 		default:
 			return c.handleError(r, fr, "unknown Raw payload type")
@@ -161,7 +161,7 @@ func (c *Codec) WriteResponse(r *rpc.Response, body any) error { //nolint:funlen
 		buf.WriteString(r.ServiceMethod)
 		buf.Write(data)
 
-		fr.WritePayloadLen(fr.Header(), uint32(buf.Len()))
+		fr.WritePayloadLen(fr.Header(), uint32(buf.Len())) //nolint:gosec
 		// copy inside
 		fr.WritePayload(buf.Bytes())
 		fr.WriteCRC(fr.Header())
@@ -182,7 +182,7 @@ func (c *Codec) WriteResponse(r *rpc.Response, body any) error { //nolint:funlen
 		buf.WriteString(r.ServiceMethod)
 		buf.Write(b)
 
-		fr.WritePayloadLen(fr.Header(), uint32(buf.Len()))
+		fr.WritePayloadLen(fr.Header(), uint32(buf.Len())) //nolint:gosec
 		// copy inside
 		fr.WritePayload(buf.Bytes())
 		fr.WriteCRC(fr.Header())
@@ -202,7 +202,7 @@ func (c *Codec) WriteResponse(r *rpc.Response, body any) error { //nolint:funlen
 			return errors.E(op, err)
 		}
 
-		fr.WritePayloadLen(fr.Header(), uint32(buf.Len()))
+		fr.WritePayloadLen(fr.Header(), uint32(buf.Len())) //nolint:gosec
 		// copy inside
 		fr.WritePayload(buf.Bytes())
 		fr.WriteCRC(fr.Header())
@@ -226,7 +226,7 @@ func (c *Codec) handleError(r *rpc.Response, fr *frame.Frame, err string) error
 	if err != "" {
 		buf.WriteString(err)
 	}
-	fr.WritePayloadLen(fr.Header(), uint32(buf.Len()))
+	fr.WritePayloadLen(fr.Header(), uint32(buf.Len())) //nolint:gosec
 	fr.WritePayload(buf.Bytes())
 
 	fr.WriteCRC(fr.Header())
diff --git a/pkg/socket/socket_test.go b/pkg/socket/socket_test.go
index 2173dcc..db8cf5e 100644
--- a/pkg/socket/socket_test.go
+++ b/pkg/socket/socket_test.go
@@ -19,7 +19,7 @@ func TestSocketRelay(t *testing.T) {
 	nf := frame.NewFrame()
 	nf.WriteVersion(nf.Header(), frame.Version1)
 	nf.WriteFlags(nf.Header(), frame.CONTROL, frame.CodecGob, frame.CodecJSON)
-	nf.WritePayloadLen(nf.Header(), uint32(len([]byte(TestPayload))))
+	nf.WritePayloadLen(nf.Header(), uint32(len([]byte(TestPayload)))) //nolint:gosec
 	nf.WritePayload([]byte(TestPayload))
 	nf.WriteCRC(nf.Header())
 	assert.Equal(t, true, nf.VerifyCRC(nf.Header()))
@@ -58,7 +58,7 @@ func TestSocketRelayOptions(t *testing.T) {
 	nf := frame.NewFrame()
 	nf.WriteVersion(nf.Header(), frame.Version1)
 	nf.WriteFlags(nf.Header(), frame.CONTROL, frame.CodecGob, frame.CodecJSON)
-	nf.WritePayloadLen(nf.Header(), uint32(len([]byte(TestPayload))))
+	nf.WritePayloadLen(nf.Header(), uint32(len([]byte(TestPayload)))) //nolint:gosec
 	nf.WritePayload([]byte(TestPayload))
 	nf.WriteOptions(nf.HeaderPtr(), 100, 10000, 100000)
 	nf.WriteCRC(nf.Header())