woowacourse-teams / 2022-ternoko

면담은 찐하게, 예약은 손쉽게! 올인원 면담 예약 서비스 터놓고 💖
https://ternoko.site
44 stars 6 forks source link

[BE] fix: 본인의 면담 내용이 아닌 경우에도 조회 가능 #445

Closed HyeonbinSa closed 2 years ago

HyeonbinSa commented 2 years ago

As-is

인터뷰 조회 시 Interview Id를 바꾸면 본인의 면담이 아닌 경우에도 조회가 가능하다. 면담 수정 과정에서 url에서 interviewId를 바꾸면 다른 사람의 인터뷰 내용을 볼 수 있음.(수정 버튼 클릭 시 실패함)

image

사진 상 면담 내용은 열음이 작성한 면담.

To-be

본인의 면담 내용이 아닌 것을 조회할 경우 조회 불가능하도록 해야함.

HyeonbinSa commented 2 years ago

현재 면담 조회 시 사용되고 있는 Controller Method

@GetMapping("/interviews/{interviewId}")
public ResponseEntity<InterviewResponse> findInterviewById(@PathVariable final Long interviewId) {
    final InterviewResponse interviewResponse = interviewService.findInterviewResponseById(interviewId);
    return ResponseEntity.ok(interviewResponse);
}

interviewId를 통해 조회하는 기능만 구현되어있기때문에 다른 사람의 면담 또한 조회가 가능합니다. 그래서 해당 기능의 수정이 필요해보입니다.

Coach ID or Crew ID 를 입력 받아 Interview에 있는 Member인지 판단하는 로직 구현이 필요해보입니다.