Skip to content

[WIP] Fix vulnerabilities in node:lts-alpine container image - #50207

Merged
pelikhan merged 3 commits into
mainfrom
copilot/container-image-scan-fix-node-lts-alpine
Aug 4, 2026
Merged

[WIP] Fix vulnerabilities in node:lts-alpine container image#50207
pelikhan merged 3 commits into
mainfrom
copilot/container-image-scan-fix-node-lts-alpine

Conversation

Copilot AI commented Aug 4, 2026

Copy link
Copy Markdown
Contributor
  • Locate the affected node:lts-alpine container build and its dependency installation
  • Update the bundled npm undici dependency to version 6.27.0 or newer alongside existing tar and brace-expansion remediations
  • Validate the patch in an isolated container build (tar 7.5.22, brace-expansion 5.0.9, undici 6.28.0)
  • Run final repository validation and security checks

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
@github-actions

github-actions Bot commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

Triage: chore (security) / medium risk

  • Category: chore
  • Risk: medium
  • Priority score: 60/100 (impact 30, urgency 20, quality 10)
  • Recommended action: defer — monitor until diff is populated (currently 0 files changed, WIP/draft)
  • Notes: Addresses 1 critical + 4 high CVEs in node:lts-alpine. CI status unstable. Revisit once agent produces the actual dependency bump diff.

Generated by 🔧 PR Triage Agent · auto · 28.5 AIC · ⌖ 2.46 AIC · ⊞ 8K ·

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
@github-actions

github-actions Bot commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

Great work! 🎉 This PR looks solid for addressing the container image vulnerabilities.

Summary:
You've successfully fixed the security vulnerabilities in the node:lts-alpine container image by:

  • Patching the bundled npm dependencies (tar, brace-expansion, undici) to meet minimum safe versions
  • Using a temp-directory overlay approach to avoid npm's private package.json resolution issues
  • Updating the workflow lock file accordingly

What's working well:

  • Focused change targeting only the vulnerable dependencies
  • Clear changeset documentation explaining the fix approach
  • Proper labeling and assignment to maintainers
  • Small, reviewable diff (10 lines total)

The PR is ready for review and merge. The WIP status is fine while awaiting final validation from maintainers.

Generated by ✅ Contribution Check · auto · 58.7 AIC · ⌖ 3.75 AIC · ⊞ 8.8K ·

@pelikhan
pelikhan marked this pull request as ready for review August 4, 2026 11:59
Copilot AI balanced review requested due to automatic review settings August 4, 2026 11:59
@pelikhan
pelikhan merged commit 59c743c into main Aug 4, 2026
1 check passed
@pelikhan
pelikhan deleted the copilot/container-image-scan-fix-node-lts-alpine branch August 4, 2026 11:59

Copilot AI 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.

Pull request overview

Updates the safe-outputs Node container to remediate the bundled npm undici vulnerability.

Changes:

  • Installs and overlays undici 6.27.0 or newer.
  • Updates release notes.
  • Normalizes a generated workflow step name.
Show a summary per file
File Description
actions/setup/js/Dockerfile.safe-outputs-mcp Patches npm’s bundled undici.
.changeset/patch-refresh-gh-aw-node.md Documents the remediation.
.github/workflows/smoke-goose.lock.yml Updates generated step capitalization.

Review details

Tip

Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

  • Files reviewed: 3/3 changed files
  • Comments generated: 0
  • Review effort level: Balanced

@github-actions

github-actions Bot commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

🎉 This pull request is included in a new release.

Release: v0.84.4

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[container-image-scan] Container findings for node:lts-alpine

3 participants