Skip to content

chore: bump OTel vulnerable dep#916

Merged
mbevc1 merged 1 commit into
mainfrom
20260529_bump_otel
May 29, 2026
Merged

chore: bump OTel vulnerable dep#916
mbevc1 merged 1 commit into
mainfrom
20260529_bump_otel

Conversation

@mbevc1

@mbevc1 mbevc1 commented May 29, 2026

Copy link
Copy Markdown
Contributor

Use patched upstream dependency and fix Snyk reporting a vulnerability.

@mbevc1 mbevc1 added go Pull requests that update go code security labels May 29, 2026
@mbevc1 mbevc1 enabled auto-merge (squash) May 29, 2026 11:54
@claude

claude Bot commented May 29, 2026

Copy link
Copy Markdown
Contributor

Claude finished @mbevc1's task in 2s —— View job


Review: PR #916 — chore: bump OTel vulnerable dep

  • Phase 1: Static analysis (go build, go vet, go mod tidy)
  • Phase 2: Review dependency changes
  • Phase 3: Post review findings

Static Analysis

All checks pass:

  • go build ./... — compiles cleanly
  • go vet ./... — no issues
  • go mod tidy && git diff --exit-code go.mod go.sum — no drift, dependencies are consistent

Review

This PR bumps three OTel core API modules from v1.43.0 → v1.44.0:

  • go.opentelemetry.io/otel
  • go.opentelemetry.io/otel/metric
  • go.opentelemetry.io/otel/trace

The corresponding go.sum entries are updated accordingly. All OTel modules are indirect dependencies (not imported directly by this project).

Version consistency: The otel/sdk and otel/exporters/* modules remain at v1.43.0. This is fine — these are independently-versioned modules in the upstream OTel Go project, and go mod tidy confirms compatibility.

No issues found. The change is minimal, targeted, and correctly addresses the vulnerable dependency. The go.sum hashes are updated to match the new versions.

LGTM — clean dependency bump to patch a Snyk-reported vulnerability. 👍

@mbevc1 mbevc1 merged commit 5d1e130 into main May 29, 2026
16 checks passed
@mbevc1 mbevc1 deleted the 20260529_bump_otel branch May 29, 2026 12:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

go Pull requests that update go code security

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants