Skip to content

fix: upgrade packages to fix new CVEs#313

Merged
iFurySt merged 7 commits intoAmoyLab:mainfrom
kx-byte:main
Apr 22, 2026
Merged

fix: upgrade packages to fix new CVEs#313
iFurySt merged 7 commits intoAmoyLab:mainfrom
kx-byte:main

Conversation

@kx-byte
Copy link
Copy Markdown
Contributor

@kx-byte kx-byte commented Apr 22, 2026

Summary by Sourcery

Update Go toolchain and core dependencies to newer versions, primarily to address security vulnerabilities in transitive libraries.

Enhancements:

  • Upgrade Go version and a wide range of direct and indirect dependencies, including OpenTelemetry, golang.org/x packages, gRPC, and various HTTP/codec utilities, to their latest compatible releases.
  • Remove the direct dependency on github.com/mattn/go-sqlite3 now that the project relies on alternative SQLite drivers.

@sourcery-ai
Copy link
Copy Markdown
Contributor

sourcery-ai Bot commented Apr 22, 2026

Reviewer's Guide

This PR updates the Go toolchain version and refreshes a set of direct and indirect Go module dependencies (notably OpenTelemetry, golang.org/x packages, gRPC, QUIC, and various encoding and DB-related libraries) to address new CVEs while cleaning up some now-unneeded indirect requirements and normalizing some indirect deps as direct ones.

File-Level Changes

Change Details Files
Bump Go toolchain and key first-party dependencies to newer, likely non-vulnerable versions.
  • Update Go version from 1.24.1 to 1.25.0 in module definition
  • Upgrade OpenTelemetry core, SDK, exporters, and Gin/http instrumentation from 1.38.0/0.63.0 to 1.43.0/0.65.0 (with otlptracegrpc at 1.40.0)
  • Bump security-sensitive golang.org/x packages (crypto, oauth2, text, net, sync, sys, arch) to newer minor versions
  • Upgrade grpc, genproto, protobuf, and otlp proto packages to newer releases
  • Refresh various ecosystem libraries (e.g., gin-related, validator, pgx, quic-go, gopkg, sonic, mimetype, go-yaml) to current minor/patch versions
go.mod
go.sum
Adjust dependency graph by promoting some previously indirect dependencies to direct and removing unused ones.
  • Promote github.com/andybalholm/brotli and github.com/klauspost/compress from indirect to direct requirements, likely to match actual import usage and stabilize versions
  • Remove unused or obsolete indirect requirements such as github.com/mattn/go-sqlite3 and github.com/rogpeppe/go-internal
  • Drop the direct gorm.io/driver/sqlite requirement while retaining modernc.org/sqlite via glebarez/sqlite, simplifying SQLite driver usage
  • Add new indirect dependencies like github.com/bytedance/gopkg and update existing indirect sets without changing application code
go.mod
go.sum

Tips and commands

Interacting with Sourcery

  • Trigger a new review: Comment @sourcery-ai review on the pull request.
  • Continue discussions: Reply directly to Sourcery's review comments.
  • Generate a GitHub issue from a review comment: Ask Sourcery to create an
    issue from a review comment by replying to it. You can also reply to a
    review comment with @sourcery-ai issue to create an issue from it.
  • Generate a pull request title: Write @sourcery-ai anywhere in the pull
    request title to generate a title at any time. You can also comment
    @sourcery-ai title on the pull request to (re-)generate the title at any time.
  • Generate a pull request summary: Write @sourcery-ai summary anywhere in
    the pull request body to generate a PR summary at any time exactly where you
    want it. You can also comment @sourcery-ai summary on the pull request to
    (re-)generate the summary at any time.
  • Generate reviewer's guide: Comment @sourcery-ai guide on the pull
    request to (re-)generate the reviewer's guide at any time.
  • Resolve all Sourcery comments: Comment @sourcery-ai resolve on the
    pull request to resolve all Sourcery comments. Useful if you've already
    addressed all the comments and don't want to see them anymore.
  • Dismiss all Sourcery reviews: Comment @sourcery-ai dismiss on the pull
    request to dismiss all existing Sourcery reviews. Especially useful if you
    want to start fresh with a new review - don't forget to comment
    @sourcery-ai review to trigger a new review!

Customizing Your Experience

Access your dashboard to:

  • Enable or disable review features such as the Sourcery-generated pull request
    summary, the reviewer's guide, and others.
  • Change the review language.
  • Add, remove or edit custom review instructions.
  • Adjust other review settings.

Getting Help

Copy link
Copy Markdown
Contributor

@sourcery-ai sourcery-ai Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey - I've left some high level feedback:

  • The OpenTelemetry dependencies are mostly aligned at v1.43.0, but go.opentelemetry.io/otel/exporters/otlp/otlptrace/otlptracegrpc is pinned at v1.40.0, which may cause subtle incompatibilities; consider aligning it with the rest of the OTel stack.
  • You removed gorm.io/driver/sqlite (and the direct github.com/mattn/go-sqlite3 dependency) while still depending on sqlite via github.com/glebarez/go-sqlite; double‑check that all call sites expecting the gorm sqlite driver are updated or that the old driver is intentionally no longer needed.
  • Several golang.org/x/* and google.golang.org/genproto modules are pinned to pseudo‑versions with future timestamps (e.g., 2025/2026), which can make builds less reproducible and may not correspond to real upstream tags; consider revisiting these versions to ensure they are intentional and resolvable via your module proxy.
Prompt for AI Agents
Please address the comments from this code review:

## Overall Comments
- The OpenTelemetry dependencies are mostly aligned at v1.43.0, but `go.opentelemetry.io/otel/exporters/otlp/otlptrace/otlptracegrpc` is pinned at v1.40.0, which may cause subtle incompatibilities; consider aligning it with the rest of the OTel stack.
- You removed `gorm.io/driver/sqlite` (and the direct `github.com/mattn/go-sqlite3` dependency) while still depending on sqlite via `github.com/glebarez/go-sqlite`; double‑check that all call sites expecting the gorm sqlite driver are updated or that the old driver is intentionally no longer needed.
- Several `golang.org/x/*` and `google.golang.org/genproto` modules are pinned to pseudo‑versions with future timestamps (e.g., 2025/2026), which can make builds less reproducible and may not correspond to real upstream tags; consider revisiting these versions to ensure they are intentional and resolvable via your module proxy.

Sourcery is free for open source - if you like our reviews please consider sharing them ✨
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.

@iFurySt iFurySt merged commit a104b92 into AmoyLab:main Apr 22, 2026
6 checks passed
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