fix(deps): bump golang.org/x/text and google.golang.org/grpc for HIGH CVEs - #147
Open
dwertent wants to merge 1 commit into
Open
fix(deps): bump golang.org/x/text and google.golang.org/grpc for HIGH CVEs#147dwertent wants to merge 1 commit into
dwertent wants to merge 1 commit into
Conversation
… CVEs Remediates two HIGH severity findings in the shared root go.mod used by the Go components, including op-node: - golang.org/x/text v0.38.0 -> v0.39.0 (CVE-2026-56852, HIGH) - google.golang.org/grpc v1.79.3 -> v1.82.1 (GHSA-hrxh-6v49-42gf, HIGH) golang.org/x/net is already at v0.56.0 (fixed) and is left unchanged. Scope: only the root Go manifests (go.mod, go.sum) are touched. The Rust / Cargo tree (rust/**) and all other non-Go components are deliberately untouched. Reported by Pipeline / CVE - Trigger run 30995014323 https://github.com/kaleido-io/kaleido-app-cloud/actions/runs/30995014323 Signed-off-by: David Wertenteil <david.wertenteil@kaleido.io>
There was a problem hiding this comment.
Pull request overview
This PR remediates two HIGH severity dependency findings in the repository’s root Go module manifests (shared across Go components like op-node) by bumping golang.org/x/text and google.golang.org/grpc, along with the expected transitive updates produced by go mod tidy.
Changes:
- Bump
golang.org/x/texttov0.39.0andgoogle.golang.org/grpctov1.82.1in the rootgo.mod. - Pull forward related transitive module versions (e.g.,
golang.org/x/mod,golang.org/x/tools,golang.org/x/telemetry, andgoogle.golang.org/genproto/...) and update checksums accordingly ingo.sum.
Reviewed changes
Copilot reviewed 1 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| go.mod | Updates root module requirements to the patched versions for x/text and grpc, plus tidy-driven transitive bumps. |
| go.sum | Refreshes module checksums to match the updated dependency graph after the version bumps. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Remediates two HIGH severity findings in the shared root
go.modused by the Go components, includingop-node.golang.org/x/textv0.38.0v0.39.0google.golang.org/grpcv1.79.3v1.82.1Transitively pulled forward by
go mod tidy:go.opentelemetry.io/otel/sdk/metric-> v1.43.0,golang.org/x/mod-> v0.37.0,golang.org/x/tools-> v0.47.0,golang.org/x/telemetry, andgoogle.golang.org/genproto/googleapis/{api,rpc}.Base branch
Targets
release-op-node/v1.18.2, which is the authoritative build pin consumed by the wrapper workflow that produces theop-nodeimage. The fix must land here to reach the shipped image.Monorepo scoping
This repository contains multiple components and ecosystems, so the change was deliberately kept minimal. The vulnerable Go modules live in the root
go.mod, which is shared across the Go components, so the root manifests are the correct and only place to fix this.Complete list of files changed — 2 files:
go.work/go.work.sumin this repository, so none were touched.op-node/**required changes — the bump is manifest-only.rust/**) was not touched, nor was any other non-Go component. Verified withgit statusbefore committing.Explicitly out of scope
golang.org/x/netis already atv0.56.0(fixed) and was deliberately left unchanged.Verification
go build ./op-node/...— passesgo build -o /dev/null ./op-node/cmd/main.go— passes (binary links)go build ./op-service/... ./op-batcher/... ./op-proposer/... ./op-supervisor/... ./op-conductor/...— passes, confirming the shared rootgo.modbump does not break sibling Go componentsgo list -m:golang.org/x/text v0.39.0google.golang.org/grpc v1.82.1golang.org/x/net v0.56.0(unchanged, already fixed)vulnchecktarget exists in this repo'sMakefile/justfile.One pre-existing build failure, unrelated to this change
A repo-wide
go build ./...fails with:This is a missing generated/embedded asset that requires a codegen step, and it reproduces identically on the unmodified base branch. It is unrelated to this dependency bump and does not affect
op-node.Reported by Pipeline / CVE - Trigger run 30995014323 (https://github.com/kaleido-io/kaleido-app-cloud/actions/runs/30995014323)