chore: improve code quality with linting and signature changes

- Add `gosec` and `revive` linters to `gin.go` and `_test\.go` files in `.golangci.yml`
- Change `Readdir` method signature in `neuteredReaddirFile` in `fs.go`
- Remove unused `err` parameter in `defaultHandleRecovery` function in `recovery.go`

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

Author: appleboy

.github/workflows/gin.yml

@@ -24,7 +24,7 @@ jobs:
       - name: Setup golangci-lint
         uses: golangci/[email protected]
         with:
-          version: v1.48.0
+          version: v1.52.2
           args: --verbose
   test:
     needs: lint

.golangci.yml

@@ -37,3 +37,9 @@ issues:
     - path: _test\.go
       linters:
         - gosec # security is not make sense in tests
+    - path: gin.go
+      linters:
+        - gosec
+    - linters:
+        - revive
+      path: _test\.go

fs.go

@@ -39,7 +39,7 @@ func (fs onlyFilesFS) Open(name string) (http.File, error) {
 }
 
 // Readdir overrides the http.File default implementation.
-func (f neuteredReaddirFile) Readdir(count int) ([]os.FileInfo, error) {
+func (f neuteredReaddirFile) Readdir(_ int) ([]os.FileInfo, error) {
 	// this disables directory listing
 	return nil, nil
 }

recovery.go

@@ -103,7 +103,7 @@ func CustomRecoveryWithWriter(out io.Writer, handle RecoveryFunc) HandlerFunc {
 	}
 }
 
-func defaultHandleRecovery(c *Context, err any) {
+func defaultHandleRecovery(c *Context, _ any) {
 	c.AbortWithStatus(http.StatusInternalServerError)
 }