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.

Sign up for GitHub

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/#143]: 불필요한 중복 에러 메시지 제거 및 코드 정리 #150

Merged
merged 1 commit into from
Sep 27, 2024
Merged

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

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 1 addition & 5 deletions src/main/java/org/terning/terningserver/exception/enums/ErrorMessage.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -15,10 +15,6 @@ public enum ErrorMessage {
FAILED_TOKEN_REISSUE(404, "토큰 재발급에 실패하였습니다"),
UNAUTHORIZED_JWT_EXCEPTION(401, "유효하지 않은 토큰입니다"),

// 회원가입
FAILED_SIGN_UP(400, "회원가입에 실패하였습니다"),
EXISTS_USER_ALREADY(409, "이미 존재하는 유저입니다"),

// 사용자 필터링 정보 생성
FAILED_SIGN_UP_USER_FILTER_CREATION(404, "사용자 필터 생성에 실패하였습니다"),
FAILED_SIGN_UP_USER_FILTER_ASSIGNMENT(404, "사용자 필터 연결에 실패하였습니다"),
Expand All @@ -32,7 +28,7 @@ public enum ErrorMessage {
FAILED_REFRESH_TOKEN_RESET(400, "리프레쉬 토큰 초기화에 실패하였습니다"),

// 계정 탈퇴
FAILED_WITHDRAW(404, "계정 탈퇴에 실패하였습니다"),
FAILED_WITHDRAW(400, "계정 탈퇴에 실패하였습니다"),
Comment on lines -35 to +31
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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);
        }
    }


// 마이페이지
INVALID_PROFILE_IMAGE(404, "유효하지 않은 프로필 이미지 입니다."),
Expand Down
Loading