Skip to content

fix: address js-yaml and extract-zip security advisories - #9690

Merged
prajwalu142 merged 1 commit into
masterfrom
CSHLD1698/fixBlockingVulnerabilities
Sep 9, 2026
Merged

fix: address js-yaml and extract-zip security advisories#9690
prajwalu142 merged 1 commit into
masterfrom
CSHLD1698/fixBlockingVulnerabilities

Conversation

@prajwalu142

Copy link
Copy Markdown
Contributor

Problem

GHSA-2883-xcg3-v3hh affects js-yaml <4.3.2. GHSA-7pqw-9j4j-h8q3 affects extract-zip through 2.0.1 with no patched release available.

Goal

Clear the blocking OSV findings for both advisories.

Fix

  • Pin js-yaml to 4.3.2 via resolutions/overrides and refresh yarn.lock
  • Ignore GHSA-7pqw-9j4j-h8q3: extract-zip is transitive via cypress and @puppeteer/browsers (browser-test only); no upstream fix beyond 2.0.1

Testing

Verified yarn.lock resolves all js-yaml ranges to 4.3.2; confirmed extract-zip parents are browser-test tooling only.

Ticket: CSHLD-1698

## Problem

GHSA-2883-xcg3-v3hh affects js-yaml <4.3.2. GHSA-7pqw-9j4j-h8q3
affects extract-zip through 2.0.1 with no patched release available.

## Goal

Clear the blocking OSV findings for both advisories.

## Fix

- Pin js-yaml to 4.3.2 via resolutions/overrides and refresh yarn.lock
- Ignore GHSA-7pqw-9j4j-h8q3: extract-zip is transitive via cypress and
  @puppeteer/browsers (browser-test only); no upstream fix beyond 2.0.1

## Testing

Verified yarn.lock resolves all js-yaml ranges to 4.3.2; confirmed
extract-zip parents are browser-test tooling only.

Ticket: CSHLD-1698

@diksha190 diksha190 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Approved
This security fix directly addresses two disclosed vulnerabilities:

  1. js-yaml (GHSA-2883-xcg3-v3hh): Pinning from 4.3.1 → 4.3.2 resolves the arbitrary code execution risk.
  2. extract-zip (GHSA-7pqw-9j4j-h8q3): Correctly identified as dev-only (Cypress, Puppeteer). Documented exclusion with clear dates and conditions.

@prajwalu142
prajwalu142 merged commit fa0d208 into master Sep 9, 2026
26 checks passed
@linear-code

linear-code Bot commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

CSHLD-1701

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.

3 participants