Skip to content

chore(deps): yaml の transitive 脆弱性を overrides で解消#112

Merged
ponponusa merged 1 commit into
developfrom
chore/frontend-security-overrides
May 16, 2026
Merged

chore(deps): yaml の transitive 脆弱性を overrides で解消#112
ponponusa merged 1 commit into
developfrom
chore/frontend-security-overrides

Conversation

@ponponusa

@ponponusa ponponusa commented May 16, 2026

Copy link
Copy Markdown
Owner

Summary

  • frontend/package.jsonoverrides を追加し、yaml-language-server 経由で混入していた yaml@2.7.1^2.8.3 (ロック上は 2.9.0) に固定。
  • 対象 advisory: GHSA-48c2-rrv3-qjmp "yaml is vulnerable to Stack Overflow via deeply nested YAML collections" (moderate)
  • ローカル npm audit の検出数: 7 → 2 件

残った 2 件(本 PR ではスコープ外)

いずれも Astro v6 系への major bump が前提のため別 PR で検討:

  • GHSA-j687-52p2-xcff Astro: XSS in define:vars via incomplete </script> tag sanitization (moderate)
  • GHSA-xr5h-phrj-8vxv Astro: Server island encrypted parameters vulnerable to cross-component replay (low)

Dependabot の見え方について

GitHub UI 上は現時点で 33 alerts と表示されますが、ほとんどは chore/frontend-deps-update (PR #111) のマージ前に発行された stale alert で、最新の frontend/package-lock.json 上では既に解消済みです。Dependabot の次回再スキャン後に自動クローズされる想定。

Test plan

  • npm run check(astro check / TypeScript 型検査)— エラーなし、警告のみ
  • npm run build(プロダクションビルド)— 成功 (5 page(s) built)
  • npm audit の検出数 7 → 2 件を確認
  • レビュアー側で必要に応じて npm ci && npm run check && npm run build でローカル再現

yaml-language-server (@astrojs/check 経由) が yaml 2.7.1 に依存して
おり GHSA-48c2-rrv3-qjmp (Stack Overflow via deeply nested YAML
collections, moderate) に該当していたため、package.json の overrides
で yaml を ^2.8.3 に固定。

これにより npm audit の脆弱性は 7 件 -> 2 件に削減。残る 2 件は
Astro v6 系への major bump (GHSA-j687-52p2-xcff / GHSA-xr5h-phrj-8vxv)
が前提となるため、本 PR では対象外とし別途検討する。

astro check / build はいずれもエラーなし。
@ponponusa ponponusa merged commit 7f6c367 into develop May 16, 2026
5 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.

1 participant