build: migrate to golangci-lint v2 and Go 1.24

- Update golangci-lint configuration to v2 format
- Upgrade golangci-lint action to v7 with v2.1.1
- Update Go version from 1.23 to 1.24
- Fix linting issues with strings.ReplaceAll
- Use switch case in place of if-else chains

.github/workflows/ci.yml

@@ -11,10 +11,10 @@ jobs:
     runs-on: ubuntu-latest
 
     steps:
-      - name: set up go 1.23
+      - name: set up go 1.24
         uses: actions/setup-go@v5
         with:
-          go-version: "1.23"
+          go-version: "1.24"
         id: go
 
       - name: checkout
@@ -38,14 +38,14 @@ jobs:
           TZ: "America/Chicago"
 
       - name: golangci-lint
-        uses: golangci/golangci-lint-action@v6
+        uses: golangci/golangci-lint-action@v7
         with:
-          version: v1.61
+          version: v2.1.1
 
       - name: golangci-lint on example directory
-        uses: golangci/golangci-lint-action@v6
+        uses: golangci/golangci-lint-action@v7
         with:
-          version: v1.61
+          version: v2.1.1
           args: --config ../../.golangci.yml
           working-directory: examples/plugin
 

.golangci.yml

@@ -1,70 +1,72 @@
-run:
+version: "2"
-  timeout: 5m
-
-linters-settings:
-  govet:
-    enable:
-      - shadow
-  goconst:
-    min-len: 2
-    min-occurrences: 2
-  misspell:
-    locale: US
-  lll:
-    line-length: 140
-  gocritic:
-    enabled-tags:
-      - performance
-      - style
-      - experimental
-    disabled-checks:
-      - wrapperFunc
-      - hugeParam
-      - rangeValCopy
-      - singleCaseSwitch
-      - ifElseChain
-
 linters:
+  default: none
   enable:
-    - staticcheck
-    - revive
-    - govet
-    - unconvert
-    - unused
-    - gosec
-    - gocyclo
     - dupl
-    - misspell
-    - unparam
-    - typecheck
-    - ineffassign
-    - stylecheck
     - gochecknoinits
     - gocritic
+    - gocyclo
+    - gosec
+    - govet
+    - ineffassign
+    - misspell
     - nakedret
-    - gosimple
     - prealloc
-  fast: false
+    - revive
-  disable-all: true
+    - staticcheck
-
+    - unconvert
-issues:
+    - unparam
-  exclude-dirs:
+    - unused
-    - vendor
+  settings:
-  exclude-rules:
+    goconst:
-    - text: "at least one file in a package should have a package comment"
+      min-len: 2
-      linters:
+      min-occurrences: 2
-        - stylecheck
+    gocritic:
-    - text: "should have a package comment"
+      disabled-checks:
-      linters:
+        - wrapperFunc
-        - revive
+        - hugeParam
-    - path: _test\.go
+        - rangeValCopy
-      linters:
+        - singleCaseSwitch
-        - gosec
+        - ifElseChain
-        - dupl
+      enabled-tags:
-    - linters:
+        - performance
-        - unparam
+        - style
-        - unused
+        - experimental
-        - revive
+    govet:
-      path: _test\.go$
+      enable:
-      text: "unused-parameter"
+        - shadow
-  exclude-use-default: false
+    lll:
+      line-length: 140
+    misspell:
+      locale: US
+  exclusions:
+    generated: lax
+    rules:
+      - linters:
+          - staticcheck
+        text: at least one file in a package should have a package comment
+      - linters:
+          - revive
+        text: should have a package comment
+      - linters:
+          - dupl
+          - gosec
+        path: _test\.go
+      - linters:
+          - revive
+          - unparam
+          - unused
+        path: _test\.go$
+        text: unused-parameter
+    paths:
+      - vendor
+      - third_party$
+      - builtin$
+      - examples$
+formatters:
+  exclusions:
+    generated: lax
+    paths:
+      - third_party$
+      - builtin$
+      - examples$

app/discovery/discovery.go

@@ -539,7 +539,7 @@ func (m URLMapper) ping() (string, error) {
 
 	resp, err := client.Get(m.PingURL)
 	if err != nil {
-		errMsg := strings.Replace(err.Error(), "\"", "", -1)
+		errMsg := strings.ReplaceAll(err.Error(), "\"", "")
 		errMsg = fmt.Sprintf("failed to ping for health %s, %s", m.PingURL, errMsg)
 		return errMsg, fmt.Errorf("%s %s: %s, %v", m.Server, m.SrcMatch.String(), m.PingURL, errMsg)
 	}

app/discovery/provider/consulcatalog/consulcatalog.go

@@ -173,13 +173,14 @@ func (cc *ConsulCatalog) List() ([]discovery.URLMapper, error) {
 
 		if v, ok := c.Labels["reproxy.keep-host"]; ok {
 			enabled = true
-			if v == "true" || v == "yes" || v == "1" {
+			switch v {
+			case "true", "yes", "1":
 				t := true
 				keepHost = &t
-			} else if v == "false" || v == "no" || v == "0" {
+			case "false", "no", "0":
 				f := false
 				keepHost = &f
-			} else {
+			default:
 				log.Printf("[WARN] invalid value for reproxy.keep-host: %s", v)
 			}
 		}

app/main.go

@@ -299,7 +299,7 @@ func makeBasicAuth(htpasswdFile string) ([]string, error) {
 		basicAuthAllowed = strings.Split(string(data), "\n")
 		for i, v := range basicAuthAllowed {
 			basicAuthAllowed[i] = strings.TrimSpace(v)
-			basicAuthAllowed[i] = strings.Replace(basicAuthAllowed[i], "\t", "", -1)
+			basicAuthAllowed[i] = strings.ReplaceAll(basicAuthAllowed[i], "\t", "")
 		}
 	}
 	return basicAuthAllowed, nil

go.mod

@@ -1,6 +1,6 @@
 module github.com/umputun/reproxy
 
-go 1.23
+go 1.24
 require (
 	github.com/didip/tollbooth/v7 v7.0.2
 	github.com/go-pkgz/lgr v0.11.1