Skip to content

fix(deps): update basic-ftp to 5.2.1 (GHSA-chqc-8p9q-pq6q)#14814

Merged
davidkonigsberg merged 3 commits into
mainfrom
dependabot-alert-1412-devin
Apr 9, 2026
Merged

fix(deps): update basic-ftp to 5.2.1 (GHSA-chqc-8p9q-pq6q)#14814
davidkonigsberg merged 3 commits into
mainfrom
dependabot-alert-1412-devin

Conversation

@github-actions
Copy link
Copy Markdown
Contributor

@github-actions github-actions Bot commented Apr 9, 2026

Description

Refs Dependabot Alert #1412

Resolves HIGH severity CRLF injection vulnerability in basic-ftp@5.2.0 (GHSA-chqc-8p9q-pq6q, CVSS 8.6) by updating to 5.2.1.

basic-ftp is a transitive dependency (puppeteer@puppeteer/browsersproxy-agentpac-proxy-agentget-uribasic-ftp). Since get-uri@6.0.5 specifies basic-ftp: ^5.0.2, regenerating the lockfile was sufficient to pick up 5.2.1 — no pnpm override was needed.

Changes Made

  • Regenerated pnpm-lock.yaml to resolve basic-ftp@5.2.1 (previously 5.2.0)
  • Deleted scaffold file .github/dependabot-alerts/alert-1412.md

Testing

  • pnpm install succeeds and lockfile resolves basic-ftp@5.2.1
  • Pre-commit hooks pass
  • CI checks pass

Link to Devin session: https://app.devin.ai/sessions/61725b6a88614f7ba6f7346d2ba06e59

@devin-ai-integration devin-ai-integration Bot changed the title [Dependabot Alert #1412] HIGH: basic-ftp vulnerability fix(deps): update basic-ftp to 5.2.1 (GHSA-chqc-8p9q-pq6q) Apr 9, 2026
…e basic-ftp 5.2.1 directly

Co-Authored-By: unknown <>
@davidkonigsberg davidkonigsberg marked this pull request as ready for review April 9, 2026 12:11
Copy link
Copy Markdown

@claude claude Bot left a comment

Choose a reason for hiding this comment

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

Claude Code Review

This repository is configured for manual code reviews. Comment @claude review to trigger a review and subscribe this PR to future pushes, or @claude review once for a one-time review.

Tip: disable this comment in your organization's Code Review settings.

@davidkonigsberg davidkonigsberg enabled auto-merge (squash) April 9, 2026 12:17
@davidkonigsberg davidkonigsberg merged commit 583a4fd into main Apr 9, 2026
96 checks passed
@davidkonigsberg davidkonigsberg deleted the dependabot-alert-1412-devin branch April 9, 2026 12:24
@devin-ai-integration
Copy link
Copy Markdown
Contributor

@davidkonigsberg — updated: removed the override per your suggestion. Simply regenerating the lockfile was sufficient since get-uri@6.0.5 specifies basic-ftp: ^5.0.2, which allows 5.2.1. The PR now only changes the lockfile (+ scaffold deletion). All CI checks pass. Ready for review.

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

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant