Feat/#464 이미지 s3 도입 #479

hectick · 2023-10-10T05:35:59Z

🔥 연관 이슈

close: S3로 이미지 관리하기 #464

📝 작업 요약

application의 profile이 prod일때는 s3를, 그 외(테스트 포함)는 local 기존 경로에 이미지를 저장하도록 했습니다.
s3 -> 이미지 업로드 구현, 이미지 삭제 미구현
local -> 이미지 업로드 구현, 이미지 삭제 구현(프로필용)

s3 이용시 사용자에게 전달되는 이미지 주소는 {클라우드 프론트 도메인}/images/post/KakaoTalk_Photo_2023-09-13-10-03-02.jpeg입니다.
현재 시각 기준 테스트 서버에 현재 s3 브랜치가 반영되어 있으니 확인해보셔도 조하요

다음 커밋이 s3 반영한 커밋이구요. 그 이전은 환경에따라 localimageclinet <-> s3imageclient를 바꿔 사용하기 위한 구조 리팩터링입니다

그리고 imageService에서 데이터베이스에 post 이미지, comment 이미지 repository에 저장하는 로직도 넣어보려다가
너무 힘들어보여서 기존 상태를 유지하였읍니다. (imageService의 todo는 무시하셔도 좋다는 이야기~)

추후 해야할 일

테스트서버 관련해서 해야하는 일

기존에 존재하는 이미지 파일 이주 -> 자동으로 하는 api 만들 예정
테스트 서버 배포 스크립트

운영서버 관련해서 해야하는 일

기존에 존재하는 이미지 파일 이주
운영서버용 클라우드 프론트 만들기
젠킨스 배포 스크립트 환경변수 수정(SERVER_TYPE, IMAGE_DOMAIN)

# 서버 접속해서 배포 파일의 Image domain 클라우드 프론트로 바꾸어 주어야 함.
image.domain=${IMAGE_DOMAIN}
# 다음 경로에 있는 이미지 수동 옮기기 필요
#file.dir=/home/ubuntu/2023-edonymyeon/resources/

…t를 이용하도록 리팩터링

…도록 변경

# Conflicts: # backend/src/main/resources/application-prod.properties

github-actions · 2023-10-10T05:39:56Z

Unit Test Results

  67 files   67 suites 30s ⏱️
357 tests 357 ✔️ 0 💤 0 ❌
360 runs 360 ✔️ 0 💤 0 ❌

Results for commit d7cda0b.

♻️ This comment has been updated with latest results.

BackFoxx

멋져요 이리내!
S3용 이미지 저장 객체와 로컬로의 이미지 저장 객체를 갈아끼우기 쉬운 구조로 만들어준 덕분에
코드들이 한 눈에 들어오고 엄청 깔끔하다는 느낌을 받았습니다.

S3에 저장된 사진을 삭제하게 되면 S3에서는 물리적인 삭제를 하고,
DB 상에서는 이렇게 deleted 표시로 논리적인 삭제가 되는 건가요?
내일 캠퍼스에서 좀 더 자세히 봐야할 것 같네요!

BackFoxx · 2023-10-10T08:38:17Z

backend/src/main/java/edonymyeon/backend/image/infrastructure/S3ImageClient.java

+    @Override
+    public void delete(final String imagePath) {
+        throw new BusinessLogicException(UNSUPPORTED_METHOD_CALL);
+    }


뭔가 상태패턴 보는 것 같고 좋네요 이리내! 👍🏻

This2sho

고생하셨습니다. 이리 저리 내🙇‍♂️

답글을 남겨주신다면 어프로브를 드리겠습니다.

backend/src/main/java/edonymyeon/backend/image/application/ImageService.java

This2sho · 2023-10-16T11:50:40Z

backend/src/main/java/edonymyeon/backend/image/application/LocalImageMigrationClient.java

