앱 부팅이 쿠키 동기화에 묶여 빈 웹뷰로 고착되는 문제 방어 - #530
Open
m-a-king wants to merge 14 commits into
Open
Conversation
- 바로가기 제거
* feat: 위시 아이템 INCOMPLETE 상태 대응 - 서버가 추출 결과를 일부만 채웠을 때 INCOMPLETE 를 내려준다 (TeamPiKi/core#945). 기존 코드는 FAILED·PENDING·PROCESSING 이 아니면 전부 정상 카드로 그려, 이름·가격이 빈 칸인 카드가 보이고 채우라는 유도가 없었다 - 위시 그리드에서 INCOMPLETE 를 FAILED 와 같은 편집 유도 카드로 보내되 문구만 "일부만 가져왔어요" 로 가른다 - name·price 를 nullable 로 바꾸면서 카드 컴포넌트도 nullable 을 받게 했다. INCOMPLETE 를 앞에서 걸러 실제로 빈 값이 정상 카드에 들어가지는 않는다 - 토너먼트: 담기 후보에서 INCOMPLETE 를 제외하고(서버가 출전을 막는다), 바스켓에서는 클릭 가능하게 둔다(값을 채워야 하므로) * feat: 토너먼트 아이템 상세에 incomplete 상태 처리 추가 * refactor: 리터럴/상수로 섞어 쓰던 곳을 ITEM_STATUS 로 통일 * feat: 토너먼트 시작 차단에 INCOMPLETE 포함 * feat: INCOMPLETE 를 수정 화면으로 링크 * chore: 불필요한 파싱 상태 주석 정리 --------- Co-authored-by: kanghaeun <xgkg0330@jnu.ac.kr>
* feat: INCOMPLETE 파싱 알림 타입·SSE 상태 대응 - 서버가 파싱이 일부만 끝난 경우 ITEM_PARSING_INCOMPLETE 알림과 status=INCOMPLETE SSE 를 보낸다 (TeamPiKi/core#945). 모르는 타입이라 switch default 로 빠져 딥링크가 동작하지 않았다 - 알림함·푸시·SSE 세 라우팅에 케이스를 더한다. 목적지는 기존 파싱 알림과 같다(위시 또는 토너먼트 담기 화면) - SSE 토스트는 실패와 갱신 대상이 같아 케이스를 합치되 문구만 info 로 가른다 — 실패가 아니라 "채워 주세요" 안내라서다 * chore: 불필요한 파싱 상태 주석 정리 --------- Co-authored-by: kanghaeun <xgkg0330@jnu.ac.kr> Co-authored-by: kanghaeun <145974230+kanghaeun@users.noreply.github.com>
* feat: 소셜 로그인 사용자 취소 메시지 타입 추가 (APP_RES_SOCIAL_LOGIN_CANCEL) * fix: 소셜 로그인 취소 시 에러 토스트 노출되는 문제 수정 (앱) * fix: 소셜 로그인 취소 시 에러 토스트 노출되는 문제 수정 (웹) * fix: 구글 로그인 취소를 반환값으로 처리 (isCancelledResponse) --------- Co-authored-by: soyeong <mb535622@sookmyung.ac.kr>
* refactor: 아이템 status 비교를 ITEM_STATUS 상수로 통일 리터럴로 남아 있던 아이템 status 비교를 ITEM_STATUS 상수로 교체. notification.ts 의 status 유니온도 다른 타입 파일과 동일하게 ITEM_STATUS 파생 형태로 맞춤. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> * refactor: 토너먼트 status 비교를 TOURNAMENT_STATUS 상수로 통일 리터럴로 남아 있던 토너먼트 status 비교·할당을 TOURNAMENT_STATUS 상수로 교체. 탭별 상태 매핑 상수도 함께 정리. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 5 <noreply@anthropic.com>
* feat: 정보 수집 실패 위시에 삭제 버튼 노출 * style: 위시 정보 수정 화면 경고 배너-이미지 간격 정합 (8px → 16px)
* chore: 인스타 랜딩 유입 소스를 UA 로 판정해 utm 파라미터 제거 인스타 바이오에 `https://piki.day/open` 만 걸 수 있도록, `?utm_source=instagram` 없이도 인앱 브라우저 UA(`isInstagramBrowser`)로 유입 소스를 판정한다. - 바이오 탭은 인스타 인앱 브라우저로 열리므로 랜딩 도달 시점에 UA 토큰이 남아 있다 - 쿼리로 들어온 `utm_source` 가 있으면 그쪽이 우선 — 다른 채널 확장 시 그대로 사용 - GA4 `landing_view` 의 `source`, Play 스토어 `referrer` 는 동일하게 유지된다 * chore: 미사용 open.piki.day 랜딩 서브도메인 제거 인스타 링크를 `piki.day/open` 으로 직접 걸기로 해서 서브도메인 경로를 걷어낸다. - `proxy.ts` 의 `open.*` 루트 rewrite 제거 - `isLandingHost` 삭제 — 앱의 associatedDomains 에 등록된 적 없는 호스트였다 - `landingHost.ts` → `serviceHost.ts` 로 이름 변경. 남은 `toServiceHost` 는 Vercel 프리뷰 등 미등록 호스트를 프로덕션으로 정규화하는 역할만 한다 * fix: Host 헤더를 엄격히 파싱해 userinfo 오리진 조작 차단
* fix: 탈퇴 후 죽은 토큰이 남아 로그인 페이지가 홈으로 되돌리던 문제 수정 - 탈퇴 성공 시 웹 브라우저에서도 서버 액션으로 httpOnly 인증 쿠키 폐기 - 쿠키 폐기 서버 액션을 clearAuthCookies 로 추출해 로그아웃과 공유 - proxy 의 쿠키 폐기 발동 조건에 withdrawn-account action 추가 closes #467 Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> * fix: 탈퇴 후 로그인 페이지로 직행하도록 수정 루트는 스플래시·온보딩 판정 경로라 탈퇴 직후 앱 첫 실행과 같은 화면을 거친다. 탈퇴는 세션 종료이므로 로그아웃과 동일하게 로그인으로 보낸다. * fix: 앱에서 온보딩이 노출되지 않던 문제 수정 앱은 웹뷰를 /login 으로 바로 열어 스플래시를 거치지 않는데, 온보딩 진입 판정이 SplashClient 안에 있어 앱 사용자는 온보딩을 볼 수 없었다. - 판정을 로그인 진입 시점(OnboardingGate)으로 옮겨 웹·앱 두 경로 모두 게이트를 통과 - 스플래시는 브랜드 연출만 담당하도록 목적지를 로그인으로 단일화 - 쿼리가 붙은 진입(세션 만료 안내, 복귀 경로)은 안내 유실·redirect 소실을 막기 위해 제외 * fix: 인증 경로에서 죽은 세션이 정리되지 않던 문제 수정 handleSessionExpired 가 /login·/ 에서 조기 반환해 죽은 쿠키가 남았다. 웹뷰의 FCM 동기화는 모든 페이지에서 돌아 이 경로로 401 을 받는데, 쿠키가 남으면 다음 진입에서 로그인 페이지가 멤버로 오인해 홈으로 되돌려보낸다. - 인증 경로에서는 리다이렉트 대신 세션 폐기만 수행 (자기 자신 루프는 그대로 회피) - 소셜 로그인 코드 교환 중인 /auth/callback 은 새 토큰을 덮어쓰지 않도록 제외 - 폐기 로직을 clearAuthSession 으로 모아 탈퇴 훅과 공유 - 서버 액션을 앱 전역에서 쓰게 되어 App Router 밖(actions/)으로 이동 * chore: 온보딩 노출 플래그 변수명 오타 수정 --------- Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
* refactor: 하단 바를 공통 CTA 컴포넌트로 통일 - open/open-app 스플래시의 직접 fixed 구현을 BottomCta 로 교체 - 담기 화면 in-flow 버튼 블록을 BottomCta 로 교체 - 98px 을 --spacing-bottom-cta 토큰으로 뽑고 pb-[98px] 을 pb-bottom-cta 로 교체 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> * fix: 토스트가 하단 CTA와 겹치는 문제 * test: 전역 토스트 offset 규칙 e2e 회귀 테스트 추가 * fix: e2e 타입 import 방식 수정 * fix: 담은 개수 배지를 인디케이터와 CTA 사이 중앙 정렬 * test: 토스트 offset e2e 를 실제 CTA 페이지 기준으로 보강 --------- Co-authored-by: Claude Opus 5 <noreply@anthropic.com> Co-authored-by: Jung Sun A <amber0809@naver.com>
* fix: 주최자·본인 아이템만 토너먼트 바스켓에서 클릭 가능하도록 제한 * feat: 상품 정보 화면에 조회 전용(readOnly) 모드 추가 * fix: 주최자·본인이 아닌 토너먼트 아이템은 조회 전용으로 노출 * feat: 토너먼트 아이템 INCOMPLETE 상태 지원 및 수정 진입 허용
시작 조건을 못 채운 토너먼트의 /match 로 주소를 직접 바꿔 들어가면 검증 없이 start 를 호출해 서버 400(TOURNAMENT-007) 이 나고 전역 에러 화면으로 떨어졌다. - (예방) 이미 조회한 토너먼트 정보로 대기실 버튼과 같은 기준을 먼저 검사 - (방어) start 가 007/013/014 를 주면 throw 대신 대기실로 이동 - 두 경로 모두 ?action=tournament-not-startable 로 사유 안내
앱을 켜면 화면이 멈춘 채, 서버에는 요청이 한 건도 오지 않는 상태가 prod 에서 관측됐다.
부팅은 sync() 가 끝나야 isSynced 가 true 가 되고 그때 웹뷰 source 가
{ html: '' } 에서 실제 URL 로 교체되는 구조다. 기존 .finally 방어는 promise 가
reject 될 때만 동작해, 네이티브 호출이 pending 으로 남으면 걸리지 않는다.
sync() 안의 CookieManager · TokenStorage 호출에는 타임아웃이 없다.
postTokenRefresh 는 이미 같은 이유(부팅이 요청에 묶임)로 5초 타임아웃을 두고 있어,
그 방어를 sync() 전체로 넓힌다. 타임아웃이 걸리면 쿠키가 심기지 않은 채 웹뷰가 떠
로그인 화면이 보일 수 있지만, 빈 화면으로 고착되는 것보다 낫다.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Discord 스레드 연동용 메타데이터입니다. discord-pr-bot 워크플로가 자동 생성하며, 수정·삭제하면 PR 과 Discord 알림 연동이 끊깁니다. |
|
Warning Review limit reached
Next review available in: 51 minutes Limit details: You’ve used the included review currently available. You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. How can I continue?Wait for the limit to reset, then comment An organization admin can change what happens after included review limits in Billing. How do review limits work?CodeRabbit enforces per-developer PR review limits within each organization. For paid Pro and Pro+ reviews, CodeRabbit uses a developer's included PR review attempts over the past 7 days to set the current hourly allowance. At typical activity levels, the full plan allowance applies. Higher sustained activity can lower the allowance until earlier attempts leave the 7-day window. Please refer docs for additional details. Review details⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (73)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Contributor
|
@coderabbitai full review |
|
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
5 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.
Situation
Task
Action
부팅 체인에서 멈추는 지점 특정
apps/app/app/index.tsx의 웹뷰 소스는 동기화 완료 여부에 묶여 있다.부팅은 한 줄로 엮여 있다. 빈 웹뷰 로드 완료 → 쿠키 동기화 실행 → 완료 시
isSynced를 세워 실제 URL 로 교체. 중간 한 곳이라도 완료 신호를 내지 않으면 웹뷰는 빈 화면인 채로 남고, 그 상태에서는 어떤 API 도 호출되지 않는다. 관측된 "요청 0건"과 정확히 일치한다.기존 방어가 닿지 않는 자리
useWebviewCookieSync는 이미 무한 스플래시를 막으려 했지만, 그 방어는 promise 가 reject 될 때만 동작한다..catch후.finally실행.finally가 불리지 않음sync()안에서 기다리는 네이티브 호출(CookieManager,TokenStorage)에는 타임아웃이 없다. 같은 파일 주석이 이미 iOS 쿠키 저장소의 취약성을 언급하고 있고, 특히 갱신 실패 분기에서만CookieManager.clearAll을 연속 호출한다.최소 방어 추가
postTokenRefresh에는 이미 5초 타임아웃이 있고 주석도 "부팅이 이 요청에 묶여있어 네트워크가 멈추면 무한 대기하므로 제한" 이다. 같은 문제의식이 한 곳에만 적용돼 있었으므로, 그 방어를sync()전체로 넓혔다. 어느 호출이 매달리든 8초 뒤에는 부팅이 진행된다.검토 후 채택하지 않은 안
await이 추가되면 같은 함정이 다시 열린다clearAll만 방어sync()전체를 타임아웃으로 감쌈 (채택)Result
await이 매달렸는지는 여전히 미확정이다.앱 환경에서의 확인 요청
작성자 환경에 Xcode 와 실기기가 없어 앱에서의 동작은 검증하지 못했다. 개발 빌드가 가능한 분이 아래를 봐주시면 좋겠다.
검증한 것과 못 한 것
apps/web이 아니라apps/app으로 좁혀졌다.후속
연관 이슈