-
Notifications
You must be signed in to change notification settings - Fork 0
[FEATURE/#10] 카카오 로그인/회원가입 기능 구현 #14
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
src/main/java/org/umc/valuedi/domain/auth/service/command/AuthCommandService.java
Outdated
Show resolved
Hide resolved
src/main/java/org/umc/valuedi/domain/auth/service/command/AuthCommandService.java
Outdated
Show resolved
Hide resolved
src/main/java/org/umc/valuedi/domain/auth/service/command/AuthCommandService.java
Show resolved
Hide resolved
src/main/java/org/umc/valuedi/domain/auth/service/command/AuthCommandService.java
Show resolved
Hide resolved
src/main/java/org/umc/valuedi/domain/auth/controller/AuthController.java
Outdated
Show resolved
Hide resolved
src/main/java/org/umc/valuedi/domain/auth/config/KakaoProperties.java
Outdated
Show resolved
Hide resolved
seamooll
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
수고하셨습니다!👍👍
코멘트 확인 부탁드립니다
src/main/java/org/umc/valuedi/domain/auth/controller/AuthController.java
Show resolved
Hide resolved
src/main/java/org/umc/valuedi/domain/auth/service/command/AuthCommandService.java
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
동일한 소셜 계정으로의 중복 가입을 방지하고, 동시 요청 시 데이터 무결성을 보장하기 위해 (provider, provider_user_id) 조합에 대한 유니크 제약 추가를 고려해보면 좋을 것 같습니다.
다만 현재 소프트 삭제(unlinked_at)를 사용하고 있어, 재연결을 허용할지 여부에 따라 유니크 제약 방식은 한 번 더 정리하면 좋을 것 같습니다!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
확인했습니다!
src/main/java/org/umc/valuedi/domain/auth/service/command/AuthCommandService.java
Outdated
Show resolved
Hide resolved
35edb76 to
61f544d
Compare
작업사항1. 카카오 로그인 로직 수정
2. AuthCommandService 로직 수정
3. 기타 수정 사항
다른 피드백 없으시면 충돌 해결 후 머지하겠습니다! |
a695da0 to
8d7e810
Compare
🔗 Related Issue
📝 Summary
카카오 계정으로 Valuedi 서비스에 로그인/회원가입할 수 있도록 구현했습니다.
member엔티티 ERD 변경사항username필드가 기존에NOT NULL이었으나, 소셜 로그인 시username이 존재하지 않아NULL로 변경하였습니다.CustomUserDetails구현 시 권한 정보가 필요하여role필드를 추가하였습니다.FeignClient를 이용한 카카오 인증/API 서버 통신 로직 구현member및member_auth_provider엔티티 생성)JwtAuthFilter구현🔄 Changes
💬 Questions & Review Points
현재 코드에서는 토큰을 발급해주기만 하고 Redis에 저장한다던가, Refresh Token을 이용해 Access Token을 재발급 해주는 기능은 구현되어 있지 않습니다. 나머지 기능들은 해당 PR 머지 후에 작업하겠습니다
카카오 로그인 및 JWT 관련 환경변수들은 노션에
환경 변수페이지에 정리해놨습니다!그리고 카카오 로그인 API는 Swagger로 테스트하는거 보다 영상으로 보는게 더 좋을거 같아서 영상도 함께 첨부했습니다!
📸 API Test Results (Swagger)
2026-01-12.8.25.11.mov
/auth/oauth/kakao/login호출 (302 Redirect 확인)✅ Checklist