[feat] 멱등성키 적용을 통한 중복 결제 문제 해결 #153
Closed
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
* [feat] Redis 환경설정 파일 생성 - port 와 host를 받아 connection을 생성 - redisTemplate에 직렬화 도구로 key : string, value = GenericJackson2JsonRedisSerializer 로 설정 * [fix] Cart의 생성자를 protected에서 public으로 변경 * [fix] CaartService에서 customerId의 cart를 가져오는 로직을 supplier로 변경 * [feat] Redis전용 CartEntity와 CartItemEntity를 생성 * [feat] Redis를 이용해서 Cart를 영속하는 레포지토리 구현 * [test] Redis를 이용해서 Cart를 영속하는 레포지토리 테스트 * [fix] ContainerSettingTest 삭제 여러개의 test-container를 구동시키면 포트 충돌로 일단 삭제 * [fix] testcode 가 돌아갈 수 있도록 환경변수 추가, inmemory cart repository를 primary로 수정
* [fix] 카트 도메인 엔티티 수정 * [feat] 카트 영속성 엔티티 구현 * [feat] DB를 사용하는 CartRepository 구현체 * [fix] CartService 오류 수정 Transactional 적용 orElse -> orElseGet : 데이터가 존재함에도 불구하고 저장되는 오류 수정 * [fix] 도메인 엔티티로 변환시 id값도 추가 * [refactor] 변수명 변경 * [fix] 생성자 접근제어자 protected * [test] JpaCartRepository 테스트 코드 작성
* [feat] 조회하고자 하는 카트가 없을 때 발생시킬 예외 구현 * [feat] 구매자의 카트를 조회하는 Dao * [feat] 카트 조회 앤드포인트 * [feat] 카트를 조회할 수 있는 Dao를 Redis를 이용해서 구현 (#135) * [test] menu 더미를 저장하고 반환하는 메서드 추가 및 상점을 24시간 열려있도록 수정 * [feat] getter 및 생성자 추가 * [feat] redis용 dao를 임시로 inMemoryCartRepository를 이용해서 구현 * [test] RedisCartDao 테스트코드 추가 * [feat] CartItemInfo에도 getter 추가 * [fix] 테스트 이후 데이터를 롤백시키기 위해 @transactional 추가 * [fix] CartResponse의 amount를 Integer로 수정 * [feat] Jpa를 이용한 카트 조회 dao --------- Co-authored-by: 김현욱 <[email protected]>
String -> UUID
Co-authored-by: Dr-KoKo <[email protected]>
Co-authored-by: Dr-KoKo <[email protected]>
Co-authored-by: Dr-KoKo <[email protected]>
Co-authored-by: Dr-KoKo <[email protected]>
Co-authored-by: Dr-KoKo <[email protected]>
@joincolumn만 이용하면 CartEntity를 통해CartItemEntity를 영속화시키지 못한다.
테스트용 yml 수정
fetchOne()의 결과로 리스트가 반환되어 오류가 발생
* [feat] 카트 조회시 X-lock 적용 * [test] TransactionTemplate를 통해서 Cart를 조회
* [feat] cartRepository의 findByCustomerId에 분산락을 적용한 버전 임시 * [feat] 분산락을 잡지 못했을 경우에 던지는 exception을 커스텀해서 던질 수 있도록 수정. * [fix] wait time 을 0초로 변경 - 중복 결제 요청들이 락을 다시 획득하기 위해서 기다릴 필요가 없음 --------- Co-authored-by: donghar <[email protected]>
This reverts commit a2b0da5.
- 기존 테스트환경에서 redis connection 오류를 수정한 main 브랜치를 머지하며 생긴 conflict 해결
Co-authored-by: Dr-KoKo <[email protected]>
Co-authored-by: Dr-KoKo <[email protected]>
Co-authored-by: Dr-KoKo <[email protected]>
Co-authored-by: Dr-KoKo <[email protected]>
Co-authored-by: Dr-KoKo <[email protected]>
Co-authored-by: Dr-KoKo <[email protected]>
… 및 적용 Co-authored-by: Dr-KoKo <[email protected]>
Co-authored-by: Dr-KoKo <[email protected]>
Co-authored-by: Dr-KoKo <[email protected]>
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.
💡 다음 이슈를 해결했어요.
Issue Link - #118
💡 이슈를 처리하면서 추가된 코드가 있어요.
💡 이런 고민을 했어요.
💡 다음 자료를 참고하면 좋아요.
✅ 셀프 체크리스트