Skip to content

fix(frontend): bump js-yaml to >=4.2.0 (CVE-2026-53550, Dependabot #140)#694

Closed
remyluslosius wants to merge 1 commit into
mainfrom
fix/js-yaml-dos
Closed

fix(frontend): bump js-yaml to >=4.2.0 (CVE-2026-53550, Dependabot #140)#694
remyluslosius wants to merge 1 commit into
mainfrom
fix/js-yaml-dos

Conversation

@remyluslosius

Copy link
Copy Markdown
Contributor

Resolves Dependabot alert #140.

The vulnerability

  • js-yaml@4.1.1GHSA-h67p-54hq-rp68 / CVE-2026-53550, quadratic-complexity DoS parsing crafted YAML merge keys. Medium, CVSS availability-only (C:N/I:N/A:L).
  • Transitive dev dependency (frontend/package-lock.json, dev scope): pulled in by eslint -> @eslint/eslintrc and openapi-typescript -> @redocly/openapi-core. Not in the shipped SPA bundle.
  • Fixed in 4.2.0.

Fix

Pin via overrides in frontend/package.json to ^4.2.0 (forces the resolution regardless of the parents' ranges). npm dedups both consumers to a single js-yaml@4.2.0.

Verification

  • npm ls js-yaml -> 4.2.0 (both consumers)
  • npm audit -> 0 vulnerabilities
  • eslint --max-warnings=0 . clean (it's the primary js-yaml consumer)
  • tsc --noEmit clean

Diff is scoped to package.json + package-lock.json.

js-yaml@4.1.1 (transitive dev dependency via eslint -> @eslint/eslintrc and
openapi-typescript -> @redocly/openapi-core) is vulnerable to a
quadratic-complexity DoS in YAML merge-key handling
(GHSA-h67p-54hq-rp68, medium, availability-only, dev scope).

Pin via package.json 'overrides' to ^4.2.0 (the patched release); npm
dedups both consumers to 4.2.0. npm audit: 0 vulnerabilities; eslint + tsc
green. Not in the shipped SPA bundle.

Signed-off-by: Remylus Losius <remyluslosius@gonaibo.com>
@remyluslosius

Copy link
Copy Markdown
Contributor Author

Superseded by #698 (consolidated GA-prep merge to avoid the rebase cascade). The same changes landed there.

@remyluslosius remyluslosius deleted the fix/js-yaml-dos branch June 27, 2026 02:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant