Skip to content

security: fix Dependabot alerts for grpc and OTel - #3835

Merged
mdelapenya merged 2 commits into
mainfrom
security/fix-dependabot-alerts
Aug 4, 2026
Merged

security: fix Dependabot alerts for grpc and OTel#3835
mdelapenya merged 2 commits into
mainfrom
security/fix-dependabot-alerts

Conversation

@mdelapenya

Copy link
Copy Markdown
Member

What does this PR do?

Fixes actionable Dependabot security alerts with one commit per vulnerability:

  • GHSA-hrxh-6v49-42gf: Bump google.golang.org/grpc to v1.82.1 in dex, couchbase, etcd, milvus, pinecone modules
  • GHSA-w8rr-5gcm-pp58: Bump go.opentelemetry.io/otel/exporters/otlp/otlplog/otlploghttp to v0.20.0 (+ related OTel log/bridge packages) in grafana-lgtm module

Alerts NOT addressed (no fix available)

  • github.com/docker/docker (compose, chroma): v29.3.1 was never published as a Go module — the latest available is v28.5.2+incompatible. Upstream docker/compose needs to migrate away from the docker/docker import path first.
  • github.com/weaviate/weaviate (weaviate): all patched versions (>= v1.38.0-rc.0) require go 1.26, incompatible with this repo's go 1.25.0 directive. Needs a Go version bump first.
  • pymdown-extensions (Pipfile.lock): Python dependency, outside scope of Go dependency management.

mdelapenya and others added 2 commits August 4, 2026 12:03
Fixes GHSA-hrxh-6v49-42gf (gRPC-Go: xDS RBAC and HTTP/2 Vulnerabilities)
in modules: dex, couchbase, etcd, milvus, pinecone.

Co-Authored-By: Claude <noreply@anthropic.com>
Bumps go.opentelemetry.io/otel/exporters/otlp/otlplog/otlploghttp from
v0.4.0 to v0.20.0 to fix unbounded HTTP response body reads. Also bumps
related log packages (otel/log, otel/sdk/log, otelslog bridge) for
compatibility.

Co-Authored-By: Claude <noreply@anthropic.com>
@mdelapenya
mdelapenya requested a review from a team as a code owner August 4, 2026 12:08
@mdelapenya mdelapenya added the dependencies Dependencies or external services label Aug 4, 2026
@netlify

netlify Bot commented Aug 4, 2026

Copy link
Copy Markdown

Deploy Preview for testcontainers-go ready!

Name Link
🔨 Latest commit 92c7f9e
🔍 Latest deploy log https://app.netlify.com/projects/testcontainers-go/deploys/6a71d6326e3a29000899bea1
😎 Deploy Preview https://deploy-preview-3835--testcontainers-go.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@github-actions github-actions Bot added the security Vulnerabilities in dependencies or in the library itself label Aug 4, 2026
@coderabbitai

coderabbitai Bot commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: f914247a-59c0-4ea5-86aa-7f82973e3d55

📥 Commits

Reviewing files that changed from the base of the PR and between 6b07dec and 92c7f9e.

⛔ Files ignored due to path filters (6)
  • modules/couchbase/go.sum is excluded by !**/*.sum
  • modules/dex/go.sum is excluded by !**/*.sum
  • modules/etcd/go.sum is excluded by !**/*.sum
  • modules/grafana-lgtm/go.sum is excluded by !**/*.sum
  • modules/milvus/go.sum is excluded by !**/*.sum
  • modules/pinecone/go.sum is excluded by !**/*.sum
📒 Files selected for processing (6)
  • modules/couchbase/go.mod
  • modules/dex/go.mod
  • modules/etcd/go.mod
  • modules/grafana-lgtm/go.mod
  • modules/milvus/go.mod
  • modules/pinecone/go.mod

Summary by CodeRabbit

  • Chores
    • Updated underlying platform components across Couchbase, DEX, etcd, Grafana LGTM, Milvus, and Pinecone integrations.
    • Incorporates the latest improvements and compatibility updates, helping maintain reliability and stability across supported integrations.
    • No changes to public interfaces or end-user functionality.

Walkthrough

The pull request updates dependency versions in six Go modules. It refreshes Google API, genproto, gRPC, protobuf, OpenTelemetry logging, and grpc-gateway references.

Changes

Dependency refresh

Layer / File(s) Summary
Google and gRPC version updates
modules/couchbase/go.mod, modules/dex/go.mod, modules/etcd/go.mod, modules/milvus/go.mod, modules/pinecone/go.mod
These modules now use newer Google API, genproto, gRPC, and protobuf versions.
Grafana observability dependency updates
modules/grafana-lgtm/go.mod
The module now uses newer OpenTelemetry logging packages, grpc-gateway, and genproto versions.

Estimated code review effort: 2 (Simple) | ~10 minutes

Suggested reviewers: stevenh

Poem

A rabbit checks each version line,
New modules fit in rows so fine.
gRPC hops, protobufs gleam,
Logs grow fresh beside the stream.
Six go.mod files softly agree.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title directly reflects the main change: fixing Dependabot security alerts for gRPC and OpenTelemetry dependencies across multiple modules.
Description check ✅ Passed The description clearly explains what the PR does, lists specific security alerts addressed with CVE identifiers, and documents alerts not addressed with reasons.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch security/fix-dependabot-alerts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@mdelapenya mdelapenya added security Vulnerabilities in dependencies or in the library itself and removed security Vulnerabilities in dependencies or in the library itself labels Aug 4, 2026
@mdelapenya
mdelapenya merged commit 632ee7b into main Aug 4, 2026
32 checks passed
@mdelapenya
mdelapenya deleted the security/fix-dependabot-alerts branch August 4, 2026 12:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Dependencies or external services security Vulnerabilities in dependencies or in the library itself

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant