[이예원, 최순 페어]코드리뷰

[soon91]

PostingService의 createPosting에서 입력된 파라미터가 requestDto 밖에 없다보니 requestDto의 username이 저장되는 거 같습니다. 그런데 요구사항을 보시면 '저장된 게시글을 Client 로 반환하기(username은 로그인 된 사용자)' 라고 되어있습니다.
이 부분을 다시 한번 확인해보시면 좋을 것 같습니다.

[s2cocos2]

auth, posting으로 패키지를 나누어 관리하는 것이 깔끔하고 정리가 잘 되어 보입니다!!
과제의 요구사항이
수정을 요청할 때 수정할 데이터와 비밀번호를 같이 보내서 서버에서 비밀번호 일치 여부를 확인 한 후 (의 기능을 없애기)
토큰을 검사한 후, 유효한 토큰이면서 해당 사용자가 작성한 게시글만 수정 가능
(의 기능을 추가)
인데
로그인을 했는데도 불구하고 PostingRequestDto에서 username, password를 또 입력해야하는 것 같습니다~!
그리고 PostingService.java에서 예외메시지를 "비밀번호가 다릅니다"로 하신이유가 있으신가요?상황에 맞게 바꾸는건 어떻신가요???

[haechanlee96]

넵 말씀해주신 사항들 보니 그렇게 하는게 좋을 것 같습니다!! 한번 수정해보도록 하겠습니다. 감사합니다!