Refactor #91 소셜 로그인 연동 기능 구현 #91

hyxklee · 2024-12-19T07:13:40Z

PR 내용

기존 로그인으로 회원가입한 사용자가 카카오 소셜 로그인으로 연동할 수 있는 기능을 구현했습니다
가입 승인 이후에 JWT Token을 반환해야하기 떄문에 일부 로직을 수정했습니다

PR 세부사항

카카오 소셜 로그인 연동을 위해 social-login , social-register, social-auth 메서드를 사용했습니다
로그인은 login/integrate로 사용되고, auth는 소셜 회원가입시 kakaoId를 프론트 측으로 전달하기 위한 api 입니다
자세한 플로우는 아래와 같습니다

소셜 회원가입

kakao 인증 (social-auth)

{
  "code": 200,
  "message": "소셜 인증에 성공했습니다.",
  "data": {
    "kakaoId": 3765018989
  }
}

소셜 회원가입 (social-register)

{
  "kakaoId": 3765018989,
  "name": "이강혁",
  "studentId": "20203189",
  "department": "인공지능전공",
  "tel": "010",
  "cardinal": 1,
  "position": "BE"
}

{
  "code": 200,
  "message": "회원 가입 신청이 성공적으로 처리되었습니다.",
  "data": null
}

소셜 로그인(social-login)

{
  "code": 403,
  "message": "가입 승인이 허가되지 않은 계정입니다.",
  "data": null
}

{
  "code": 200,
  "message": "소셜 로그인에 성공했습니다.",
  "data": {
    "id": 3,
    "kakaoId": 3765018989,
    "status": "LOGIN",
    "accessToken": "eyJ0eXAiOiJKV1QiLCJhbGciOiW1haWwiOm51bGx9.ROvg88jFJ_D_RN_C69LQXie7uczNurTWEQE4pgCwGzTInKohs5tdOCYKzhaqVx_w",
    "refreshToken": "eyJ0eXAiOiJKV1QiLCJhIiOiJSZWZyZXNoVG9rZW4iLCJpZCI6MywiZXhwIjoxNzM0NTaRkr4XwfLTtyJC0Wow2Obqc8AJfFwoEmROQA"
  }
}

소셜 로그인

연동 전

social-login api 요청

{
  "code": 200,
  "message": "소셜 로그인에 성공했습니다.",
  "data": {
    "id": null,
    "kakaoId": 3765018989,
    "status": "INTEGRATE",
    "accessToken": null,
    "refreshToken": null
  }
}

integrate api 요청

이메일이 틀릴시

{
  "code": 404,
  "message": "존재하지 않는 유저입니다.",
  "data": null
}

비밀번호 오류시

{
  "code": 400,
  "message": "비밀번호가 일치하지 않습니다.",
  "data": null
}

가입 승인이 안될시(정말 만약에)

{
  "code": 403,
  "message": "가입 승인이 허가되지 않은 계정입니다.",
  "data": null
}

ID, 비번이 맞을시

{
  "code": 200,
  "message": "소셜 로그인 연동에 성공했습니다.",
  "data": {
    "id": 2,
    "kakaoId": 3765018989,
    "status": "LOGIN",
    "accessToken": "eyJ0eXAiOiJKV1QixMiJ9.eyJzdHJpbmdAc3RyaW5nLmNvbSJ9.eWf6Nvom",
    "refreshToken": "eyJ0eXAiOiJKV1MiJ9.eyJzdWIiOiJSZWZyZXNoVG9rZW4iLCJpZCI6MiwiZXhwIjoxNzM0NTk3MDI3Q"
  }
}

연동 후, 소셜 회원가입 후

{
  "code": 200,
  "message": "소셜 로그인에 성공했습니다.",
  "data": {
    "id": 3,
    "kakaoId": 3765018989,
    "status": "LOGIN",
    "accessToken": "eyJ0eXAiOiJKV1QJIUzUxMiJ9.eyJzdWIiOiJBY2Nlc3NUb2tlbiIsInJvbGUiOiJVU0VSIiwiaWQxbhVULxSGpgCwGzTInKoMEaoFhsqVx_w",
    "refreshToken": "eyJ0eXAiOiJKV1QiLCJhbGciOiJIUzUxMiJ9..6nF8xnyFN-z4_-wwGuidBdYvxWofy5ZXlFCzwaRkr4XwfLTtyJC0Wow2Obqc8AJfFwoEmROQA"
  }
}

주의사항

