Repository navigation
fix: 올리브영 중계의 안전 복구와 의존성 보안 보완 - #311
Merged
Merged
Conversation
5 of 7 tasks
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
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.
올리브영 중계에서 브라우저와 감시자가 이미 종료됐는데도 cleanup 실패와 잔여 마커 때문에 이후 조회가 계속 실패하는 문제를 복구합니다. 준비 완료 때 확보한 소유권 정보와 현재 프로세스·파일을 대조해 완전 소멸을 증명할 수 있을 때만 마커를 보존 이동하고 다음 요청의 세션 재생성을 허용합니다.
감시자의 숫자형 비정상 종료만 대상입니다. 살아 있거나 재사용된 PID, 잔여 그룹/crashpad, 시작 전 실패, 시그널 종료, 기한 초과, 다른 UID·inode·심볼릭 링크·FIFO 등은 계속 차단합니다. 종료 검증은 기존 기한 안에서 한 번만 공유합니다. 실패한 원본 요청이나 인증·누적 원장·API 계약은 바꾸지 않습니다.
별도 커밋으로 새 의존성 취약점 6개(High 2개 포함)를 수정합니다. body-parser/fast-uri/zod와 기존 undici override를 패치하고 필요한 qs 버전을 맞췄습니다. Wrangler/Miniflare 버전은 유지합니다.
검증: 회귀 실패 재현 후 수정, 실제 파일·FIFO와 supervisor/lifecycle 연동 포함 집중 검사 통과, 전체 3,906개 검사·기존 커버리지 4지표 100%·build/discovery·npm audit 0개. 독립 리뷰에서 발견한 FIFO blocking과 불충분한 symlink 테스트를 보완한 후 재승인받았습니다.