[🔨fix] 회원가입 관련 상태 코드 수정 및 예외 처리 변경

[jsoonworld]

📄 Work Description

• 회원가입 관련 상태 코드를 더 적절하게 수정했습니다.
  • 회원가입 실패 시 401 Unauthorized에서 400 Bad Request로 변경.
  • 이미 존재하는 유저일 경우 401 Unauthorized에서 409 Conflict로 변경.
• AuthServiceImpl에서 잘못된 리프레시 토큰 사용 시 예외 처리를 INVALID_USER에서 FAILED_TOKEN_REISSUE로 수정.

⚙️ ISSUE

• closed [FIX] 회원가입 에러 #141

💬 To Reviewers

• 클라이언트 사이드에서 회원가입 관련 이슈가 있다고 해서 서버 로직을 확인했지만, 서버 로직에는 문제가 없었습니다.
• 클라이언트 로직을 함께 살펴보고 문제를 해결하였으며, 그 과정에서 상태 코드 수정 요청이 있었습니다.
• 회원가입 실패 및 중복 유저 관련 상태 코드를 더 명확하게 하기 위해 수정했으니, 코드 변경 사항을 확인해 주시기 바랍니다.
• 상태 코드 변경이 서버와 클라이언트 간 통신에 문제가 없도록 추가적인 검토를 부탁드립니다.

🔗 Reference

• HTTP 상태 코드 관련 문서

[junggyo1020]

이미 존재하는 유저를 409로 보내준 부분 좋네요!! 고생하셨습니다:)

[JungYoonShin]

수고하셨습니다!! 궁금한 점이 2가지 있는데요! 이에 대한 장순님의 의견이 궁금합니다 :)

1. 저희 회원가입시 이미 존재하는 유저에 대해 예외처리를 하는 로직이 있나요?! (단순 궁금해서ㅎㅎ)

• 회원가입 실패 시 401 Unauthorized에서 400 Bad Request로 변경.
• 이미 존재하는 유저일 경우 401 Unauthorized에서 409 Conflict로 변경.

✚ 추가로 개인적인 의견이지만, 회원가입에 실패하였습니다라는 예외처리가 조금 추상적이라는 생각이 들었습니다!

회원가입의 실패 이유가 다양하다고 생각하는데(ex. 유효하지않은 authId, 카카오측으로 부터 사용자 정보 조회에서 발생한 오류 등등), 이런식으로 구체적인 상황에 맞게 에러코드를 구분하면 클라 및 서버측에서도 에러대응하기에 조금 더 좋을 것 같습니다!

2. ErroMessage에서 FAILED_WITHDRAW(404, "계정 탈퇴에 실패하였습니다"),로 관리하고 계시던데, 404로 하신 이유가 있을까요?!

@jsoonworld