Repository navigation
fix(stomp): 같은 계정 다른 탭에도 일정·북마크 변경을 탭 ID로 구분해 반영 - #260
Merged
Merged
Conversation
- 탭마다 한 번 만드는 getTabId 추가(모듈 변수, randomUUID 없으면 getRandomValues로 v4) - 이미 preflight를 하는 PATCH·PUT·DELETE·JSON POST에만 헤더를 붙임 - 백엔드가 이 값을 STOMP payload originTabId로 돌려줘 이 탭의 반향을 가려낼 수 있게 함 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
- 이벤트 originTabId가 이 탭 ID와 같을 때만 반향으로 보고 건너뜀(isOwnEcho) - originTabId가 없는 옛 이벤트는 지금처럼 actorUserId로 판정해 배포 순서와 무관 - SCHEDULE_ITEM_UPDATED: 이 탭 반향은 건너뛰고, 다른 탭 변경은 진행 중인 조회에 합류하지 않고 다시 받음 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
- bookmarks 이벤트도 isOwnEcho로 판정(originTabId 같을 때만 건너뜀, 없으면 actorUserId) Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
- isOwnEcho는 originTabId가 이 탭 ID와 같을 때만 반향으로 판정(계정 판정 제거) - 탭 ID를 못 만든 탭의 변경도 같은 계정의 다른 탭이 놓치지 않게 함 - SCHEDULE_ITEM_UPDATED의 본인 반향 합류(joinInFlight) 분기 제거 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
- case마다 반복하던 isOwnEcho를 한 번만 둬 새 이벤트 종류에서 빠뜨리지 않게 함 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
- 반향은 originTabId가 이 탭이고 actorUserId도 나일 때만으로 판정 - 탭 ID는 방 멤버 모두에게 보이므로 서버가 인증으로 정한 actorUserId를 함께 봄 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
- 검증된 경로 캐시 부모 브랜치 병합 - 탭별 본인 변경 반향 처리와 시간 보존 동작 유지
|
This branch was successfully deployed
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.
노션
변경 내용
randomUUID가 없는 비보안 컨텍스트는getRandomValues로 v4). 이미 CORS preflight를 하는 PATCH·PUT·DELETE·JSON POST에만X-Client-Tab-Id헤더로 싣습니다.isOwnEcho하나로 모았습니다.originTabId가 이 탭이고actorUserId도 나일 때만 건너뛰고, 같은 계정의 다른 탭이나 탭 ID가 없는 이벤트는 다른 사람의 변경처럼 받습니다.SCHEDULE_ITEM_UPDATED합류 분기와invalidateSchedulePlanPlaces의joinInFlight옵션을 지웠습니다.변경 이유
테스트
npm run lintnpm testnpm run build🤖 Generated with Claude Code