Skip to content

fix(deps): bump basic-ftp to 5.3.1 to resolve GHSA-rpmf-866q-6p89#8705

Merged
mukeshsp merged 1 commit intomasterfrom
SI-512-bump-basic-ftp-5.3.1
May 7, 2026
Merged

fix(deps): bump basic-ftp to 5.3.1 to resolve GHSA-rpmf-866q-6p89#8705
mukeshsp merged 1 commit intomasterfrom
SI-512-bump-basic-ftp-5.3.1

Conversation

@Doddanna17
Copy link
Copy Markdown
Contributor

ticket: SI-512

Summary

  • Bumps basic-ftp from 5.2.2 to 5.3.1 in root resolutions to patch GHSA-rpmf-866q-6p89
  • Updates yarn.lock accordingly
  • Only package.json and yarn.lock changed — no source code touched

Why this works

get-uri@6.0.5 (the direct consumer of basic-ftp) requires basic-ftp: ^5.0.2, so 5.3.1 is a fully compatible upgrade. The previous resolutions pin of 5.2.2 was keeping the patched version out.

What this fixes

GHSA-rpmf-866q-6p89 (HIGH, CVSS 7.5): DoS via unbounded multiline FTP control response buffering in basic-ftp. Patched in 5.3.1. This advisory was appearing 5 times in the yarn audit output (same advisory across 5 different dependency paths), blocking the chore(root): publish modules CI job.

Dependency chain

@bitgo/sdk-api > proxy-agent > pac-proxy-agent > get-uri > basic-ftp (5.2.2 → 5.3.1)

Impact

Unblocks the publish of @bitgo/sdk-coin-hbar containing the explainTransaction fix for HBAR staking (PR #8700), which is needed to unblock end-to-end HBAR stake signing on staging.

Related

Patches DoS via unbounded multiline FTP control response buffering.
get-uri accepts ^5.0.2 so 5.3.1 is a compatible drop-in upgrade.
Unblocks the publish CI job blocked by 5 HIGH yarn audit findings
(all the same advisory across different dep paths).

Ticket: SI-512
@linear-code
Copy link
Copy Markdown

linear-code Bot commented May 7, 2026

@Doddanna17 Doddanna17 marked this pull request as ready for review May 7, 2026 05:17
@Doddanna17 Doddanna17 requested a review from a team as a code owner May 7, 2026 05:17
@mukeshsp mukeshsp merged commit 1eeb320 into master May 7, 2026
22 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