Skip to content

Security/endorlabs findings - #5181

Open
sourabhpatel-sumo wants to merge 2 commits into
develop/6from
security/endorlabs-findings
Open

Security/endorlabs findings#5181
sourabhpatel-sumo wants to merge 2 commits into
develop/6from
security/endorlabs-findings

Conversation

@sourabhpatel-sumo

Copy link
Copy Markdown

What is this change?

Bumps actions/checkout, actions/setup-go, dominikh/staticcheck-action, and the semgrep container image in CI, plus golang.org/x/crypto(v0.54.0→v0.56.0) and google.golang.org/grpc (v1.82.1→v1.83.2) in both go.mod and types/go.mod.

Why is this change necessary?

Clears CVEs flagged by an Endor Labs scan against develop/6:

  • grpc → v1.83.2: fixes GHSA-vp52-pcj8-j9qc (CVE-2026-84304), a reachable HTTP/2 DATA frame heap-exhaustion vuln in the transport this repo serves over etcd. Endor's own recommendation (v1.83.1) still leaves GHSA-2v4p-qf9q-27wj open — v1.83.2 clears both.
  • x/crypto → v0.56.0: fixes 3 SSH CVEs (GO-2026-6303/6354/6355). Not reachable here (only bcrypt is imported from this module), but bumped for a clean scan.
  • CI actions: checkout v3/v4→v7, setup-go v5→v7, staticcheck-action v1.3.1→v1.4.1 clears 34 of 45 npm-in-action findings (uuid, form-data, minimatch CVEs pulled in via staticcheck-action's bundled setup-go-faster dependency). Also swapped the semgrep container from returntocorp/semgrep to semgrep/semgrep — the former's Docker Hub page says "We've moved!"; both tags currently resolve to the same image digest, so this is a rename with no behavioral change.

Does your change need a Changelog entry?

Yes — not yet added. Should go under ## [Unreleased]### Security in CHANGELOG-6.md, following the pattern already used for the v6.14.2 entry (e.g. "Resolved CVE-2026-84304 (grpc HTTP/2 DoS) and 3 SSH CVEs by upgrading google.golang.org/grpc to v1.83.2 and golang.org/x/crypto to v0.56.0").

How did you verify this change?

  • Confirmed checkout@v7's fork-PR gate doesn't apply (no pull_request_target/workflow_run triggers in this repo's workflows).
  • Confirmed GOTOOLCHAIN=local is satisfied by the pinned go-version matching go.mod.
  • Confirmed the Alpine semgrep container has a musl node24 runner build available before switching images.
  • Confirmed returntocorp/semgrep and semgrep/semgrep currently resolve to the same image digest.

grpc: fixes GHSA-vp52-pcj8-j9qc (CVE-2026-84304), a reachable HTTP/2
DATA frame heap exhaustion in the transport this repo serves over
etcd. Endor recommended v1.83.1, which still leaves
GHSA-2v4p-qf9q-27wj open; v1.83.2 clears it too.

x/crypto: fixes 3 SSH CVEs (GO-2026-6303/6354/6355). Unreachable here
- the repo imports only bcrypt - but bumped for a clean scan.

Signed-off-by: Sourabh Patel <sourabh.patel.ctr@sumologic.com>
checkout v3/v4 -> v7, setup-go v5 -> v7, staticcheck-action v1.3.1 ->
v1.4.1. Clears 34 of 45 npm-in-action findings from the Endor scan,
including all uuid, form-data and minimatch CVEs pulled in via
staticcheck-action's bundled setup-go-faster dependency.

Verified: no pull_request_target/workflow_run triggers (checkout v7's
fork-PR gate doesn't apply), GOTOOLCHAIN=local satisfied by the pinned
go-version matching go.mod, and the Alpine semgrep container has a
musl node24 runner build available.

Also switch the semgrep container from returntocorp/semgrep to
semgrep/semgrep - the former's Docker Hub description says "We've
moved!". Confirmed both tags currently resolve to the same image
digest, so this is a rename with no behavioral change.

Signed-off-by: Sourabh Patel <sourabh.patel.ctr@sumologic.com>
@sourabhpatel-sumo
sourabhpatel-sumo force-pushed the security/endorlabs-findings branch from e20e4fd to e1b4925 Compare September 9, 2026 16:30
@sourabhpatel-sumo

Copy link
Copy Markdown
Author

Need to wait for upstream sensu/core#55 to be merged.

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants