[9.3](backport #50644) Update go to 1.26.3 (#50842)

mergify[bot] · belimawr · web-flow · commit 0081ff47967c · 2026-05-22T15:49:54.000-04:00
* Fix go vet errors and remove unnecessary code * [libbeat] Update TestGetUrl so all tested URLs are valid * Revert #50686 now that only Go 1.26 is supported (cherry picked from commit 51e7321) # Conflicts: # libbeat/docs/version.asciidoc Assisted-By: Cursor, Model: Composer --------- Co-authored-by: Tiago Queiroz <tiago.queiroz@elastic.co>
diff --git a/.go-version b/.go-version
@@ -1 +1 @@
-1.25.9
+1.26.3
diff --git a/auditbeat/Dockerfile b/auditbeat/Dockerfile
@@ -1,4 +1,4 @@
-FROM golang:1.25.9-bookworm
+FROM golang:1.26.3-bookworm
 
 RUN \
     apt-get update \
diff --git a/changelog/fragments/1778788830-update-go-to-1.26.yaml b/changelog/fragments/1778788830-update-go-to-1.26.yaml
@@ -0,0 +1,45 @@
+# REQUIRED
+# Kind can be one of:
+# - breaking-change: a change to previously-documented behavior
+# - deprecation: functionality that is being removed in a later release
+# - bug-fix: fixes a problem in a previous version
+# - enhancement: extends functionality but does not break or fix existing behavior
+# - feature: new functionality
+# - known-issue: problems that we are aware of in a given version
+# - security: impacts on the security of a product or a user’s deployment.
+# - upgrade: important information for someone upgrading from a prior version
+# - other: does not fit into any of the other categories
+kind: feature
+
+# REQUIRED for all kinds
+# Change summary; a 80ish characters long description of the change.
+summary: Update Go to 1.26.3
+
+# REQUIRED for breaking-change, deprecation, known-issue
+# Long description; in case the summary is not enough to describe the change
+# this field accommodate a description without length limits.
+# description:
+
+# REQUIRED for breaking-change, deprecation, known-issue
+# impact:
+
+# REQUIRED for breaking-change, deprecation, known-issue
+# action:
+
+# REQUIRED for all kinds
+# Affected component; usually one of "elastic-agent", "fleet-server", "filebeat", "metricbeat", "auditbeat", "all", etc.
+component: all
+
+# AUTOMATED
+# OPTIONAL to manually add other PR URLs
+# PR URL: A link the PR that added the changeset.
+# If not present is automatically filled by the tooling finding the PR where this changelog fragment has been added.
+# NOTE: the tooling supports backports, so it's able to fill the original PR number instead of the backport PR number.
+# Please provide it if you are adding a fragment for a different PR.
+# pr: https://github.com/owner/repo/1234
+
+# AUTOMATED
+# OPTIONAL to manually add other issue URLs
+# Issue URL; optional; the GitHub issue related to this changeset (either closes or is part of).
+# If not present is automatically filled by the tooling with the issue linked to the PR number.
+# issue: https://github.com/owner/repo/1234
diff --git a/dev-tools/kubernetes/filebeat/Dockerfile.debug b/dev-tools/kubernetes/filebeat/Dockerfile.debug
@@ -1,4 +1,4 @@
-FROM golang:1.25.9-bookworm as builder
+FROM golang:1.26.3-bookworm as builder
 
 ENV PATH=/usr/bin:/bin:/usr/sbin:/sbin:/usr/local/bin:/go/bin:/usr/local/go/bin
 
diff --git a/dev-tools/kubernetes/heartbeat/Dockerfile.debug b/dev-tools/kubernetes/heartbeat/Dockerfile.debug
@@ -1,4 +1,4 @@
-FROM golang:1.25.9-bookworm as builder
+FROM golang:1.26.3-bookworm as builder
 
 ENV PATH=/usr/bin:/bin:/usr/sbin:/sbin:/usr/local/bin:/go/bin:/usr/local/go/bin
 
diff --git a/dev-tools/kubernetes/metricbeat/Dockerfile.debug b/dev-tools/kubernetes/metricbeat/Dockerfile.debug
@@ -1,4 +1,4 @@
-FROM golang:1.25.9-bookworm as builder
+FROM golang:1.26.3-bookworm as builder
 
 ENV PATH=/usr/bin:/bin:/usr/sbin:/sbin:/usr/local/bin:/go/bin:/usr/local/go/bin
 
diff --git a/filebeat/input/journald/pkg/journalctl/chroot_test.go b/filebeat/input/journald/pkg/journalctl/chroot_test.go
@@ -136,7 +136,7 @@ func TestNewFactoryChroot(t *testing.T) {
 		}
 	case err := <-waitErrChan:
 		t.Fatalf("error waiting for container to finish: %s", err)
-	case <-time.After(30 * time.Second):
+	case <-time.After(time.Minute):
 		t.Fatal("Container is stuck, stopping the test. Look at the container logs for more information.")
 	}
 }
