ci: bump golangci-lint to v1.58.1

While here, also enable new linters.

Signed-off-by: Robin Hahling <[email protected]>

Author: rolinh

.github/workflows/build.yaml

@@ -26,7 +26,7 @@ jobs:
       - name: Run static checks
         uses: golangci/golangci-lint-action@23faadfdeb23a6f9e511beaba149bb123b5b145a
         with:
-          version: v1.55.2
+          version: v1.58.1
           # use our .golangci.yml and configure output to be logged in the GHA, in addition to annotating the commit.
           # see https://github.com/golangci/golangci-lint-action/issues/119#issuecomment-981090648 for output
           args: --config=.golangci.yml --verbose --out-${NO_FUTURE}format colored-line-number

.golangci.yml

@@ -11,14 +11,15 @@ linters:
     - bidichk
     - bodyclose
     - contextcheck
+    - copyloopvar
     - decorder
     - dupword
     - durationcheck
+    - err113
     - errchkjson
     - errname
     - gocheckcompilerdirectives
     - gocritic
-    - goerr113
     - gofmt
     - goheader
     - goimports
@@ -29,6 +30,7 @@ linters:
     - importas
     - ineffassign
     - interfacebloat
+    - intrange
     - makezero
     - mirror
     - misspell
@@ -40,9 +42,11 @@ linters:
     - perfsprint
     - predeclared
     - promlinter
+    - protogetter
     - reassign
     - revive
     - rowserrcheck
+    - sloglint
     - sqlclosecheck
     - staticcheck
     - tagalign
@@ -72,11 +76,25 @@ linters-settings:
     disable:
       - fieldalignment
       - shadow
+  importas:
+    no-unaliased: true
+    alias:
+      - pkg: github.com/cilium/cilium/api/v1/flow
+        alias: flowpb
+      - pkg: github.com/cilium/cilium/api/v1/observer
+        alias: observerpb
+      - pkg: github.com/cilium/cilium/api/v1/relay
+        alias: relaypb
+      - pkg: github.com/cilium/fake/flow
+        alias: fakeflow
+  perfsprint:
+    strconcat: false
   revive:
     rules:
       - name: package-comments
         disabled: true
-
+  sloglint:
+    static-msg: true
 issues:
   # Default rules exclude Go doc comments check, which is rather unfortunate.
   # In order to enable Go doc checks, defaults rules have to be disabled.
@@ -86,5 +104,5 @@ issues:
     - Potential file inclusion via variable # EXC0010 gosec: False positive is triggered by 'src, err := io.ReadFile(filename)'
 
   exclude-rules:
-    - linters: [goerr113]
+    - linters: [err113]
       text: "do not define dynamic errors, use wrapped static errors instead"