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

[BE/#84] S3 API 구현 #89

Merged
merged 8 commits into from
Nov 21, 2023
Merged

[BE/#84] S3 API 구현 #89

merged 8 commits into from
Nov 21, 2023

Conversation

koomin1227
Copy link
Member

@koomin1227 koomin1227 commented Nov 21, 2023

이슈

체크리스트

  • 파일 업로드 구현
  • 파일 삭제 구현

고민한 내용

구현 방식

FileFieldsInterceptor를 s3-multer를 사용하여 구현하는 방법과 s3 처리를 담당하는 클래스를 만들어 활용하는 방법중 고민했다.

  • FileFieldsInterceptor를 사용하면 요청이 컨트롤러에 넘어오기전에 파일 처리를 다 끝내고 코드가 깔끔해지는 장점이 있었지만 처리 속도가 느려 응답받는 시간이 오래걸리는 문제가 있었다. 또한 용량이 큰 파일이 오면 요청시간과 서버 시간이 달라 에러가 나는 문제가 발생했다.
  • s3 처리를 담당하는 클래스를 사용하면 처리 속도도 빨라지고 추후 이미지에 대한 추가적인 처리 로직을 추가 할 때 보다 용이해질 것 같아 이 방법을 사용했다.

스크린샷

@koomin1227 koomin1227 added BE BE 관련 environment 공통 개발 환경 설정 labels Nov 21, 2023
@koomin1227 koomin1227 self-assigned this Nov 21, 2023
ACL: 'public-read',
Body: file.buffer,
});
await this.s3.send(command);
Copy link
Collaborator

Choose a reason for hiding this comment

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

에러 처리가 추가적으로 필요해 보입니다

@koomin1227 koomin1227 merged commit 2666286 into BE Nov 21, 2023
1 check passed
@koomin1227 koomin1227 deleted the BE-S3API-#84 branch November 21, 2023 14:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BE BE 관련 environment 공통 개발 환경 설정
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants