Skip to content

npm audit high 취약점 overrides로 조치 (#22) - #23

Merged
hayarobi merged 1 commit into
mainfrom
fix/npm-audit-overrides
Jul 29, 2026
Merged

npm audit high 취약점 overrides로 조치 (#22)#23
hayarobi merged 1 commit into
mainfrom
fix/npm-audit-overrides

Conversation

@dohoon84

@dohoon84 dohoon84 commented Jul 29, 2026

Copy link
Copy Markdown

Summary

  • Closes CI npm audit high 취약점으로 빌드 검사 실패 #22
  • #21 PR의 Test build CI에서 취약점 감사(npm audit --audit-level=high)에 걸려 실패하여, 해당 이슈를 분리 조치한 PR입니다.
  • Docusaurus 전이 의존성 취약점을 package.json overrides로 패치 버전에 고정했습니다.
  • CI 감사가 통과하도록 lockfile을 갱신하고, README Security 절을 현재 조치에 맞게 수정했습니다.

배경 (PR #21 CI 실패)

PR #21 작업 중 CI Security audit (high/critical) 단계에서 아래와 같이 high 취약점이 보고되며 검사가 실패했습니다.

# npm audit report (발췌)

brace-expansion  <=5.0.7
Severity: high
DoS via exponential-time / unbounded expansion
https://github.com/advisories/GHSA-3jxr-9vmj-r5cp
https://github.com/advisories/GHSA-mh99-v99m-4gvg

fast-uri  3.0.0 - 3.1.3
Severity: high
host confusion via literal backslash / failed IDN canonicalization
https://github.com/advisories/GHSA-v2hh-gcrm-f6hx
https://github.com/advisories/GHSA-4c8g-83qw-93j6

js-yaml  <=3.14.2 || 4.0.0 - 4.2.0
Severity: high
Quadratic-complexity DoS in merge key handling
https://github.com/advisories/GHSA-h67p-54hq-rp68
https://github.com/advisories/GHSA-52cp-r559-cp3m

postcss  <=8.5.17
Severity: high
Path Traversal in Previous Source Map Auto-Loading
https://github.com/advisories/GHSA-r28c-9q8g-f849

shell-quote  <=1.8.4
Severity: high
Quadratic-complexity DoS in parse()
https://github.com/advisories/GHSA-395f-4hp3-45gv

svgo  3.0.0 - 3.3.3
Severity: high
removeScripts plugin leaves some executable scripts intact
https://github.com/advisories/GHSA-2p49-hgcm-8545

9 vulnerabilities (2 low, 1 moderate, 6 high)
Error: Process completed with exit code 1.

문서 변경(#21)과 보안 의존성 조치를 분리하기 위해 본 PR(#23)로 올렸습니다.

변경 내용

  • body-parser, brace-expansion, minimatch, dompurify, fast-uri, js-yaml, postcss, shell-quote, svgo, webpack-dev-server 등 overrides 추가
  • gray-matterjs-yaml은 3.15.0으로 분리 고정
  • 로컬 검증: npm audit 0건, npm run build 성공

Test plan

  • npm installnpm audit --audit-level=high 통과
  • npm run build 성공
  • CI Test build 워크플로 통과 확인

Override vulnerable Docusaurus toolchain packages so CI audit --audit-level=high passes.

@hayarobi hayarobi left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@hayarobi
hayarobi merged commit 34994ad into main Jul 29, 2026
1 check 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.

CI npm audit high 취약점으로 빌드 검사 실패

2 participants