Skip to content

fix: Fix multiple vulnerabilities by upgrading to go1.25.9 and otel-go#6713

Merged
PeterSchafer merged 2 commits into
mainfrom
fix/upgrade_otel_go
Apr 8, 2026
Merged

fix: Fix multiple vulnerabilities by upgrading to go1.25.9 and otel-go#6713
PeterSchafer merged 2 commits into
mainfrom
fix/upgrade_otel_go

Conversation

@PeterSchafer
Copy link
Copy Markdown
Contributor

Pull Request Submission Checklist

  • Follows CONTRIBUTING guidelines
  • Commit messages
    are release-note ready, emphasizing
    what was changed, not how.
  • Includes detailed description of changes
  • Contains risk assessment (Low | Medium | High)
  • Highlights breaking API changes (if applicable)
  • Links to automated tests covering new functionality
  • Includes manual testing instructions (if necessary)
  • Updates relevant GitBook documentation (PR link: ___)
  • Includes product update to be announced in the next stable release notes

What does this PR do?

This PR updates multiple dependencies to address high severity vulnerabilities.

Where should the reviewer start?

How should this be manually tested?

What's the product update that needs to be communicated to CLI users?

N/A

Risk assessment (Low | Medium | High)?

Low, as the dependency bumps are either fix or minor upgrades.

SNYK-GOLANG-GOOPENTELEMETRYIOOTELBAGGAGE-15928416
SNYK-GOLANG-GOOPENTELEMETRYIOOTELINTERNALGLOBAL-15928418
SNYK-GOLANG-GOOPENTELEMETRYIOOTELPROPAGATION-15928420
SNYK-GOLANG-STDCRYPTOTLS-15928849
@PeterSchafer PeterSchafer requested review from a team as code owners April 8, 2026 10:35
@snyk-io
Copy link
Copy Markdown

snyk-io Bot commented Apr 8, 2026

Snyk checks have passed. No issues have been found so far.

Status Scan Engine Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues
Licenses 0 0 0 0 0 issues
Code Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 8, 2026

Warnings
⚠️ There are multiple commits on your branch, please squash them locally before merging!

Generated by 🚫 dangerJS against d26e83f

@snyk-pr-review-bot
Copy link
Copy Markdown

PR Reviewer Guide 🔍

🧪 No relevant tests
🔒 No security concerns identified
⚡ Recommended focus areas for review

OpenTelemetry Version Mismatch 🟠 [major]

The PR upgrades go.opentelemetry.io/otel, otel/metric, and otel/trace to version 1.41.0 (lines 247, 248, 251), but leaves go.opentelemetry.io/otel/sdk and go.opentelemetry.io/otel/sdk/metric at version 1.40.0. In the OpenTelemetry Go ecosystem, API and SDK versions must be synchronized. A version mismatch can lead to silent failures where telemetry data is not exported because the newer API does not find the expected features in the older SDK. Furthermore, if the 'multiple vulnerabilities' referenced in the PR title are in the SDK logic, this change fails to remediate them.

go.opentelemetry.io/otel v1.41.0 // indirect
go.opentelemetry.io/otel/metric v1.41.0 // indirect
go.opentelemetry.io/otel/sdk v1.40.0 // indirect
go.opentelemetry.io/otel/sdk/metric v1.40.0 // indirect
go.opentelemetry.io/otel/trace v1.41.0 // indirect
📚 Repository Context Analyzed

This review considered 5 relevant code sections from 4 files (average relevance: 0.89)

@PeterSchafer PeterSchafer enabled auto-merge April 8, 2026 10:54
@PeterSchafer PeterSchafer merged commit 3523206 into main Apr 8, 2026
9 checks passed
@PeterSchafer PeterSchafer deleted the fix/upgrade_otel_go branch April 8, 2026 11:16
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.

3 participants