[Feature/#2] 소셜 로그인 구현

[Mingyum-Kim]

1️⃣ 작업 내용 (Contents)

OAuthClient를 인터페이스로 두고 Kakao와 Apple을 분리하는 걸 구현하는 과정에서 애를 좀 먹었다는 .. 😢
일단 서비스 테스트만 작성했는데, 실제로 인증 서버에서 승인 가능한 accessToken을 받아올 수가 없는지라 mockito를 사용해서 그 부분만 처리하였습니다!
작업 내용이 너무 많아서 검토하기 힘들 것 같은데, 천천히 내일까지 리뷰해주시면 감사하겠습니다 ❗❗❗

2️⃣ 링크 (Links)

#2

3️⃣ 희망 리뷰 완료 일 (Expected due date)

8월 11-12일

[jemlog]

• 저희 프로젝트 위키에 적혀있는 디렉토리 구조에 따르면 auth 밑에 dto라는 디렉토리가 있어야 하는데, 민겸님께서는 application과 presentation 디렉토리 하단에 각각 dto라는 디렉토리를 만드셨네요! 🧐 저도 사실 민겸님처럼 구성하는 편이긴 한데, 민겸님 방식으로 진행할까요?! 참고로 저는 dto 디렉토리 밑에 request, response로 한번 더 나누긴 합니다~
• 그때 카페에서 이야기 나눴던 방식은 access token과 refresh token을 함께 발급하는 방식이었는데 혹시 refresh token은 차후에 개발할 예정일까요??

[Mingyum-Kim]

• 말씀하셨던대로 Controller와 Service 등 계층 별로 dto를 다르게 가져갈 예정이라면 제 방법대로 하는 게 좋을 것 같아요 ! request와 response 폴더 한 번 더 나누는 것으로 수정하겠습니다~
• 우선적으로 access token 만 구현하였습니다! refresh token 과 기타 인증 로직과 함께 다른 PR에 반영하도록 할게요 👍

[jemlog]

baseEntity의 경우 민겸님이 따로 LocalDateTime.now()로 초기화 하지 않아도 자동으로 현재 시간이 기록됩니다~

[Mingyum-Kim]

userRepository.save() 함수 호출 시 created_at, updated_at이 NULL이 될 수 없다는 에러가 터져서 초기화를 해놓은 건데, 원래 아무런 설정을 하지 않아도 now()로 설정이 되어야하는건가요 ?

[jemlog]

혹시 메인 springbootapplication 클래스 위해 EnableJpaAuditing 어노테이션 붙이셨을까요?!

[jemlog]

저희가 IOS 분들이랑 한번 더 회의를 해야 결정이 될 것 같긴 한데, 차후에 공통 응답 스펙으로 한번 더 감싸면 좋을 것 같습니다~~!!

[Mingyum-Kim]

공통 응답 스펙으로 한번 더 감싼다는게 정확히 어떤 방식인지 사실 이해가 안가서요 .. ! 일단 제 방식대로 코딩했는데 나중에 한번더 얘기해보면 좋을 것 같아요 ~

[jemlog]

@Data
@AllArgsConstructor
@NoArgsConstructor
public class ApiResponse<T> {

    private int code;
    private String message;
    private T data;


    public ApiResponse(final int code, final String message) {
        this.code = code;
        this.message = message;
    }

    public static <T> ResponseEntity<ApiResponse> success(final SuccessCode successCode, final T data) {
        return ResponseEntity.status(successCode.getStatus())
                .body(new ApiResponse<>(successCode.getStatus(), successCode.getMessage(), data));
    }

    public static <T> ResponseEntity<ApiResponse> success(final SuccessCode successCode) {
        return ResponseEntity.status(successCode.getStatus())
                .body(new ApiResponse<>(successCode.getStatus(), successCode.getMessage(), null));
    }
}

이런 공통 응답 클래스 만들어서

    // OAuth2를 사용한 소셜 인증
    @PostMapping("/social")
    public ResponseEntity<ApiResponse> socialLogin(@Valid @RequestBody OAuthLoginRequest oAuthLoginRequest) {

        OAuthResponseDto oAuthResponseDto = oAuthService.login(oAuthLoginRequest.toDto());
        return ApiResponse.success(SuccessCode.INSERT_SUCCESS, oAuthResponseDto);
    }

이런 식으로 클라이언트에서 같은 양식의 응답을 받을 수 있도록 하는거에요~

[jemlog]

👍👍 이런 팩토리 메서드 쓰면 깔끔해지고 좋은 것 같아요~

[jemlog]

혹시 getUserInfo 메서드에 uri로 KAKAO_API_URL을 넣고, kakaoOauthLoginClient() 메서드에 다시 한번 KAKAO_API_URL을 넣은 이유가 있을까요?!

[Mingyum-Kim]

기본 세팅 시 baseUrl에는 'kapi.kakao.com'을, API 호출 시 uri에 'v2/user/me'를 넣는 구조라서 저렇게 작성했습니다! 주소는 다시 수정하도록 할게요 :)

