Skip to content

롤백 처리 시, 이미지 삭제#183

Draft
vanillacake369 wants to merge 4 commits intomainfrom
feat/delete-image-if-rollback
Draft

롤백 처리 시, 이미지 삭제#183
vanillacake369 wants to merge 4 commits intomainfrom
feat/delete-image-if-rollback

Conversation

@vanillacake369
Copy link
Copy Markdown
Collaborator

@vanillacake369 vanillacake369 commented Jan 20, 2024

개요

롤백 처리 시, 이미지 삭제함수를 함수형으로 구현

작업사항

아직 S3 이미지 삭제 처리방안이 떠오르지 않아 도와주시면 감사하겠습니다 ㅎㅎ

  • 트랜잭션 롤백 처리 시, 이미지 삭제 요청 처리를 클래스로 분리
  • 함수형 인터페이스를 통해 구현
  • 예외케이스에 트랜잭션 예외처리 추가

의문점 및 궁금한 사항들

  1. 이미지 저장 시, 키값에 파일명과 UUID.random() 값을 저장하여 이미지 삭제를 어떻게 처리해야할지 모르겠어요

    포스트를 보아도 방법이 각자 다 달라서,,

  2. 기프티콘 예외처리를 2000번대에서 상품과 같이 하고 있는데 같이 이야기해야할 필요가 있다고 생각해요
  3. 현재 구현안 vs 아래 링크 방식 :: 어느 것이 더욱 효율적인가?? 이야기 해보아요

    https://velog.io/@sontulip/s3-rollback-problem
    제가 짠 거라,,, 당연한 소리지만,,,저는 지금 방식도 나쁘지만은 않은 것 같아요

관련 이슈

- USER_ADD_POINT_FAIL
- GIFTICON_NOT_FOUND
- GIFTICON_SAVE_FAIL
- SELL_BID_DELETE_FAIL
- SELL_BID_SAVE_FAIL
- NOTE : 아직 image 삭제 기능이 구현되어있지 않아 주석처리 되어있음
@vanillacake369 vanillacake369 added the improvement 성능 개선사항입니다. label Jan 20, 2024
@vanillacake369 vanillacake369 self-assigned this Jan 20, 2024
@vanillacake369 vanillacake369 changed the title Feat/delete image if rollback 롤백 처리 시, 이미지 삭제 Jan 20, 2024
@github-actions
Copy link
Copy Markdown

Test Results

56 tests   46 ✅  4s ⏱️
27 suites  10 💤
27 files     0 ❌

Results for commit 8d502a7.

@vanillacake369 vanillacake369 marked this pull request as ready for review January 20, 2024 20:18
@vanillacake369 vanillacake369 marked this pull request as draft January 30, 2024 03:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

improvement 성능 개선사항입니다.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

롤백 처리 시, 이미지 삭제

2 participants