Skip to content

fix: bump transitive deps to resolve brace-expansion and ip-address CVEs - #87

Merged
angelocordon merged 2 commits into
mainfrom
fix/secvuln-dep-upgrades
Aug 12, 2026
Merged

fix: bump transitive deps to resolve brace-expansion and ip-address CVEs#87
angelocordon merged 2 commits into
mainfrom
fix/secvuln-dep-upgrades

Conversation

@angelocordon

Copy link
Copy Markdown
Contributor

Summary

Addresses three SECVULN tickets by upgrading @openapitools/openapi-generator-cli from 2.39.1 to 2.40.1, which allows transitive dependencies to resolve to their patched versions.

Vulnerabilities addressed

Package Old version New version Advisory SECVULN
brace-expansion 5.0.6 5.0.9 GHSA-mh99-v99m-4gvg SECVULN-51496
brace-expansion 5.0.6 5.0.9 GHSA-rgw5-rvv9-x895 SECVULN-51925
ip-address 10.2.0 10.5.0 GHSA-mwp4-54f8-5fhr SECVULN-51926

Details

SECVULN-51496 / SECVULN-51925 — brace-expansion DoS
brace-expansion@5.0.6 has two related DoS vulnerabilities. 5.0.8 added an EXPANSION_MAX_LENGTH bound, but intermediate arrays were still unbounded (fixed fully in 5.0.9). An attacker passing a crafted string to expand() can trigger an uncatchable OOM crash or a multi-minute CPU stall.

SECVULN-51926 — ip-address SSRF / trust-boundary bypass
ip-address@10.2.0 decodes leading-zero octets as decimal while OS resolvers decode them as octal. isPrivate()/isLoopback() etc. mis-classify addresses like 012.0.0.1 (classified public, but resolves to 10.0.0.1), enabling SSRF bypass. Fixed in 10.3.1; we now resolve to 10.5.0.

Dependency chain

Both packages are transitive dev-only dependencies:

@openapitools/openapi-generator-cli
  ├── glob -> minimatch -> brace-expansion
  └── proxy-agent -> socks-proxy-agent -> socks -> ip-address

Additional fix

Migrated the existing pnpm.overrides config from the pnpm field in package.json (silently ignored since pnpm v10) to pnpm-workspace.yaml, which is the correct location.

@angelocordon
angelocordon requested a review from a team as a code owner August 12, 2026 05:44
Upgrade @openapitools/openapi-generator-cli from 2.39.1 to 2.40.1,
which allows transitive dependencies to resolve to their fixed
versions:

- brace-expansion: 5.0.6 -> 5.0.9 (GHSA-mh99-v99m-4gvg,
  GHSA-rgw5-rvv9-x895 — DoS via unbounded expansion length and
  incomplete intermediate array bounds; SECVULN-51496,
  SECVULN-51925)
- ip-address: 10.2.0 -> 10.5.0 (GHSA-mwp4-54f8-5fhr — SSRF/
  trust-boundary bypass via leading-zero octet mis-decoding;
  SECVULN-51926)

Also migrate the existing pnpm.overrides config from the ignored
pnpm field in package.json to pnpm-workspace.yaml, which is the
correct location for pnpm v10.

Co-authored-by: OpenCode (sonnet-4.6-us-west) <opencode.noreply@hashicorp.com>
@angelocordon
angelocordon force-pushed the fix/secvuln-dep-upgrades branch from 5d51da2 to ba1a456 Compare August 12, 2026 05:49
@angelocordon
angelocordon enabled auto-merge (squash) August 12, 2026 05:54
@angelocordon
angelocordon merged commit 2c38bd9 into main Aug 12, 2026
3 checks passed
@angelocordon
angelocordon deleted the fix/secvuln-dep-upgrades branch August 12, 2026 14:33
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