Fix issues for golanci-lint, update go version. (#3)

Co-authored-by: Michal Ližičiar <[email protected]>

.golangci.yml

@@ -71,15 +71,17 @@ linters:
     - gocyclo
     - gofmt
     - goimports
-    - gomnd
+    - mnd
     - gomodguard #
     - goprintffuncname
     - gosec
     - gosimple
     - govet
     - ineffassign
     - lll
-    - megacheck #
+    - gosimple
+    - staticcheck
+    - unused
     - misspell
     - nakedret
     - noctx
@@ -99,14 +101,11 @@ linters:
     - gocognit
     - godot
     - godox
-    - goerr113 #
-    - interfacer
-    - maligned #
-    - nestif #
-    - prealloc #
-    - scopelint
-    - testpackage #
-    - wsl #
+    - err113
+    - nestif
+    - prealloc
+    - testpackage
+    - wsl
   presets:
     - bugs
     - unused
@@ -148,7 +147,7 @@ linters-settings:
     local-prefixes: github.com/eset/grpc-rest-proxy
   golint:
     min-confidence: 0
-  gomnd:
+  mnd:
     settings:
       mnd:
         # don't include the "operation" and "assign"

go.mod

@@ -1,6 +1,6 @@
 module github.com/eset/grpc-rest-proxy
 
-go 1.21.8
+go 1.22.5
 
 require (
 	github.com/go-chi/chi/v5 v5.0.8

pkg/service/transformer/transformer.go

@@ -61,7 +61,7 @@ func GetRPCResponse(responseDesc protoreflect.MessageDescriptor) *dynamicpb.Mess
 
 // https://chromium.googlesource.com/external/github.com/grpc/grpc/+/refs/tags/v1.21.4-pre1/doc/statuscodes.md
 //
-//nolint:gocyclo,gomnd
+//nolint:gocyclo,mnd
 func GetHTTPStatusCode(code codes.Code) int {
 	switch code {
 	case codes.OK:

pkg/transport/http/server.go

@@ -6,6 +6,7 @@ package http
 import (
 	"context"
 	"errors"
+	"fmt"
 	"net"
 	"net/http"
 
@@ -39,7 +40,7 @@ func (server *Server) Close() {
 
 	err := server.httpServer.Shutdown(ctx)
 	if err != nil {
-		server.logger.With(err).Error("shutdown http server")
+		server.logger.Error(fmt.Sprintf("shutdown http server error: %s", err))
 		return
 	}
 	server.logger.Info("graceful shutdown successful")

pkg/transport/transport.go

@@ -125,6 +125,6 @@ func statusJSON(w http.ResponseWriter, _ *http.Request) {
 	w.Header().Set("Content-Type", "application/json")
 	_, err := io.WriteString(w, `{"status":"OK"}`)
 	if err != nil {
-		logging.Error("write http response body error for /status: %s", err)
+		logging.Error(fmt.Sprintf("write http response body error for /status: %s", err))
 	}
 }