diff --git a/go.mod b/go.mod
@@ -1,6 +1,6 @@
 module github.com/elastic/beats/v7
 
-go 1.25.9
+go 1.26.3
 
 require (
 	cloud.google.com/go/bigquery v1.74.0
diff --git a/heartbeat/Dockerfile b/heartbeat/Dockerfile
@@ -1,4 +1,4 @@
-FROM golang:1.25.9-bookworm
+FROM golang:1.26.3-bookworm
 
 RUN \
     apt-get update \
diff --git a/heartbeat/monitors/active/http/task_test.go b/heartbeat/monitors/active/http/task_test.go
@@ -97,8 +97,6 @@ func TestSplitHostnamePort(t *testing.T) {
 		},
 	}
 	for _, test := range urlTests {
-		test := test
-
 		t.Run(test.name, func(t *testing.T) {
 			url := &url.URL{
 				Scheme: test.scheme,
@@ -120,7 +118,7 @@ func TestSplitHostnamePort(t *testing.T) {
 					t.Errorf("Unexpected host for %#v: expected %q, got %q", request, test.expectedHost, host)
 				}
 				if port != test.expectedPort {
-					t.Errorf("Unexpected port for %#v: expected %q, got %q", request, test.expectedPort, port)
+					t.Errorf("Unexpected port for %#v: expected %d, got %d", request, test.expectedPort, port)
 				}
 			}
 
diff --git a/libbeat/common/url_test.go b/libbeat/common/url_test.go
@@ -61,20 +61,18 @@ func TestGetUrl(t *testing.T) {
 		"192.168.1.1:80/hello": "http://192.168.1.1:80/hello",
 
 		// schema + ipv6
-		"http://[2001:db8::1]:80":                              "http://[2001:db8::1]:80",
-		"http://[2001:db8::1]":                                 "http://[2001:db8::1]:9200",
-		"https://[2001:db8::1]:9200":                           "https://[2001:db8::1]:9200",
-		"http://FE80:0000:0000:0000:0202:B3FF:FE1E:8329":       "http://[FE80:0000:0000:0000:0202:B3FF:FE1E:8329]:9200",
-		"http://[2001:db8::1]:80/hello":                        "http://[2001:db8::1]:80/hello",
-		"http://[2001:db8::1]/hello":                           "http://[2001:db8::1]:9200/hello",
-		"https://[2001:db8::1]:9200/hello":                     "https://[2001:db8::1]:9200/hello",
-		"http://FE80:0000:0000:0000:0202:B3FF:FE1E:8329/hello": "http://[FE80:0000:0000:0000:0202:B3FF:FE1E:8329]:9200/hello",
+		"http://[2001:db8::1]:80":                                "http://[2001:db8::1]:80",
+		"http://[2001:db8::1]":                                   "http://[2001:db8::1]:9200",
+		"https://[2001:db8::1]:9200":                             "https://[2001:db8::1]:9200",
+		"http://[FE80:0000:0000:0000:0202:B3FF:FE1E:8329]":       "http://[FE80:0000:0000:0000:0202:B3FF:FE1E:8329]:9200",
+		"http://[2001:db8::1]:80/hello":                          "http://[2001:db8::1]:80/hello",
+		"http://[2001:db8::1]/hello":                             "http://[2001:db8::1]:9200/hello",
+		"https://[2001:db8::1]:9200/hello":                       "https://[2001:db8::1]:9200/hello",
+		"http://[FE80:0000:0000:0000:0202:B3FF:FE1E:8329]/hello": "http://[FE80:0000:0000:0000:0202:B3FF:FE1E:8329]:9200/hello",
 
 		// no schema + ipv6
-		"2001:db8::1":            "http://[2001:db8::1]:9200",
 		"[2001:db8::1]:80":       "http://[2001:db8::1]:80",
 		"[2001:db8::1]":          "http://[2001:db8::1]:9200",
-		"2001:db8::1/hello":      "http://[2001:db8::1]:9200/hello",
 		"[2001:db8::1]:80/hello": "http://[2001:db8::1]:80/hello",
 		"[2001:db8::1]/hello":    "http://[2001:db8::1]:9200/hello",
 	}
diff --git a/libbeat/docs/version.asciidoc b/libbeat/docs/version.asciidoc
@@ -1,8 +1,8 @@
 :stack-version: 9.3.4
 :doc-branch: 9.3
-:go-version: 1.25.8
+:go-version: 1.26.3
 :release-state: unreleased
 :python: 3.7
 :docker: 1.12
 :docker-compose: 2.0
-:libpcap: 0.8
+:libpcap: 0.8
diff --git a/libbeat/reader/readfile/fs_metafields_other.go b/libbeat/reader/readfile/fs_metafields_other.go
@@ -49,7 +49,7 @@ func setFileSystemMetadata(fi file.ExtendedFileInfo, fields mapstr.M, includeOwn
 	if includeOwner {
 		o, err := user.LookupId(strconv.FormatUint(osstate.UID, 10))
 		if err != nil {
-			return fmt.Errorf("failed to lookup uid %q: %w", osstate.UID, err)
+			return fmt.Errorf("failed to lookup uid %d: %w", osstate.UID, err)
 		}
 		_, err = fields.Put(ownerKey, o.Username)
 		if err != nil {
@@ -60,7 +60,7 @@ func setFileSystemMetadata(fi file.ExtendedFileInfo, fields mapstr.M, includeOwn
 	if includeGroup {
 		g, err := user.LookupGroupId(strconv.FormatUint(osstate.GID, 10))
 		if err != nil {
-			return fmt.Errorf("failed to lookup gid %q: %w", osstate.GID, err)
+			return fmt.Errorf("failed to lookup gid %d: %w", osstate.GID, err)
 		}
 		_, err = fields.Put(groupKey, g.Name)
 		if err != nil {
diff --git a/metricbeat/Dockerfile b/metricbeat/Dockerfile
@@ -1,4 +1,4 @@
-FROM golang:1.25.9-bookworm
+FROM golang:1.26.3-bookworm
 COPY --from=docker:26.0.0-alpine3.19 /usr/local/bin/docker  /usr/local/bin/
 COPY --from=docker:26.0.0-alpine3.19 /usr/local/libexec/docker/cli-plugins/docker-compose /usr/local/lib/docker/cli-plugins/docker-compose
 
diff --git a/metricbeat/module/http/_meta/Dockerfile b/metricbeat/module/http/_meta/Dockerfile
@@ -1,4 +1,4 @@
-FROM golang:1.25.9-bookworm
+FROM golang:1.26.3-bookworm
 
 COPY test/main.go main.go
 
diff --git a/metricbeat/module/vsphere/_meta/Dockerfile b/metricbeat/module/vsphere/_meta/Dockerfile
@@ -1,5 +1,5 @@
 ARG VSPHERE_GOLANG_VERSION
-FROM golang:1.25.9-bookworm
+FROM golang:1.26.3-bookworm
 
 RUN apt-get install curl git
 RUN go install github.com/vmware/govmomi/vcsim@v0.30.4
diff --git a/packetbeat/Dockerfile b/packetbeat/Dockerfile
@@ -1,4 +1,4 @@
-FROM golang:1.25.9-bookworm
+FROM golang:1.26.3-bookworm
 
 RUN \
     apt-get update \
diff --git a/x-pack/filebeat/input/http_endpoint/input.go b/x-pack/filebeat/input/http_endpoint/input.go
@@ -479,7 +479,7 @@ func (m *mux) ServeHTTP(w http.ResponseWriter, r *http.Request) {
 	if clean != r.URL.Path {
 		url := *r.URL
 		url.Path = clean
-		http.Redirect(w, r, url.String(), cleanRedirectCode)
+		http.Redirect(w, r, url.String(), http.StatusTemporaryRedirect)
 		return
 	}
 	m.mu.RLock()
diff --git a/x-pack/filebeat/input/http_endpoint/redirect_go126.go b/x-pack/filebeat/input/http_endpoint/redirect_go126.go
diff --git a/x-pack/filebeat/input/http_endpoint/redirect_pre126.go b/x-pack/filebeat/input/http_endpoint/redirect_pre126.go
diff --git a/x-pack/metricbeat/module/stan/_meta/Dockerfile b/x-pack/metricbeat/module/stan/_meta/Dockerfile
@@ -2,7 +2,7 @@ ARG STAN_VERSION=0.15.1
 FROM nats-streaming:$STAN_VERSION
 
 # build stage
-FROM golang:1.25.9-bookworm AS build-env
+FROM golang:1.26.3-bookworm AS build-env
 RUN apt-get install git mercurial gcc
 RUN git clone https://github.com/nats-io/stan.go.git /stan-go
 RUN cd /stan-go/examples/stan-bench && git checkout tags/v0.5.2 && go build .

Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,4 @@`
`1`		`-FROM golang:1.25.9-bookworm`
	`1`	`+FROM golang:1.26.3-bookworm`
`2`	`2`
`3`	`3`	`RUN \`
`4`	`4`	`apt-get update \`
Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,4 @@`
`1`		`-FROM golang:1.25.9-bookworm as builder`
	`1`	`+FROM golang:1.26.3-bookworm as builder`
`2`	`2`
`3`	`3`	`ENV PATH=/usr/bin:/bin:/usr/sbin:/sbin:/usr/local/bin:/go/bin:/usr/local/go/bin`
`4`	`4`
Original file line number	Diff line number	Diff line change
`@@ -136,7 +136,7 @@ func TestNewFactoryChroot(t *testing.T) {`
`136`	`136`	`}`
`137`	`137`	`case err := <-waitErrChan:`
`138`	`138`	`t.Fatalf("error waiting for container to finish: %s", err)`
`139`		`- case <-time.After(30 * time.Second):`
	`139`	`+ case <-time.After(time.Minute):`
`140`	`140`	`t.Fatal("Container is stuck, stopping the test. Look at the container logs for more information.")`
`141`	`141`	`}`
`142`	`142`	`}`
Original file line number	Diff line number	Diff line change
`@@ -97,8 +97,6 @@ func TestSplitHostnamePort(t *testing.T) {`
`97`	`97`	`},`
`98`	`98`	`}`
`99`	`99`	`for _, test := range urlTests {`
`100`		`- test := test`
`101`		`-`
`102`	`100`	`t.Run(test.name, func(t *testing.T) {`
`103`	`101`	`url := &url.URL{`
`104`	`102`	`Scheme: test.scheme,`
`@@ -120,7 +118,7 @@ func TestSplitHostnamePort(t *testing.T) {`
`120`	`118`	`t.Errorf("Unexpected host for %#v: expected %q, got %q", request, test.expectedHost, host)`
`121`	`119`	`}`
`122`	`120`	`if port != test.expectedPort {`
`123`		`- t.Errorf("Unexpected port for %#v: expected %q, got %q", request, test.expectedPort, port)`
	`121`	`+ t.Errorf("Unexpected port for %#v: expected %d, got %d", request, test.expectedPort, port)`
`124`	`122`	`}`
`125`	`123`	`}`
`126`	`124`