Skip to content

Feat : (Damain) 이미지(파일) 업로드 Aws Multipart Upload + Pre-Signed URL 추가#80

Merged
jay91537 merged 2 commits intodevelopfrom
54-refactor-global-이미지-업로드-로직-리팩토링
Jun 11, 2025

Hidden character warning

The head ref may contain hidden characters: "54-refactor-global-\uc774\ubbf8\uc9c0-\uc5c5\ub85c\ub4dc-\ub85c\uc9c1-\ub9ac\ud329\ud1a0\ub9c1"
Merged

Feat : (Damain) 이미지(파일) 업로드 Aws Multipart Upload + Pre-Signed URL 추가#80
jay91537 merged 2 commits intodevelopfrom
54-refactor-global-이미지-업로드-로직-리팩토링

Conversation

@jay91537
Copy link
Contributor

@jay91537 jay91537 commented Jun 7, 2025

🌱관련 이슈

📌작업 내용 및 특이사항

  • 이미지와 대용량 파일을 효율적으로 업로드 하기 위해 aws Multipart Upload + PreSigned URL 방식의 업로드 추가
  • 기존 Stream업로드 방식과 함께 사용
  • image테이블에 is_s3_upload 칼럼과 s3_upload_id 칼럼 추가

📝참고사항

📚기타

@jay91537 jay91537 requested a review from uoo6uoow June 7, 2025 06:23
@jay91537 jay91537 self-assigned this Jun 7, 2025
@jay91537 jay91537 linked an issue Jun 7, 2025 that may be closed by this pull request
2 tasks
InitiateMultipartUploadResult initiateResult = amazonS3.initiateMultipartUpload(
new InitiateMultipartUploadRequest(bucket, key)
.withObjectMetadata(new ObjectMetadata())
);
Copy link
Contributor

Choose a reason for hiding this comment

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

이런 부분에서는 따로 예외, 에러 처리를 해주지 않아도 되나요?

return new MultipartPreSignedUrlResponseDTO(uploadId, key, imageUrl, preSignedUrls);
} catch (Exception e) {
log.error("Multipart Pre-Signed URL 생성 중 에러 발생", e);
throw new RuntimeException("Multipart Pre-Signed URL 생성 중 에러 발생");
Copy link
Contributor

Choose a reason for hiding this comment

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

이런 에러 같은 경우는 기존에 만들어둔 에러 response에 들어가지 않는 종류인가요?

});
} catch (Exception e) {
log.error("Multipart Upload 완료 처리 중 에러 발생", e);
throw new RuntimeException("Multipart Upload 완료 처리 중 에러 발생");
Copy link
Contributor

Choose a reason for hiding this comment

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

여기도요

@jay91537 jay91537 merged commit 7c782d4 into develop Jun 11, 2025
2 checks passed
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] (Global) 이미지 업로드 로직 리팩토링

2 participants