Go: Bump to v1.24.1. (#12935)

.github/workflows/golangci-lint.yml

@@ -30,5 +30,4 @@ jobs:
       - name: golangci-lint
         uses: golangci/golangci-lint-action@2226d7cb06a077cd73e56eedd38eecad18e5d837 # v6.5.0
         with:
-          version: v1.62
           only-new-issues: true

GOLANG_VERSION

@@ -1 +1 @@
-1.23.6
+1.24.1

go.mod

@@ -1,6 +1,6 @@
 module k8s.io/ingress-nginx
 
-go 1.23.6
+go 1.24.1
 
 require (
 	dario.cat/mergo v1.0.1

images/custom-error-pages/rootfs/go.mod

@@ -1,6 +1,6 @@
 module k8s.io/ingress-nginx/custom-error-pages
 
-go 1.23.6
+go 1.24.1
 
 require github.com/prometheus/client_golang v1.21.1
 

images/ext-auth-example-authsvc/rootfs/go.mod

@@ -1,6 +1,6 @@
 module example.com/authsvc
 
-go 1.23.6
+go 1.24.1
 
 require k8s.io/apimachinery v0.32.2
 

images/kube-webhook-certgen/rootfs/go.mod

@@ -1,6 +1,6 @@
 module github.com/jet/kube-webhook-certgen
 
-go 1.23.6
+go 1.24.1
 
 require (
 	github.com/onrik/logrus v0.11.0

internal/ingress/annotations/mirror/main_test.go

@@ -117,7 +117,7 @@ func TestParse(t *testing.T) {
 		ing.SetAnnotations(testCase.annotations)
 		result, err := ap.Parse(ing)
 		if err != nil {
-			t.Errorf(err.Error())
+			t.Error(err)
 		}
 		if !reflect.DeepEqual(result, testCase.expected) {
 			t.Errorf("expected %+v but returned %+v, annotations: %s", testCase.expected, result, testCase.annotations)

internal/ingress/metric/collectors/nginx_status_test.go

@@ -110,7 +110,7 @@ func TestStatusCollector(t *testing.T) {
 					w.WriteHeader(http.StatusOK)
 
 					if r.URL.Path == "/nginx_status" {
-						_, err := fmt.Fprintf(w, c.mock)
+						_, err := fmt.Fprint(w, c.mock)
 						if err != nil {
 							t.Fatal(err)
 						}

magefiles/go.mod

@@ -1,6 +1,6 @@
 module k8s.io/ingress-nginx/magefiles
 
-go 1.23.6
+go 1.24.1
 
 require (
 	github.com/blang/semver/v4 v4.0.0