Skip to content

fix: bump vulnerable Go deps in the 2.2.0 rock - #58

Closed
iamsharduld wants to merge 1 commit into
mainfrom
fix/bump-vulnerable-go-deps
Closed

fix: bump vulnerable Go deps in the 2.2.0 rock#58
iamsharduld wants to merge 1 commit into
mainfrom
fix/bump-vulnerable-go-deps

Conversation

@iamsharduld

Copy link
Copy Markdown
Contributor

Bumps golang.org/x/text v0.37.0 to v0.39.0 and google.golang.org/grpc v1.81.0 to
v1.82.1 during the build.

Upstream v2.2.0 pins both at versions with known HIGH advisories (CVE-2026-56852 and
GHSA-hrxh-6v49-42gf), which fails the OCI Factory vulnerability gate and is blocking
oci-factory#1160. Both already have
fixed releases upstream, so picking them up seemed better than adding them to
ignored-vulnerabilities. Note they are not in the suppression list the 1.18.1 entry
carries, so restoring that list would not have fixed the gate on its own.

Verified locally with trivy 0.69.3 using the same flags OCI Factory runs: the stock build
exits 1 with 4 HIGH findings, with this change it exits 0 with none. pyroscope --version
still reports the same upstream revision, and a monolithic v2 run came up cleanly (raft,
memberlist, and profiles flushed to object storage) so the grpc bump does not disturb the
internal gRPC paths.

Can be dropped once upstream bumps these itself.

Upstream v2.2.0 pins golang.org/x/text at v0.37.0 and google.golang.org/grpc
at v1.81.0, both of which carry known HIGH advisories (CVE-2026-56852 and
GHSA-hrxh-6v49-42gf). That fails the OCI Factory vulnerability gate and is
blocking canonical/oci-factory#1160.

Both have fixed releases upstream, so bump to v0.39.0 and v1.82.1 rather than
adding them to ignored-vulnerabilities. Neither is in the suppression list the
1.18.1 entry carries, so restoring that list would not have fixed the gate.

Can be dropped once upstream bumps these itself.
@michaeldmitry

Copy link
Copy Markdown
Contributor

can be closed in favor of #59

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants