[Fix] 보안 취약점 문제 해결 - #60
Merged
Merged
Conversation
ghdtjgus76
requested review from
SeieunYoo,
eugene028,
hamo-o and
kongnayeon
as code owners
June 14, 2024 08:58
This reverts commit a1228ab.
SeieunYoo
approved these changes
Jun 16, 2024
SeieunYoo
left a comment
SeieunYoo
left a comment
Collaborator
There was a problem hiding this comment.
마침 스토리북 최신 버전으로 올렸어야 했는데 짱짱🙌🙌
hamo-o
approved these changes
Jun 18, 2024
hamo-o
left a comment
hamo-o
left a comment
Member
There was a problem hiding this comment.
수고하셨어요!!
overrides로 종속성의 종속성을 조정하는 방법만 알고 있었는데,
yarn2+와 pnpm에서는 같은 역할의 resolutions 필드를 지원하는군요..! 배워갑니당
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
🎉 변경 사항
보안 취약점 문제 해결했습니다.
🚩 관련 이슈
🙏 여기는 꼭 봐주세요!
GHSA-78xj-cgh5-2h22
위와 같이 보안 문제가 발생해서 알아보니, 설치한 패키지 중 내부적으로 ip 패키지를 사용하는 패키지가 있었습니다.
해당 패키지에서 보안 관련 이슈가 있어 저희 레포에서도 관련해서 Security 이슈로 뜬 것 같구요
현재 ip 패키지 최신 버전에서도 이 문제가 해결되지 않은 걸로 보입니다.
내부적으로 ip를 사용하는 패키지는 storybook, turbo/gen 이렇게 두 가지가 있었습니다.
그래서 좀 찾아보니 아래 PR을 찾을 수 있었습니다.
storybookjs/storybook#27529
스토리북 8.1.6 버전부터는 해당 문제가 해결된 거 같아서 일단 스토리북은 최신 버전으로 업데이트했습니다.
그리고 turbo/gen은 저희가 사용하는 패키지가 아니라서 그냥 삭제했습니다.
braces도 3.0.3 이상 버전으로 업데이트해야 한다고 해서 같이 해뒀습니다.