Skip to content

fix: upgrade extras/dagger OpenTelemetry SDK to v1.43.0#3276

Closed
chainloop-development[bot] wants to merge 1 commit into
mainfrom
chainloop/fix-ghsa-hfvc-g4fc-pqhx-20260708-141841
Closed

fix: upgrade extras/dagger OpenTelemetry SDK to v1.43.0#3276
chainloop-development[bot] wants to merge 1 commit into
mainfrom
chainloop/fix-ghsa-hfvc-g4fc-pqhx-20260708-141841

Conversation

@chainloop-development

@chainloop-development chainloop-development Bot commented Jul 8, 2026

Copy link
Copy Markdown

Summary

Updated the unused extras/dagger Go module to go.opentelemetry.io/otel/sdk v1.43.0, the first fixed release for GHSA-hfvc-g4fc-pqhx. This also refreshed the module's related OpenTelemetry dependencies and checksums to keep the module graph consistent.

Vulnerability Fixed

GHSA-hfvc-g4fc-pqhx (HIGH): opentelemetry-go BSD kenv command not using an absolute path enables PATH hijacking.

Changes Made

  • Bumped go.opentelemetry.io/otel/sdk in extras/dagger/go.mod from v1.38.0 to v1.43.0, the first fixed release identified by the assessment.
  • Updated the related OpenTelemetry module versions and go.sum entries in extras/dagger so the dependency graph remains internally consistent after the SDK bump.
  • Accepted the Go directive update in extras/dagger/go.mod required by the resolved module metadata for go.opentelemetry.io/otel/sdk@v1.43.0.

Verification

Ran syft dir:. -o cyclonedx-json | grype --only-fixed -o json from the repository root and scanned the patched source tree. Outcome: resolved — Grype did not report GHSA-hfvc-g4fc-pqhx, and Syft cataloged go.opentelemetry.io/otel/sdk@v1.43.0 from /extras/dagger/go.mod.

Risk Assessment

View the risk assessment in Chainloop

Review in cubic

migmartri added a commit to migmartri/chainloop that referenced this pull request Jul 9, 2026
Combined upgrade of the extras/dagger module dependencies to resolve
all four open security advisories in a single change, since the
individual bot PRs (chainloop-dev#3273, chainloop-dev#3274, chainloop-dev#3275, chainloop-dev#3276) conflict with each
other by modifying the same go.mod lines.

Advisories resolved:
- GHSA-p77j-4mvh-x3m3 (CRITICAL): gRPC-Go authorization bypass via
  missing leading slash in :path. Fixed by grpc v1.80.0 (>= v1.79.3).
- GHSA-mh2q-q3fh-2475 (HIGH): OpenTelemetry-Go baggage header DoS.
  Fixed by OTel v1.43.0 (>= v1.41.0).
- GHSA-hfvc-g4fc-pqhx (HIGH): OpenTelemetry-Go BSD kenv PATH hijack.
  Fixed by OTel SDK v1.43.0.
- GO-2026-5026 (HIGH): golang.org/x/net idna Punycode bypass.
  Fixed by x/net v0.55.0.

Verification performed:
- dagger develop (codegen) succeeds
- go build ./... compiles
- go vet ./... passes
- golangci-lint run: 0 issues
- go mod verify: all modules verified
- dagger call loads and instantiates the module
- with-instance function executes
- init function executes (fails at auth layer, not code)
- init --enterprise pulls enterprise image and executes

🤖 _Posted by Maximus bot on behalf of @migmartri_
migmartri added a commit to migmartri/chainloop that referenced this pull request Jul 9, 2026
Combined upgrade of the extras/dagger module dependencies to resolve
all four open security advisories in a single change, since the
individual bot PRs (chainloop-dev#3273, chainloop-dev#3274, chainloop-dev#3275, chainloop-dev#3276) conflict with each
other by modifying the same go.mod lines.

Advisories resolved:
- GHSA-p77j-4mvh-x3m3 (CRITICAL): gRPC-Go authorization bypass via
  missing leading slash in :path. Fixed by grpc v1.80.0 (>= v1.79.3).
- GHSA-mh2q-q3fh-2475 (HIGH): OpenTelemetry-Go baggage header DoS.
  Fixed by OTel v1.43.0 (>= v1.41.0).
- GHSA-hfvc-g4fc-pqhx (HIGH): OpenTelemetry-Go BSD kenv PATH hijack.
  Fixed by OTel SDK v1.43.0.
- GO-2026-5026 (HIGH): golang.org/x/net idna Punycode bypass.
  Fixed by x/net v0.55.0.

Assisted-by: Claude Code
Signed-off-by: Miguel Martinez <migmartri@gmail.com>
Signed-off-by: Miguel Martinez Trivino <miguel@chainloop.dev>
@migmartri migmartri closed this Jul 9, 2026
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.

1 participant