Skip to content

Commit 3142e16

Browse files
vuanhphungIsaac
andcommitted
chore(security): override basic-ftp to ^6.2.1 instead of suppressing GHSA-c475-qrg2-pj4r
get-uri caps basic-ftp at ^5.3.1, but an override clears the advisory the same way the existing ip-address override does. The Client API get-uri uses is unchanged in 6.x. Co-authored-by: Isaac <no-reply@databricks.com> Signed-off-by: Vu Anh Phung <vu.phung@databricks.com>
1 parent 8c96c8d commit 3142e16

4 files changed

Lines changed: 12 additions & 16 deletions

File tree

‎SECURITY-OVERRIDES.md‎

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -13,12 +13,13 @@ This file documents the provenance and exit condition for each override. **When
1313

1414
## Entries
1515

16-
### `basic-ftp: ^5.3.1`
16+
### `basic-ftp: ^6.2.1`
1717

1818
- **Class**: runtime
1919
- **Path**: `proxy-agent → pac-proxy-agent → get-uri → basic-ftp`
20-
- **CVEs cleared**: GHSA-5rq4-664w-9x2c, GHSA-6v7q-wjvx-w8wg, GHSA-rp42-5vxx-qpwr, GHSA-rpmf-866q-6p89
21-
- **Exit**: `get-uri` bumps its `basic-ftp` dep range to include `^5.3.1`.
20+
- **CVEs cleared**: GHSA-5rq4-664w-9x2c, GHSA-6v7q-wjvx-w8wg, GHSA-rp42-5vxx-qpwr, GHSA-rpmf-866q-6p89, GHSA-c475-qrg2-pj4r
21+
- **Why an override is needed**: `get-uri` (through 8.0.1) caps `basic-ftp` at `^5.3.1`, below the 6.2.1 fix. The `Client` API get-uri uses is unchanged in 6.x; the 6.0 major only defaults `allowSeparateTransferHost` to `false`, so a PASV reply naming a different host is rejected.
22+
- **Exit**: `get-uri` widens its `basic-ftp` range to include `^6.2.1`.
2223

2324
### `@75lb/deep-merge: ^1.1.2`
2425

‎osv-scanner.toml‎

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -35,8 +35,3 @@
3535
id = "GHSA-vfj7-8cjw-p6xm"
3636
ignoreUntil = 2027-04-05
3737
reason = "braces: no fixed release (3.0.3 is latest). Dev-only (mocha -> chokidar, globby -> fast-glob -> micromatch); not in shipped dist/."
38-
39-
[[IgnoredVulns]]
40-
id = "GHSA-c475-qrg2-pj4r"
41-
ignoreUntil = 2027-04-05
42-
reason = "basic-ftp: fix is 6.2.1 but get-uri (latest 8.0.1) pins ^5.3.1. Vulnerable Client.list() is only reached via pac+ftp:// proxy URLs; ProxyOptions.protocol allows only http/https/socks*."

‎package-lock.json‎

Lines changed: 7 additions & 7 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

‎package.json‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -101,7 +101,7 @@
101101
"@databricks/databricks-sql-kernel-win32-arm64-msvc": "1.1.0"
102102
},
103103
"overrides": {
104-
"basic-ftp": "^5.3.1",
104+
"basic-ftp": "^6.2.1",
105105
"@75lb/deep-merge": "^1.1.2",
106106
"ws": "^8.18.0",
107107
"ip-address": "^10.1.1",

0 commit comments

Comments
 (0)