chore: update codebase to comply with new Go linter rules and standards

- Update golangci-lint-action usage to v7 and set version to v2.0 in GitHub workflow
- Add version field and replace linters settings in `.golangci.yml`
- Update linter exclusions and presets, adjust paths and formatters configurations in `.golangci.yml`
- Replace `ioutil` package with `io` package for `ReadAll` and `NopCloser` in multiple files to accommodate changes in Go's standard library

Signed-off-by: appleboy <[email protected]>

Author: appleboy

.github/workflows/go.yml

@@ -20,8 +20,9 @@ jobs:
         with:
           go-version-file: "go.mod"
       - name: Setup golangci-lint
-        uses: golangci/golangci-lint-action@v6
+        uses: golangci/golangci-lint-action@v7
         with:
+          version: v2.0
           args: --verbose
   test:
     strategy:

.golangci.yml

@@ -1,23 +1,18 @@
+version: "2"
 linters:
-  enable-all: false
-  disable-all: true
-  fast: false
+  default: none
   enable:
     - bodyclose
     - dogsled
     - dupl
     - errcheck
-    - exportloopref
     - exhaustive
     - gochecknoinits
     - goconst
     - gocritic
     - gocyclo
-    - gofmt
-    - goimports
     - goprintffuncname
     - gosec
-    - gosimple
     - govet
     - ineffassign
     - lll
@@ -27,25 +22,40 @@ linters:
     - nolintlint
     - rowserrcheck
     - staticcheck
-    - stylecheck
-    - typecheck
     - unconvert
     - unparam
     - unused
     - whitespace
+  exclusions:
+    generated: lax
+    presets:
+      - comments
+      - common-false-positives
+      - legacy
+      - std-error-handling
+    rules:
+      - linters:
+          - bodyclose
+          - unparam
+        path: authenticator_test.go
+      - linters:
+          - lll
+        path: parser_test.go
+      - linters:
+          - lll
+        path: signatureheader_test.go
+    paths:
+      - third_party$
+      - builtin$
+      - examples$
+formatters:
+  enable:
+    - gofmt
     - gofumpt
-
-issues:
-  exclude-rules:
-    - path: authenticator_test.go
-      linters:
-        - bodyclose
-        - unparam
-    - path: parser_test.go
-      linters:
-        - lll
-    - path: signatureheader_test.go
-      linters:
-        - lll
-run:
-  timeout: 3m
+    - goimports
+  exclusions:
+    generated: lax
+    paths:
+      - third_party$
+      - builtin$
+      - examples$

authenticator_test.go

@@ -3,7 +3,7 @@ package httpsign
 import (
 	"context"
 	"fmt"
-	"io/ioutil"
+	"io"
 	"net/http"
 	"net/http/httptest"
 	"strings"
@@ -250,7 +250,7 @@ func TestHttpValidRequestBody(t *testing.T) {
 	r.ServeHTTP(w, req)
 
 	assert.Equal(t, http.StatusOK, w.Code)
-	body, err := ioutil.ReadAll(w.Result().Body)
+	body, err := io.ReadAll(w.Result().Body)
 	assert.NoError(t, err)
 	assert.Equal(t, body, []byte(sampleBodyContent))
 }
@@ -275,7 +275,7 @@ func TestHttpValidRequestHost(t *testing.T) {
 	r.ServeHTTP(w, req)
 
 	assert.Equal(t, http.StatusOK, w.Code)
-	body, err := ioutil.ReadAll(w.Result().Body)
+	body, err := io.ReadAll(w.Result().Body)
 	assert.NoError(t, err)
 	assert.Equal(t, body, []byte(sampleBodyContent))
 }

validator/digestvalidate.go

@@ -6,7 +6,7 @@ import (
 	"encoding/base64"
 	"errors"
 	"fmt"
-	"io/ioutil"
+	"io"
 	"net/http"
 
 	"github.com/gin-gonic/gin"
@@ -46,11 +46,11 @@ func calculateDigest(r *http.Request) (string, error) {
 		return "", nil
 	}
 	// TODO: Read body using buffer to prevent using too much memory
-	body, err := ioutil.ReadAll(r.Body)
+	body, err := io.ReadAll(r.Body)
 	if err != nil {
 		return "", err
 	}
-	r.Body = ioutil.NopCloser(bytes.NewBuffer(body))
+	r.Body = io.NopCloser(bytes.NewBuffer(body))
 	h := sha256.New()
 	h.Write(body)
 	if err != nil {