Skip to content
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

Refactor: 사용자 중복 OAuth 로그인 검증 로직 #86

Merged
merged 1 commit into from
Sep 29, 2024

Conversation

loveysuby
Copy link
Member

🚀 Related Issue

close: #83

📌 Tasks

  • 중복 사용자 로그인 검증 로직을 추가하였습니다.
    • 소셜 공급자에서 취급하는 사용자 정보를 WAS 내에서 대조합니다.

📝 Details

  • 사용자 외부 로그인 연결 정보를 취급하는 UserSocialAccount를 도메인으로 관리합니다.
    • User 최초 생성 시 UserSocialAccount 레코드를 함께 생성합니다.
    • 재로그인 요청 시 UserSocialAccount를 조회, userId를 추출합니다.
  • UserSocialAccountUser의 조회 용도로만 사용되므로, 별도 연관관계를 부여하지 않았습니다.
    • User DELETE 시 일괄 조회-삭제를 수행하여 처리합니다.

📚 Remarks

Points or opinions to share teams

@loveysuby loveysuby added ♻️ refactor Refactoring code or architecture 🧾 API API feature ✨ feat Improve performance or feature labels Sep 29, 2024
@loveysuby loveysuby self-assigned this Sep 29, 2024
@loveysuby loveysuby linked an issue Sep 29, 2024 that may be closed by this pull request
@loveysuby loveysuby merged commit d6bdc41 into develop Sep 29, 2024
1 check passed
@loveysuby loveysuby deleted the fix/83 branch September 29, 2024 20:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🧾 API API feature ✨ feat Improve performance or feature ♻️ refactor Refactoring code or architecture
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Fix] User 중복 가입자 등록 문제
1 participant