[feat] Kakao OAuth 회원가입, 로그인, 로그아웃 기능

[jiwon1217]

🛰️ Issue Number

• [feat] Kakao OAuth 회원가입, 로그인, 로그아웃 기능 #2

🪐 작업 내용

• Kakao OAuth 회원가입, 로그인, 로그아웃 기능을 구현했습니다.
• 아직 커스텀 예외처리는 적용하지 않았습니다.
• Kakao OAuth 애플리케이션 정보는 수업 시간에 생성했던 환경을 그대로 사용했습니다. 이 부분도 어떻게 관리할 지 논의가 필요해보입니다.
• 제가 구현한 기능의 흐름은 다음과 같습니다
  • 뇌이싱 시작하기 버튼 클릭 → /oauth2/authorization/kakao 으로 요청 → Kakao 로그인 화면에서 ID/Password 입력 후 OAuth 인증 → 닉네임 설정 페이지에서 닉네임 입력 후 /signup 요청 → 회원가입 완료
  • 만약 Kakao OAuth 인증만 한 상태로 닉네임 설정을 하지 않으면 User 테이블에 nickname이 null인 user가 생성됨 → 그래서 사용자 정보를 불러와서 nickname이 null인 사용자면 닉네임 설정 페이지로 분기 되도록 json 응답 처리
    • 이것에 대한 json 응답은 API 명세서에 추가했습니다.

@Override
    public void onAuthenticationSuccess(HttpServletRequest request, HttpServletResponse response,
        Authentication authentication) throws IOException {
        UserPrincipal principal = (UserPrincipal) authentication.getPrincipal();
        response.setContentType("application/json;charset=UTF-8");

        if (principal.getUserNickname() == null) {
            // 닉네임 설정 필요 → 202 Accepted
            response.setStatus(HttpServletResponse.SC_ACCEPTED);
            objectMapper.writeValue(response.getWriter(), Map.of(
                "message", "닉네임을 설정하세요."
            ));
        } else {
            // 정상 로그인 → 200 OK
            response.setStatus(HttpServletResponse.SC_OK);
            objectMapper.writeValue(response.getWriter(), Map.of(
                "message", "로그인 성공"
            ));
        }
    }

• 혹시 몰라서 인증 정보는 세션과 SecurityContextHolder 두 군데 모두 저장했습니다.

private void updateSessionAfterSignup(HttpSession session, User user) {
        session.removeAttribute("OAuthUser");
        session.setAttribute("user", new SessionUser(user));
    }

 public static void setAuthentication(User user) {
        UserPrincipal userPrincipal = new UserPrincipal(user, Collections.emptyMap());
        UsernamePasswordAuthenticationToken authentication =
            new UsernamePasswordAuthenticationToken(userPrincipal, null,
                userPrincipal.getAuthorities());
        SecurityContextHolder.getContext().setAuthentication(authentication);
    }

• handler가 많이 추가돼서 service/handler로 한 번더 패키지를 분리했습니다.
• SessionUser, UserPrincipal 모두 값을 저장하고 전달하는 역할 수행하기 때문에 dto 패키지에 생성했습니다.
• Mapper를 추가하지 않아서 관련 리팩토링 작업은 다음 이슈에서 진행하겠습니다

📚 Reference

✅ Check List

• 코드가 정상적으로 컴파일되나요?
• 테스트 코드를 통과했나요?
• merge할 브랜치의 위치를 확인했나요?
• Label을 지정했나요?

[silver-eunjoo]

고생하셨습니다 ! 혹시 현재 로그인한 유저의 id를 가져올 수 있게 UserDetails의 principal의 id를 꺼내올 수 있는 메서드도 구현해주실 수 있으실까요?

[jiwon1217]

고생하셨습니다 ! 혹시 현재 로그인한 유저의 id를 가져올 수 있게 UserDetails의 principal의 id를 꺼내올 수 있는 메서드도 구현해주실 수 있으실까요?

public Long getUserId() {
    return user.getId();
}

public String getUserNickname() {
    return user.getNickname();
}