+
+    @Override
+    public void migrate(final File image, final String directory, final String storeName) {
+        System.out.println("아무것도 안하지롱");


이리내 같네요 🚽

This2sho · 2023-10-16T11:53:53Z

backend/src/main/java/edonymyeon/backend/image/application/ImageMigrationService.java

+
+@RequiredArgsConstructor
+@Service
+public class ImageMigrationService {


이 서비스는 migration 끝나면 제거할건가유?

핫픽스 해야되지 않을까욤

This2sho · 2023-10-16T12:00:44Z

...c/main/java/edonymyeon/backend/image/commentimage/repository/CommentImageInfoRepository.java

@@ -12,4 +12,7 @@ public interface CommentImageInfoRepository extends JpaRepository<CommentImageIn
    @Modifying //todo: 옵션.. 이대로 괜찮은가?
    @Query("update CommentImageInfo c set c.deleted = true where c.id in :ids")
    void deleteAllById(@Param("ids") List<Long> ids);
+
+    @Query(value = "select * from comment_image_info", nativeQuery = true)


혹시 이거 findAll()을 안쓰고 native Query를 사용하는 이유 알려주실 수 있나염?

JPARepo를 사용하면 삭제된 이미지는 where절에서 걸려서 조회 안되서 긍듯여

CommentImageInfo 클래스 들어가보면 아래처럼럼 where절이 붙어있는데요. 이렇게하면 쿼리가 db에 난사될때 where절이 디폴트로 붙어서 나간다고 해요.

where절을 개무시하고 deleted=true인 이미지들(soft delete된 이미지들) 것들도 다 s3로 이동하기 위해서 native하게 해주엇슴니다
이것도 이미지 이주가 끝나면 삭제되겠죵..

오홍 그렇고만요 👍

jyeost

이리내상 ~~ 로컬에서는 잘 돌아가는 거 확인했습니다,,,
실 서버에서는.... 힘들어서...스루ㅜㅜ
깔끔하게 잘 구현해주셨습니다 !! 수고 많으셨어여!!!

jyeost · 2023-10-16T13:36:38Z

backend/src/main/java/edonymyeon/backend/image/application/ImageService.java

+
+    private final ImageFileNameStrategy imageFileNameStrategy;
+
+    private final Domain domain;


이미지 서비스가 생기면서 각각 서비스에서 사용하던
도메인을 한곳에서 관리하니 편하네여!!!!

jyeost · 2023-10-16T13:49:34Z

backend/src/main/java/edonymyeon/backend/post/application/PostReadService.java

@@ -128,7 +129,7 @@ private WriterDetailResponse getWriterResponse(final Member member) {
        return new WriterDetailResponse(
                member.getId(),
                member.getNickname(),
-                domain.getDomain() + member.getProfileImageInfo().getStoreName()
+                imageService.findBaseUrl(ImageType.POST) + member.getProfileImageInfo().getStoreName()


이미지 서비스에 이미지 타입과 파일 이름을 넘겨서 완성된 파일 경로를 가져오는 것은 어떤가요...??

넴 메서드를 하나 더 만들어보겟습니다

convert메서드 사용하면 안되는 건가염??

jyeost · 2023-10-16T14:40:02Z

...c/main/java/edonymyeon/backend/image/commentimage/repository/CommentImageInfoRepository.java

@@ -12,4 +12,7 @@ public interface CommentImageInfoRepository extends JpaRepository<CommentImageIn
    @Modifying //todo: 옵션.. 이대로 괜찮은가?
    @Query("update CommentImageInfo c set c.deleted = true where c.id in :ids")
    void deleteAllById(@Param("ids") List<Long> ids);
+
+    @Query(value = "select * from comment_image_info", nativeQuery = true)


JPARepo를 사용하면 삭제된 이미지는 where절에서 걸려서 조회 안되서 긍듯여

jyeost · 2023-10-16T14:41:45Z

backend/src/main/java/edonymyeon/backend/image/application/LocalImageClient.java

+        return true;
+    }
+
+    @Override


어케 구현할지 궁금했는데 @Profile을 이용해서 완전 깔쌈하게 구현해주셨군여 !!!!
감덩감덩 쓰나미쓰나미

jyeost · 2023-10-16T15:15:58Z

backend/src/main/java/edonymyeon/backend/image/domain/Domain.java

    private String domain;

-    public List<String> removeDomainFromUrl(final List<String> imageUrls) {
+    public List<String> removeDomainFromUrl(final List<String> imageUrls, final String typeDirectory) {


이 메서드 명이 뭔가... 어색한것 같아여...
밑에는 convertToImageUrl()인데 얘도 맞춰서 convertToFileName이나 convertToStoredName같은 걸로 바뀌면 좋을 것 같아여

그럼 이 밑에 메서드랑보다 상위 메서드인게 더 잘 보일 것 같아염

backend/src/main/java/edonymyeon/backend/image/application/ImageService.java

This2sho

굳굳 👍

# Conflicts: # backend/src/main/java/edonymyeon/backend/comment/application/CommentService.java # backend/src/main/java/edonymyeon/backend/global/exception/ExceptionInformation.java # backend/src/main/java/edonymyeon/backend/image/ImageFileUploader.java # backend/src/main/java/edonymyeon/backend/member/application/MemberService.java # backend/src/main/java/edonymyeon/backend/post/application/PostReadService.java # backend/src/test/java/edonymyeon/backend/auth/application/AuthServiceTest.java

# Conflicts: # backend/src/main/java/edonymyeon/backend/member/application/MemberService.java # backend/src/test/java/edonymyeon/backend/member/docs/MemberControllerDocsTest.java

hectick · 2023-10-18T12:54:25Z

마지막 리뷰 해주세염 @BackFoxx @jyeost @This2sho
케로씨 핫게시글 pr,호이씨 회원정보 수정pr이랑 꽤 충돌햇는데 유심히 살펴봐주세옴~
그리고 프로필 이미지 이주 로직도 추가햇습니다~

This2sho · 2023-10-18T13:09:14Z

backend/src/main/java/edonymyeon/backend/image/domain/ImageInfoFactory.java

+
+@RequiredArgsConstructor
+@Component
+public class ImageInfoFactory {


오홍 네이밍 좋구여👍

This2sho

버저닝 변경된거 까지 확인했습니다. 👍
고생하셨어유 이리 저리 이리이리내씨

hectick added 12 commits October 6, 2023 17:19

feat: 이미지를 저장할 수 있는 ImageService 및 ImageClient 생성

1d873ec

refactor: 로컬 환경에서 이미지를 저장, 삭제할 때 ImageFileUploader 대신 LocalImageClien…

5a0865b

…t를 이용하도록 리팩터링

refactor: 게시글, 댓글 조회시 이미지 url을 Domain을 통해서가 아니라 ImageService를 통해서 알아내…

cddb4c9

…도록 변경

refactor: properties 파일의 domain 변수명을 image.domain으로 구체화 및 테스트 코드 추가

2e7fa41

fix: 이미지 삭제 버그 수정

7b00a10

refactor: 메서드명 수정

8a2247c

feat: prod 환경에서 s3에 이미지를 저장하는 기능 구현

947af09

Merge remote-tracking branch 'origin/dev' into feat/#464_이미지_s3_도입

36efbbc

# Conflicts: # backend/src/main/resources/application-prod.properties

refactor: application-prod.properties 다듬기

970d320

fix: S3Client configuration

9fb67b8

fix: final 제거

2743167

fix: contentType 지정

da9f56f

hectick added the ❤️ Backend for BackEnd label Oct 10, 2023

hectick added this to the 6차 데모데이 milestone Oct 10, 2023

hectick self-assigned this Oct 10, 2023

hectick added ⚡️ feature New feature or request 🛠️ refactor labels Oct 10, 2023

refactor: 주석 제거

39a5b0f

BackFoxx requested review from BackFoxx, jyeost and This2sho October 10, 2023 05:54

BackFoxx approved these changes Oct 10, 2023

View reviewed changes

feat: 이미지 s3로 이주하는 api 생성

ea1f06b

This2sho reviewed Oct 16, 2023

View reviewed changes

jyeost approved these changes Oct 16, 2023

View reviewed changes

This2sho approved these changes Oct 17, 2023

View reviewed changes

hectick added 3 commits October 18, 2023 13:55

fix: 머지 후 애플리케이션 동작하도록 코드 수정

ebe422d

refactor:

55e18df

hectick added 8 commits October 18, 2023 20:24

Merge remote-tracking branch 'origin/dev' into feat/#464_이미지_s3_도입

9e46be3

# Conflicts: # backend/src/main/java/edonymyeon/backend/member/application/MemberService.java # backend/src/test/java/edonymyeon/backend/member/docs/MemberControllerDocsTest.java

refactor: 어노테이션 순서 변경

2e05fb6

docs: 회원정보수정 문서화 수정

e37c0fe

refactor: 변수명 변경

ed721e0

refactor: 캡슐화

bef925f

refactor: ImageInfoFactory 클래스 분리 및 image 관련 클래스들 패키지 변경

1880dea

refactor: 오래된 파일 경로 원상복구

de6231a

feat: 프로필 자동 삭제 기능 도입

b435108

This2sho reviewed Oct 18, 2023

View reviewed changes

This2sho approved these changes Oct 18, 2023

View reviewed changes

Merge remote-tracking branch 'origin/dev' into feat/#464_이미지_s3_도입

d7cda0b

hectick merged commit 65cecf8 into dev Oct 23, 2023

hectick deleted the feat/#464_이미지_s3_도입 branch October 23, 2023 07:31

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Feat/#464 이미지 s3 도입 #479

Feat/#464 이미지 s3 도입 #479

hectick commented Oct 10, 2023 •

edited

Loading

github-actions bot commented Oct 10, 2023 •

edited

Loading

BackFoxx left a comment

BackFoxx Oct 10, 2023

This2sho left a comment

This2sho Oct 16, 2023

hectick Oct 16, 2023

This2sho Oct 16, 2023

hectick Oct 16, 2023

This2sho Oct 16, 2023

jyeost Oct 16, 2023

hectick Oct 16, 2023 •

edited

Loading

This2sho Oct 17, 2023

jyeost left a comment

jyeost Oct 16, 2023

jyeost Oct 16, 2023

hectick Oct 17, 2023

jyeost Oct 17, 2023

jyeost Oct 16, 2023

jyeost Oct 16, 2023

jyeost Oct 16, 2023

This2sho left a comment

hectick commented Oct 18, 2023 •

edited

Loading

This2sho Oct 18, 2023

This2sho left a comment


		private final ImageFileNameStrategy imageFileNameStrategy;

		private final Domain domain;

Feat/#464 이미지 s3 도입 #479

Feat/#464 이미지 s3 도입 #479

Conversation

hectick commented Oct 10, 2023 • edited Loading

🔥 연관 이슈

📝 작업 요약

추후 해야할 일

github-actions bot commented Oct 10, 2023 • edited Loading

Unit Test Results

BackFoxx left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

This2sho left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

hectick Oct 16, 2023 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

jyeost left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

This2sho left a comment

Choose a reason for hiding this comment

hectick commented Oct 18, 2023 • edited Loading

Choose a reason for hiding this comment

This2sho left a comment

Choose a reason for hiding this comment

hectick commented Oct 10, 2023 •

edited

Loading

github-actions bot commented Oct 10, 2023 •

edited

Loading

hectick Oct 16, 2023 •

edited

Loading

hectick commented Oct 18, 2023 •

edited

Loading