Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

PostingController의 getAttachList() 메서드가 Entity를 반환하는 문제를 해결한다. #420

Open
2 tasks
gusah009 opened this issue Sep 20, 2022 · 0 comments
Labels
📑 chore 오타 수정, 빌드 업무 수정, 패키지 매니저 수정 등 잡다한 수정 및 추가사항 🤝 FE needs 해당 이슈를 해결하기 위해 프론트엔드 팀과 협업해야 함 👀 good first issue Good for newcomers

Comments

@gusah009
Copy link
Member

사전 수행 issue

없음.

이슈 내용

@GetMapping(value = "/attach/{pid}")
public ListResult<FileEntity> getAttachList(@PathVariable("pid") Long postingId) {

  return responseService.getSuccessListResult(
      fileService.findAllByPostingId(postingService.getPostingById(postingId)));
}

위 코드는 PostingController에서 특정 포스팅의 첨부파일 목록을 모두 가져오는 코드입니다.

하지만 Entity가 클라이언트에게 곧바로 노출되어 있어 API 유지보수시에 문제가 발생할 가능성이 높습니다.

따라서 FileEntity 대신 관련 정보를 담은 Dto를 생성해서 반환해야 할 것 같습니다.

  • FileEntity 대신 관련 정보를 담은 Dto를 반환하도록 변경
    • ❗️ 해당 작업 수행 과정에서 응답 데이터 형식이나 url이 변경된다면 반드시 프론트엔드에 알리고 함께 수정할 것!!
@gusah009 gusah009 added 📑 chore 오타 수정, 빌드 업무 수정, 패키지 매니저 수정 등 잡다한 수정 및 추가사항 👀 good first issue Good for newcomers 🤝 FE needs 해당 이슈를 해결하기 위해 프론트엔드 팀과 협업해야 함 labels Sep 20, 2022
@gusah009 gusah009 changed the title PostingController의 getAttachList() 메서드가 Entity를 반환하는 문제 PostingController의 getAttachList() 메서드가 Entity를 반환하는 문제를 해결한다. Sep 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
📑 chore 오타 수정, 빌드 업무 수정, 패키지 매니저 수정 등 잡다한 수정 및 추가사항 🤝 FE needs 해당 이슈를 해결하기 위해 프론트엔드 팀과 협업해야 함 👀 good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

1 participant