UserPrincipal에 해당 메서드 추가해뒀습니다 !

[sehee123]

[L4-변경제안]
여기서 User 엔티티대신 User에서 필요한 부분만 분리해서 dto로 변경한다음에 넣어주는건 어떨까요?
User 엔티티 전체를 넣어서 principal을 생성하면 provider, providerId, lastLogin같은 불필요한 정보들이 모두 포함되어있을 필요는 없다고 생각됩니다.
그리고 User라는게 엔티티이다보니 영속성 대상이 되는것도 . . 뭔가 꼬이지 않을까요 (?)

[jiwon1217]

다른 분들이 UserPrincipal을 읽어와서 각자 맡은 기능을 구현하실 때, 구체적으로 어떤 필드가 필요할 지 몰라서 분리를 안하고 user 전체를 넣었습니다 ! 그래서 각자 맡은 부분에 대한 초기 PR이 올라오면 그것을 보고 mapper 변환 작업 진행하면서 같이 분리해보겠습니다 👍
닉네임 수정 API를 아직 구현하지 않아서 생각하지 못한 부분이네요,,, 말씀대로 User 값이 변경되는 코드를 수행한 후에는 현재는 user 전체를 저장하기 때문에 다시 메서드를 호출해서 권한을 세팅해줘야 할 것 같습니다.
다른 분들이 작업하시는 코드 보고 달아주신 리뷰 반영해보겠습니다 !

[silver-eunjoo]

고생하셨습니다 ! 혹시 현재 로그인한 유저의 id를 가져올 수 있게 UserDetails의 principal의 id를 꺼내올 수 있는 메서드도 구현해주실 수 있으실까요?

public Long getUserId() {
    return user.getId();
}

public String getUserNickname() {
    return user.getNickname();
}

UserPrincipal에 해당 메서드 추가해뒀습니다 !

[L4-변경제안]
옹.. 이게 그 부분이었군요 !

혹시 이렇게 사용하면 어떻게 사용해야 하나요?

지금까지는 SecurityContextHolder에서 꺼내쓰는 방식으로만 구현해봤어서요 !!

UserPrincipal을 컨트롤러에서 @AuthenticationPrincipal UserPrincipal userPrincipal 파라미터를 추가해주면 사용할 수 있는 건가요?
만약, 이 방법으로 사용해야 한다면, 서비스에서도 바로 호출해서 사용할 수 있도록 SecurityContextHolder에서 꺼내 쓰는 방식으로 사용할 수 있도록 따로 클래스와 메서드를 빼는 것은 어떤가요?

[dlsrks1021]

[L5-참고의견]
validateDuplicateNickname 메서드는 validateNickname 내부에 위치시켜도 좋을 것 같습니다.

[dlsrks1021]

[L5-참고의견]
닉네임 중복은 E409001 에러코드가 존재하는 것으로 확인됩니다.

[jiwon1217]

[L4-변경제안] 옹.. 이게 그 부분이었군요 !

혹시 이렇게 사용하면 어떻게 사용해야 하나요?

지금까지는 SecurityContextHolder에서 꺼내쓰는 방식으로만 구현해봤어서요 !!

UserPrincipal을 컨트롤러에서 @AuthenticationPrincipal UserPrincipal userPrincipal 파라미터를 추가해주면 사용할 수 있는 건가요? 만약, 이 방법으로 사용해야 한다면, 서비스에서도 바로 호출해서 사용할 수 있도록 SecurityContextHolder에서 꺼내 쓰는 방식으로 사용할 수 있도록 따로 클래스와 메서드를 빼는 것은 어떤가요?

넵 말씀하신대로 util 메서드를 추가하면 @AuthenticationPrincipal UserPrincipal userPrincipal 파라미터를 추가하지 않아도 되기 때문에 코드가 깔끔해질 것 같습니다. util 클래스로 분리해서 메서드 추가해보겠습니다 !

[LimKangHyun]

고생하셨습니다!

[sehee123]

수고하셨습니다. 🙌