리뷰 남깁니다 ~

[openmpy]

1. Controller 부분에서 api 주소 마지막에 슬래시(/)는 빼는게 맞다고 합니다.
   참고 자료: https://blog.restcase.com/7-rules-for-rest-api-uri-design

수정 전

@PostMapping("/")

수정 후

@PostMapping

---

3. LectureService 부분에서 checkAuthority 메서드 아래 처럼 줄일 수 있을거 같습니다.

수정 전

private boolean checkAuthority(String tokenValue) {
    boolean isManager = true;
    String token = jwtUtil.substringToken(tokenValue);
    //권한이 Manager인지 확인
    Claims info = jwtUtil.getUserInfoFromToken(token);
    String authority = (String) info.get(JwtUtil.AUTHORIZATION_KEY);
    if (!(AuthEnum.valueOf(authority)).equals(AuthEnum.MANAGER)) {
        return !isManager;
    }
    return isManager;
}

수정 후

private boolean checkAuthority(String tokenValue) {
    String token = jwtUtil.substringToken(tokenValue);
    //권한이 Manager인지 확인
    Claims info = jwtUtil.getUserInfoFromToken(token);
    String authority = (String) info.get(JwtUtil.AUTHORIZATION_KEY);
    return (AuthEnum.valueOf(authority)).equals(AuthEnum.MANAGER);
}

---

4. LectureService, TutorService 에서도 checkAuthority 메서드가 중복적으로 사용되는데 util 형식으로 만드는 것도 좋아보입니다.

---

조심스럽게 리뷰 남겨봤습니다!
잘 작성하셔서 딱히 피드백 남길게 없네요 👍
혹시나 피드백에 대해서 궁금하신 점이 있으시다면 편하게 연락 주세요!
고생 많으셨습니다 ~~
다음 과제 잘 부탁드리겠습니다 😃