fix: dev(stage) 환경의 refreshToken 쿠키 이름 분리#715
Conversation
- prod와 stage가 동일한 쿠키 도메인을 공유하여 발생하는 로그인 충돌 문제 해결 - 쿠키 이름을 환경별로 분리할 수 있도록 cookie-name 설정 추가
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (4)
Walkthrough이 변경은 리프레시 토큰 쿠키의 이름을 하드코딩된 값에서 설정 가능한 값으로 전환합니다. Estimated code review effort🎯 2 (Simple) | ⏱️ ~12 minutes Suggested reviewers
상세 변경 내용1.
|
| Check name | Status | Explanation |
|---|---|---|
| Title check | ✅ Passed | 제목은 PR의 주요 변경 사항을 명확하게 반영하고 있습니다. 'dev(stage) 환경의 refreshToken 쿠키 이름 분리'라는 표현은 무엇을 했는지 정확히 전달합니다. |
| Description check | ✅ Passed | PR 설명이 관련 이슈, 작업 내용을 포함하고 있습니다. 문제 상황과 해결 방안이 명확하게 기술되어 있어 템플릿의 핵심 요소를 충족합니다. |
| Linked Issues check | ✅ Passed | PR의 코드 변경 사항이 이슈 #711의 목표와 일치합니다. 환경별 쿠키 이름 분리를 위해 RefreshTokenCookieProperties에 cookieName 필드를 추가하고 설정 파일에서 환경별 값을 구성했습니다. |
| Out of Scope Changes check | ✅ Passed | 모든 변경 사항이 refreshToken 쿠키 이름 분리라는 단일 목표에 집중되어 있습니다. 범위를 벗어난 변경은 없습니다. |
| Docstring Coverage | ✅ Passed | No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. |
✏️ Tip: You can configure your own custom pre-merge checks in the settings.
✨ Finishing Touches
📝 Generate docstrings
- Create stacked PR
- Commit on current branch
🧪 Generate unit tests (beta)
- Create PR with unit tests
- Commit unit tests in branch
fix/711-separate-dev-refresh-token-cookie-name
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 @coderabbitai help to get the list of available commands and usage tips.
관련 이슈
작업 내용
prod 와 stage 의 refreshToken 키 네임이 동일하여 발생하는 로그인 문제를 해결하기 위해 stage 서버의 key name을 stageRefreshToken 으로 수정하였습니다.
특이 사항
리뷰 요구사항 (선택)