Repository navigation
[기능] fix: 공유 토큰 쿠키 상위 도메인 삭제와 권한 없음 시 토큰 유지 - #22
Merged
Merged
Conversation
백엔드가 finditem.kr 계열 요청에는 토큰 쿠키를 .finditem.kr 도메인으로 발급하므로, 세션 만료로 토큰을 지울 때 Host 헤더를 보고 같은 도메인을 붙인다. 토큰을 운영 앱과 공유하게 되므로, 권한이 없는 계정은 토큰을 지우지 않고 로그인 화면에 안내만 띄워 운영 앱 세션이 끊기지 않게 한다. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
기능 PR과 테스트 PR을 나누는 규칙에 따라 단위 테스트와 e2e 수정을 이 브랜치에서 빼고 test/auth-cookie-domain 브랜치로 옮긴다. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
2 of 5 tasks
Jest Test Results386 tests 386 ✅ 24s ⏱️ Results for commit 1912b28. |
Open
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
1 participant
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.
Pull Request
PR 종류
관련 이슈
작업 내용
.finditem.kr도메인으로 발급합니다. 세션 만료로 토큰을 지울 때도 같은 도메인을 붙여야 지워지므로, 미들웨어가Host헤더를 보고 도메인을 정하도록 했습니다.src/utils/getAuthCookieDomain으로 분리했습니다. finditem.kr과 그 하위 도메인이면.finditem.kr을, localhost 등 그 밖의 호스트면 도메인을 붙이지 않습니다.참고 사항
체크리스트
🤖 Generated with Claude Code