소셜 회원가입시 이메일을 입력을 받아야할지 고민입니다. 기존 로그인이 없어진다면 이메일을 쓸 일이 없기 때문
토큰 발급 시기가 변함에 따라 register 메서드를 일부 수정했습니

체크 리스트

# Conflicts: # src/main/java/leets/weeth/domain/user/application/usecase/UserUseCase.java # src/main/java/leets/weeth/domain/user/application/usecase/UserUseCaseImpl.java

jj0526 · 2024-12-20T08:22:47Z

src/main/java/leets/weeth/domain/user/application/usecase/UserUseCaseImpl.java

+    @Override
+    @Transactional
+    public SocialLoginResponse integrate(NormalLogin dto) {
+        User user = userGetService.find(dto.email());


이렇게 되면 카카오 계정 이메일이랑 일반 이메일이 일치하지 않는 경우도 있나요?

어떤 부분에서 불일치를 말씀하시는 걸까요??

아! 이메일이 일치하는 경우만 통합되네요

전화번호나 학번을 이용해서 통합하는건 어려울까요? 현재 사용하는 이메일과 카카오 이메일이 다른 경우도 있을 수 있다 생각해요

연동은 kakaoId를 통해서 연동합니다!
여기서 입력 받는 이메일은 사용자 인증을 위해 ID/비번을 받는 부분이라서 연동에 직접적으로 사용되진 않습니다
사용자 인증이 완료되면 null인 kakaoId 컬럼에 데이터가 삽입돼서 연동이 되는 방식입니당

huncozyboy

고생하셨습니다 !
이메일과 관련해서는 소셜 로그인의 장점이 간편한 회원가입 + 로그인이라 초기 정보입력시 필드를 최소화 하는 것이 좋아보이기도 하고, 저희 서비스단에서 중복 가입은 허용이 안되도록 해놓아서 이메일을 따로 입력받지 않아도 될거같다는 개인적인 생각이있습니다.
해당 부분 관련해서는 프론트분들의 의견을 들어봐도 좋을거같습니다

huncozyboy · 2024-12-20T13:44:03Z

src/main/java/leets/weeth/domain/user/domain/service/UserGetService.java

+        return userRepository.findByKakaoId(kakaoId)
+                .orElse(null);


현재 kakaoId로 유저를 조회할 때, 값이 없으면 예외처리를 하지않고 null을 반환하는 것이 카카오 ID가 없는 것이 예외상황이 아닌 정상적인 상황으로 간주하고 mapper.toIntegrateResponse(kakaoId)를 이용하신 로직으로 이해했습니다
해당 메서드에 null을 반환하는 대신에 Optional을 사용해주는 방식은 어떨까요 ??

Optional로 반환해주는 것이 값의 여부를 구분하기에 더 명시적일 거 같아요
수정하겠습니다!

huncozyboy · 2024-12-20T14:07:59Z

src/main/java/leets/weeth/domain/user/presentation/UserController.java

        return CommonResponse.createSuccess(USER_APPLY_SUCCESS.getMessage());
    }

+    @PatchMapping("/integrate")


해당 엔드포인트도좋지만 동사보다는 명사의 형태로 설계하는 것이 지향된다고 알고있어,
social-link나 kakao-link와 같은 형태로 개선하면 더 RESTful한 설계가 될거같은데 어떻게 생각하실까요 ??

앗 그렇네요 좀 더 좋은 엔드포인트로 고민해보겠습니닷

jj0526 · 2024-12-23T02:50:21Z

src/main/java/leets/weeth/domain/user/application/usecase/UserUseCaseImpl.java

