chore: introduce linters and remove unused code in file handling functions

- Add `gosec` and `revive` linters
- Remove unnecessary argument from `Readdir` function
- Remove unused `err` variable in `defaultHandleRecovery` function

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

Author: appleboy

.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)
 }