[♻️ refactor/#143]: 불필요한 중복 에러 메시지 제거 및 코드 정리

[jsoonworld]

📄 Work Description

• 불필요한 중복 에러 메시지 제거 및 코드 정리
• 계정 탈퇴 시 잘못된 404 에러 코드를 400 에러 코드로 수정

⚙️ ISSUE

• closed [♻️ refactor] 소셜 로그인 > 회원가입 및 계정 탈퇴 예외 처리 코드 및 메시지 구체화 작업 #143

💬 To Reviewers

• 계정 탈퇴 실패 관련 에러 코드 변경 사항을 검토 부탁드립니다.
• 이외 다른 에러 메시지 중에서도 불필요한 중복이 있을 수 있으니 확인 부탁드립니다.
• 현재 회원가입을 할 때 클라이언트 측에서만 검증 로직을 처리하고 서버에서는 따로 검증을 하지 않고 있는데, 이번 기회에 서버에서도 검증 로직을 도입하려고 합니다. 이에 대한 의견을 듣고 싶습니다.
• 또한, 기존의 소셜 로그인 상태 코드들에 대해서도 전반적인 의견을 부탁드립니다.

🔗 Reference

• HTTP 응답 상태 코드

[JungYoonShin]

우선 수고하셨습니다~! 요청하신 사항에 대한 코멘트 남겨요 :)

현재 회원가입을 할 때 클라이언트 측에서만 검증 로직을 처리하고 서버에서는 따로 검증을 하지 않고 있는데, 이번 기회에 서버에서도 검증 로직을 도입하려고 합니다. 이에 대한 의견을 듣고 싶습니다.

검증 로직 추가하는 건 전 좋다고 생각합니다~!

[JungYoonShin]

404에서 400으로 에러코드를 변경하신 이유가 궁금해요!

아래 코드에서 사용중이신 걸로 확인했는데요!

• 400(Bad request)는 보통 클라의 요청이 잘못되었을 경우 해당하는데 아래 코드에서는 user를 테이블에서 삭제하려는 시점에서 발생하는 오류 같아서요..!
• 개인적인 생각이지만 500에러가 더 적합하지 않을까 싶슴다!

    @Override
    public void deleteUser(User user) {
        try {
            userRepository.delete(user);
        } catch (Exception e) {
            throw new CustomException(FAILED_WITHDRAW);
        }
    }