Skip to content

chore: resolve open dependabot security alerts#1980

Open
jonathannorris wants to merge 1 commit into
mainfrom
chore/dependabot-alerts
Open

chore: resolve open dependabot security alerts#1980
jonathannorris wants to merge 1 commit into
mainfrom
chore/dependabot-alerts

Conversation

@jonathannorris

@jonathannorris jonathannorris commented Jun 15, 2026

Copy link
Copy Markdown
Member

Summary

Resolves the cleanly patchable open Dependabot security alerts.

  • Bumped github.com/containerd/containerd/v2 to v2.2.4 in test/integration (resolves the containerd alert).
  • Added a js-cookie override (>=3.0.7) in playground-app to patch the transitive react-use dependency.

Dependabot Alerts Resolved

Alert Package Severity Fix
#189 github.com/containerd/containerd/v2 high Bumped to v2.2.4 via go get
#190 js-cookie high Forced >=3.0.7 via npm override

Unresolvable / not fixed here

esbuild (#191, high) in playground-app. The fix requires esbuild >=0.28.1, but vite 6/7 cannot build against esbuild 0.28 (transform-target error), and the only vite line that drops esbuild is vite 8, which pulls in lightningcss (MPL-2.0) and trips the FOSSA License Compliance check. The playground app is a dev-only tool that is not built in CI, and the advisory is a dev-server issue. Leaving esbuild at 0.25.x pending an upstream vite path that does not introduce a copyleft dependency.

github.com/docker/docker (#152, #151, #184, #185, #186) in test/integration. The legacy docker/docker module path is end-of-life at v28.5.2+incompatible; the patch lives in github.com/moby/moby/v2. It remains a transitive dependency of testcontainers/buildx/compose and cannot be patched on our side until those upstreams fully migrate off docker/docker. (Bumping testcontainers to v0.42 to drop it breaks the pinned go-sdk-contrib/tests/flagd/v2 test framework, which uses the older nat.Port API.)

Alert Package Severity
#191 esbuild high
#186 github.com/docker/docker high
#185 github.com/docker/docker medium
#184 github.com/docker/docker high
#152 github.com/docker/docker high
#151 github.com/docker/docker medium

Verification

  • test/integration: go build ./... and go vet ./... pass.
  • playground-app: npm run build succeeds; js-cookie resolves to 3.0.8.

@jonathannorris jonathannorris requested review from a team as code owners June 15, 2026 14:21
@dosubot dosubot Bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Jun 15, 2026
@jonathannorris jonathannorris marked this pull request as draft June 15, 2026 14:21
@netlify

netlify Bot commented Jun 15, 2026

Copy link
Copy Markdown

Deploy Preview for polite-licorice-3db33c canceled.

Name Link
🔨 Latest commit cbc8680
🔍 Latest deploy log https://app.netlify.com/projects/polite-licorice-3db33c/deploys/6a300ca077886c0007773529

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code Review

This pull request updates various dependencies, including upgrading Go modules in the integration tests and updating frontend packages in the playground application. However, critical issues were identified in the frontend dependency updates: the specified versions for @vitejs/plugin-react (^6.0.2) and vite (^8.0.16) do not exist on the public npm registry, which will cause build and installation failures.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

Comment thread playground-app/package.json Outdated
Comment thread playground-app/package.json Outdated
Signed-off-by: Jonathan Norris <jonathan.norris@dynatrace.com>
@jonathannorris jonathannorris force-pushed the chore/dependabot-alerts branch from fe15340 to cbc8680 Compare June 15, 2026 14:30
@sonarqubecloud

Copy link
Copy Markdown

@jonathannorris jonathannorris marked this pull request as ready for review June 15, 2026 20:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants