#193 [feat] 회원 탈퇴 시 이미지 삭제 로직 추가 #196
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
hellozo0
approved these changes
Jan 26, 2024
| //Designer designer = designerJpaRepository.findById(userId).orElseThrow(() -> new NotFoundException(DESIGNER_NOT_FOUND_EXCEPTION)); | ||
| String s3Key = offerImageUrl.substring(54); | ||
| String preSignedUrl = s3Service.getPreSignedUrlToDownload(s3Key); | ||
| Designer designer = designerJpaRepository.findById(userId).orElseThrow(() -> new NotFoundException(DESIGNER_NOT_FOUND_EXCEPTION)); |
There was a problem hiding this comment.
p5 :
이친구는 이전에는 주석이 되었다가 살아났는데 designer 를 해당 함수에서 사용하지 않는 것으로 보이는데 어디서 사용하는 것일까요.>>>???!!
There was a problem hiding this comment.
@hellozo0
해당 서비스 로직은 디자이너가 사용하는 서비스 로직이라서 비즈니스 로직 시작 전에 해당 메서드를 사용하는 사람이 디자이너인지 아닌지 구분하는 용도로만 사용합니다...!
Comment on lines
+78
to
+79
| private String getImageUrlToKey(final String imageUrl) { | ||
| return imageUrl.substring(IMAGE_URL_PREFIX_LENGTH); |
There was a problem hiding this comment.
p5 :
이전 코드에서는 imagekey(?)값으로 이미지를 가져오지 않았는데 어떤 이유로 사용되었는지 알 수 있을까요..?!?!
There was a problem hiding this comment.
@hellozo0
이미지 url이 https://~~~/bucket/path 이런 식일 때
이미지를 삭제할 때 url 전체를 넣어서 삭제하는 것이 아닌, 앞에 있는 https://~~~ 를 제거해서 bucket/path 값을 넣어야 합니다...!
그래서 substring 로직을 추가했습니다!
pkl0912
approved these changes
Jan 26, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
관련 이슈번호
해결하는 데 얼마나 걸렸나요? (예상 작업 시간 / 실제 작업 시간)
해결하려는 문제가 무엇인가요?
어떻게 해결했나요?
유저 탈퇴시
디자이너 탈퇴 시
로직 수정 사항
그래서 substring 하는 로직이 delete 할 때도 중복되어서 해당 로직을 s3 service 내로 이동했습니다.