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

Refactor/#44/스페이스 생성 api 에서 실제 이미지 데이터 저장 #47

Conversation

seongjunnoh
Copy link
Collaborator

📝 요약

기존 스페이스 생성 api는 썸네일 이미지의 url을 임의의 문자열로 지정해 db에 저장하였는데,
프론트단에서 직접 이미지 파일을 전달받아 aws s3에 저장하고 return 받은 url을 db에 저장하도록 코드를 수정하였습니다.

이슈 번호 : #44

🔖 변경 사항

  • json 형태가 아니라 form 데이터 형식으로 request dto를 전달받으므로, controller단에서 dto를 전달받을때 @ModelAttribute 어노테이션을 이용하도록 코드를 수정하였습니다.
  • request dto로 받을 MultipartFile 타입 이미지 파일의 validation을 위해 ConstraintValidator 를 구현한 ValidFileValidator 를 추가하고, 이를 이용하기 위해 @ValidFile 어노테이션을 생성하였습니다.

✅ 리뷰 요구사항

📸 확인 방법 (선택)

image postman으로 폼 데이터 형식의 request body를 구성하여 요청을 날리면, 응답으로 url 주소를 반환하도록 하였습니다. 이를 클릭하시면 요청보낼시에 입력한 이미지 파일을 확인할 수 있습니다.

📌 PR 진행 시 이러한 점들을 참고해 주세요

* P1 : 꼭 반영해 주세요 (Request Changes) - 이슈가 발생하거나 취약점이 발견되는 케이스 등
* P2 : 반영을 적극적으로 고려해 주시면 좋을 것 같아요 (Comment)
* P3 : 이런 방법도 있을 것 같아요~ 등의 사소한 의견입니다 (Chore)

@seongjunnoh seongjunnoh linked an issue Jul 27, 2024 that may be closed by this pull request
3 tasks
Copy link
Collaborator

@drbug2000 drbug2000 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@seongjunnoh seongjunnoh merged commit 7c02a32 into develop Jul 28, 2024
3 checks passed
@hyunn522 hyunn522 deleted the refactor/#44/스페이스-생성-api-에서-실제-이미지-데이터-저장 branch July 30, 2024 12:27
seongjunnoh added a commit that referenced this pull request Oct 30, 2024
…-이미지-데이터-저장

Refactor/#44/스페이스 생성 api 에서 실제 이미지 데이터 저장
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Refactor : 스페이스 생성 api 에서 실제 이미지 데이터 저장
2 participants