[BE] 토큰 관련 코드를 리팩토링한다.

[JINU-CHANG]

❗ Issue

• [BE] 토큰 관련 코드를 리팩토링한다. #747

✨ 구현한 기능

• 토큰 에러 메시지 적용

  • 액세스 토큰과 리프레시 토큰의 에러 메시지를 다르게 하기 위해 리팩토링을 진행했습니다.

• TODO로 남겨둔 부분 리팩토링

  • 토큰이 존재하는지 검증하는 부분이 1) null 체크, 2) 토큰 유무 체크 이렇게 if문내에 존재했는데 실수하기 쉬운 코드라고 생각했습니다. 하나의 메소드에서 검증이 가능하도록 리팩토링했습니다.

// TODO 리팩토링 전
if (request.getCookies() == null || cookieResolver.isTokenNotExist(request.getCookies())) {
    throw new BangggoodException(ExceptionCode.AUTHENTICATION_TOKEN_EMPTY);
}

// 리팩토링 후
public void checkLoginRequired(HttpServletRequest request) {
      if (isTokenEmpty(request)) {
          throw new BangggoodException(ExceptionCode.AUTHENTICATION_TOKEN_EMPTY);
      }

      if (isRefreshTokenEmpty(request.getCookies())) {
          throw new BangggoodException(ExceptionCode.AUTHENTICATION_REFRESH_TOKEN_EMPTY);
      }
  }

• RefreshToken이 도입되면서 JwtTokenProvider의 책임이 많아져 JwtTokenResolver를 생성하고 역할을 분리했습니다.

📢 논의하고 싶은 내용

🎸 기타

• 이외 작업하면서 no-usage인 코드 제거했습니다.

[shin-jisong]

잘 구현해 주셔서 특별한 코멘트 사항이 없네요
수고하셨습니다~!

[shin-jisong]

소소하지만 다른 테스트는 문장형이 아니었던 것 같아요!

[tkdgur0906]

깔끔하게 잘 짜주셨네요~
고생하셨습니다~

[tsulocalize]

even하게 코드가 짜여있네요. 합격입니다.