fix: 로그인 리다이렉트 uri 변경 #14
Conversation
feat: 카카오 로그인시 쿠키 발급 및 클라이언트로 리다이렉트
Walkthroughsrc/main/resources/config submodule pointer updated from commit 30170bc860eb7a3ae50683a11ef0227323149919 to 467a38400a3d787ea99cfa494a4c67baa36b0f1d. No other lines or files changed. No exported/public declarations affected. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Suggested reviewers
Poem
Tip 🔌 Remote MCP (Model Context Protocol) integration is now available!Pro plan users can now connect to remote MCP servers from the Integrations page. Connect with popular remote MCPs such as Notion and Linear to add more context to your reviews and chats. 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. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. CodeRabbit Commands (Invoked using PR/Issue comments)Type Other keywords and placeholders
CodeRabbit Configuration File (
|
📝 테스트 커버리지 리포트입니다!
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (3)
src/main/resources/config (3)
1-1: CI/CD에서 서브모듈 체크아웃 보장 필요파이프라인에서 서브모듈을 초기화하지 않으면 빌드/배포 시 이전 설정이 사용되거나 실패할 수 있습니다. GitHub Actions를 사용 중이라면 다음 설정을 권장합니다.
- name: Checkout uses: actions/checkout@v4 with: submodules: recursive fetch-depth: 0 - name: Ensure submodules synced run: | git submodule sync --recursive git submodule update --init --recursive현재 CI에 유사 설정이 있는지 한 번 확인 부탁드립니다. 필요 시 워크플로우 수정 PR 분리 제출을 제안드립니다.
1-1: 가시성/재현성 향상: 서브모듈 커밋에 태그 추가 또는 PR 본문에 커밋 메타데이터 명시 권장SHA 고정은 재현성 측면에서 바람직합니다. 추가로,
- 설정 저장소에서 해당 커밋에
config-v2025-08-22-login-redirect같은 주석 태그(annotated tag)를 달거나,- 본 PR 본문에 서브모듈 커밋 해시/제목/변경 요약을 인라인
하면 변경 의도를 이해하기 쉬워집니다.
.gitmodules의branch=트래킹은 재현성을 떨어뜨릴 수 있어 특별한 이유가 없다면 현행(커밋 고정) 유지가 안전합니다.
1-1: 환경/프로바이더별 Redirect URI 정합성 체크리스트
- OAuth 프로바이더 콘솔(예: Google/Kakao/Naver/Apple)에 dev/stg/prod 모든 도메인의 새 URI가 화이트리스트로 등록되었는지.
- 스킴/호스트/포트 및 trailing slash 일치 여부. 프록시 뒤에 있다면
X-Forwarded-*처리(SPRING:ForwardedHeaderFilter/server.forward-headers-strategy)가 올바른지.- Spring Security를 사용한다면
spring.security.oauth2.client.registration.*.redirect-uri패턴이 의도대로 동작하는지(기본{baseUrl}/login/oauth2/code/{registrationId}vs 커스텀 경로).- 302 응답의
Location헤더가 기대 URI로 향하는지에 대한 간단한 통합/E2E 체크 추가.필요 시 간단한 E2E 테스트(로그인 시작→프로바이더 모의→콜백 URI 검증) 템플릿을 제공할 수 있습니다.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
💡 Knowledge Base configuration:
- MCP integration is disabled by default for public repositories
- Jira integration is disabled by default for public repositories
- Linear integration is disabled by default for public repositories
You can enable these sources in your CodeRabbit configuration.
📒 Files selected for processing (1)
src/main/resources/config(1 hunks)
🔇 Additional comments (1)
src/main/resources/config (1)
1-1: 서브모듈 업데이트 검증 필요 —src/main/resources/config
자동 스크립트 실행 시https://github.com/sudhdkso/moddo-infra-config.git리포지토리 클론 과정에서 “Repository not found” 오류가 발생하여(old=30170bc860eb7a3ae50683a11ef0227323149919,new=467a38400a3d787ea99cfa494a4c67baa36b0f1d) 변경 내역을 자동 확인할 수 없었습니다.
PR 본문에 다음 정보를 추가해 주세요:
- 이전 커밋과 새 커밋 간 diff 확인용 GitHub 커밋 범위 URL
- 변경된 파일 목록 및 주요 수정 사항(특히 로그인 리다이렉트 URI 관련)
- 필요 시 diff 하이라이트 스크린샷 또는 CLI 실행 결과 스니펫
위 증적이 포함되어야 리뷰 시 변경 내용을 명확히 추적할 수 있습니다. 필요하시면 정리 지원을 요청해 주세요.
#️⃣연관된 이슈
🔀반영 브랜치
🔧변경 사항
💬리뷰 요구사항(선택)
Summary by CodeRabbit