[jemlog]

엇 혹시 애플 로그인이 KAKAO와 동일한 과정으로 유저 정보가 들어오나요?! 아마 토큰을 받아와서 복호화를 거치는 등의 복잡한 과정이 포함되있었던것 같아서 궁금해서 말씀드려요~~!!

[Mingyum-Kim]

네 그렇네요 .. ㅜㅜ 확인해보니 복호화하는 과정이 있는 것 같습니다 이것도 다시 수정할게요!
참고 : https://hello-gg.tistory.com/m/65

[jemlog]

socialId는 모든 유저가 고유하게 가지고 있는 식별자인데, 혹시 provider와 함께 조회를 하는 이유가 있을까요?!

[Mingyum-Kim]

카카오와 애플이 소셜 로그인시 각각의 id를 발급하기 때문에 provider와 id를 함께 조회해야한다고 생각했는데, id로만 조회하는 게 더 낫다고 생각하시나요?

[jemlog]

저는 socialID 자체가 중복될 수 없는 값이고, 카카오와 애플도 socialID 형식이 달라서 socialID 컬럼 하나만으로 모든 유저를 식별할 수 있다고 생각했습니다! socialID만으로 필터링하나, socialID + Provider로 필터링하나 응답 결과가 똑같을꺼에요!

[jemlog]

혹시 메인 springbootapplication 클래스 위해 EnableJpaAuditing 어노테이션 붙이셨을까요?!

[jemlog]

@Data
@AllArgsConstructor
@NoArgsConstructor
public class ApiResponse<T> {

    private int code;
    private String message;
    private T data;


    public ApiResponse(final int code, final String message) {
        this.code = code;
        this.message = message;
    }

    public static <T> ResponseEntity<ApiResponse> success(final SuccessCode successCode, final T data) {
        return ResponseEntity.status(successCode.getStatus())
                .body(new ApiResponse<>(successCode.getStatus(), successCode.getMessage(), data));
    }

    public static <T> ResponseEntity<ApiResponse> success(final SuccessCode successCode) {
        return ResponseEntity.status(successCode.getStatus())
                .body(new ApiResponse<>(successCode.getStatus(), successCode.getMessage(), null));
    }
}

이런 공통 응답 클래스 만들어서

    // OAuth2를 사용한 소셜 인증
    @PostMapping("/social")
    public ResponseEntity<ApiResponse> socialLogin(@Valid @RequestBody OAuthLoginRequest oAuthLoginRequest) {

        OAuthResponseDto oAuthResponseDto = oAuthService.login(oAuthLoginRequest.toDto());
        return ApiResponse.success(SuccessCode.INSERT_SUCCESS, oAuthResponseDto);
    }

이런 식으로 클라이언트에서 같은 양식의 응답을 받을 수 있도록 하는거에요~

[jemlog]

저는 socialID 자체가 중복될 수 없는 값이고, 카카오와 애플도 socialID 형식이 달라서 socialID 컬럼 하나만으로 모든 유저를 식별할 수 있다고 생각했습니다! socialID만으로 필터링하나, socialID + Provider로 필터링하나 응답 결과가 똑같을꺼에요!