+    @Override
+    @Transactional
+    public SocialLoginResponse integrate(NormalLogin dto) {
+        User user = userGetService.find(dto.email());


아! 이메일이 일치하는 경우만 통합되네요

jj0526 · 2024-12-23T02:51:29Z

src/main/java/leets/weeth/domain/user/application/usecase/UserUseCaseImpl.java

+    @Override
+    @Transactional
+    public SocialLoginResponse integrate(NormalLogin dto) {
+        User user = userGetService.find(dto.email());


전화번호나 학번을 이용해서 통합하는건 어려울까요? 현재 사용하는 이메일과 카카오 이메일이 다른 경우도 있을 수 있다 생각해요

jj0526 · 2024-12-23T03:03:15Z

src/main/java/leets/weeth/domain/user/application/dto/request/UserRequestDto.java

+    ) {
+    }
+
+    public record Register(


SignUp과 Register 이름이 헷갈릴거같은데 kakao를 앞에 추가하는건 어떤가요?

socialRegister로 수정하겠습니당

huncozyboy · 2024-12-23T04:19:52Z

고생하셨어요 문제 없어보입니다 !

jj0526

수고하셨어요! 연동할땐 카카오 로그인과 기존 로그인을 같이 받는거였네요

hyxklee added 20 commits December 8, 2024 15:16

feat: kakaoId 변수 추가

3feedc8

feat: 연동 상태 추가

a9af30b

feat: 상수 메시지 추가

9e4c1c6

feat: 커스텀 예외 추가 구현

3ec0f99

feat: 소셜 로그인 및 연동을 위한 API 추가

adf3ea0

feat: 소셜 로그인 및 연동을 위한 메서드

c5bfb8a

refactor: 불필요한 의존성 제거

bd1ed67

feat: 매퍼 메서드 추가 구현

682d77e

feat: 카카오 id로 조회 메서드 구현

e91af1a

refactor: 컨벤션 수정

ee18be7

feat: dto에 kakaoId 추가

730baf2

feat: 소셜 로그인 및 연동을 위한 메서드 추가

2ed744f

feat: 소셜 로그인 및 연동을 위한 메서드 추가

7329234

feat: 경로 추가 허용

cec41d2

feat: todo 작성

2b2c1b5

refactor: api 명 및 메서드 변경

4fd4444

refactor: login 로직 이전

0c92538

refactor: api 설명 변경

d3c0e77

Merge branch 'refs/heads/develop' into refactor/#88/소셜-로그인-수정

edcf2c5

# Conflicts: # src/main/java/leets/weeth/domain/user/application/usecase/UserUseCase.java # src/main/java/leets/weeth/domain/user/application/usecase/UserUseCaseImpl.java

refactor: 충돌병합

15a118c

hyxklee added feature refactor labels Dec 19, 2024

hyxklee requested review from jj0526 and huncozyboy December 19, 2024 07:13

hyxklee self-assigned this Dec 19, 2024

hyxklee changed the title ~~Refactor #80 소셜 로그인 연동 기능 구현~~ Refactor #91 소셜 로그인 연동 기능 구현 Dec 19, 2024

hyxklee linked an issue Dec 19, 2024 that may be closed by this pull request

Refactor #88 카카오 소셜 로그인 연동을 위한 로직 수정 #88

Closed

jj0526 reviewed Dec 20, 2024

View reviewed changes

huncozyboy reviewed Dec 20, 2024

View reviewed changes

refactor: Optionalˆ로 수정

e0391fb

refactor: 엔드포인트 수정

eb65b49

jj0526 reviewed Dec 23, 2024

View reviewed changes

refactor: 메서드 명 수정

db51d08

jj0526 approved these changes Dec 23, 2024

View reviewed changes

hyxklee merged commit 1c79845 into develop Dec 23, 2024
2 checks passed

hyxklee deleted the refactor/#88/소셜-로그인-수정 branch December 23, 2024 11:03

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Refactor #91 소셜 로그인 연동 기능 구현 #91

Refactor #91 소셜 로그인 연동 기능 구현 #91

hyxklee commented Dec 19, 2024

jj0526 Dec 20, 2024

hyxklee Dec 22, 2024

jj0526 Dec 23, 2024

jj0526 Dec 23, 2024

hyxklee Dec 23, 2024

huncozyboy left a comment •

edited

Loading

huncozyboy Dec 20, 2024

hyxklee Dec 22, 2024

huncozyboy Dec 20, 2024

hyxklee Dec 22, 2024

jj0526 Dec 23, 2024

jj0526 Dec 23, 2024

jj0526 Dec 23, 2024

hyxklee Dec 23, 2024

huncozyboy commented Dec 23, 2024

jj0526 left a comment •

edited

Loading

Refactor #91 소셜 로그인 연동 기능 구현 #91

Refactor #91 소셜 로그인 연동 기능 구현 #91

Conversation

hyxklee commented Dec 19, 2024

PR 내용

PR 세부사항

소셜 회원가입

소셜 로그인

연동 전

연동 후, 소셜 회원가입 후

관련 스크린샷

주의사항

체크 리스트

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

huncozyboy left a comment • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

huncozyboy commented Dec 23, 2024

jj0526 left a comment • edited Loading

Choose a reason for hiding this comment

huncozyboy left a comment •

edited

Loading

jj0526 left a comment •

edited

Loading