[feat] 결제시 동일한 카트에 대한 중복 결제 문제 #150

Dr-KoKo · 2024-08-26T13:22:06Z

💡 다음 이슈를 해결했어요.

Issue Link - #118

분산락을 이용한 중복 주문/결제 방지

사용자는 분산락 취득을 시도합니다.
락 취득 성공 여부에 따라 주문/결제가 수행됩니다.
2-1) 성공하면 로직 수행
2-2) 실패하면 DuplicateOrderException 발생

배타적 락과 비교

주문/결제는 카트를 조회하고 2) 카트에 담긴 상품을 토대로 주문/결제를 진행합니다. 따라서 Cart에 배타락을 걸어서 중복 주문/결제를 막을 수 있습니다.

주문/결제 요청은 DB Connection을 물고 block 됩니다. 그로 인해 DB Connection의 소진이 발생할 가능성이 높습니다.
주문/결제는 단 한번만 수행되어야 합니다. 후순위 요청은 경합 발생시 실패 요청을 (대기하지 않고) 실패 응답을 해주는 것이 효율적입니다.

따라서 1) DB Connection의 소진 염려, 2) 느린 응답 시간으로 인해 분산락이 더 효율적이라고 판단하였습니다.

낙관적 락과 비교

카트에 낙관적 락을 걸 수 있습니다. 낙관적 락은 조회된 카트정보의 버전을 관리하며 충돌을 감지합니다.

충돌이 감지되면 그 동안의 작업 내용을 rollback 해야 합니다. 주문/결제는 5개의 쓰기가 필요하기 때문에 충돌시 발생하는 비용이 큽니다.
낙관적 락은 어찌 되었든 임계 영역의 코드를 수행하게 됩니다. 카트에 상품을 담는 요청과의 경합에서 원치 않는 결제가 수행될 염려가 있습니다.

따라서 1) 충돌로 인한 지나친 비용, 2) 임계 영역의 코드 수행으로 인한 위험성으로 인해 분산락이 더 효율적이라고 판단하였습니다.

Redis를 이용한 Cart 저장 및 조회

Scale-out을 염두하여 서버의 메모리에 저장하던 카트 정보를 Redis에 저장하도록 변경하였습니다.

💡 이슈를 처리하면서 추가된 코드가 있어요.

(없다면 이 문항을 지워주세요.)

💡 다음 자료를 참고하면 좋아요.

NamedLock

✅ 셀프 체크리스트

내 코드를 스스로 검토했습니다.
필요한 테스트를 추가했습니다.
모든 테스트를 통과합니다.
브랜치 전략에 맞는 브랜치에 PR을 올리고 있습니다.
커밋 메세지를 컨벤션에 맞추었습니다.
wiki를 수정했습니다.

* [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 테스트 코드 작성

@transactional

* [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]>

@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 해결

kimhyun5u

수고하셨습니다!

june-777

👍👍

…118_Dr-KoKo_Hyeon-Uk_고도화_결제시_동일한_카트에_대한_중복_결제_문제 # Conflicts: # build.gradle # src/main/java/camp/woowak/lab/infra/aop/DistributedLock.java # src/main/java/camp/woowak/lab/infra/aop/DistributedLockAop.java # src/main/resources/application.yaml # src/test/java/camp/woowak/lab/web/dao/store/StoreDummiesFixture.java # src/test/resources/application.yml

두 개로 분리되어 있던 생성자를 하나로 통일

디폴트 예외 설정의 누락으로 인한 컴파일 오류 해결

Hyeon-Uk and others added 22 commits August 22, 2024 19:42

[feat] CartEntity의 CustomerId 변경

d9c9816

String -> UUID

[fix] CartDao 이중 등록 제거

32ecb91

[fix] CartItemEntity에 id가 null로 갱신되는 문제 해결

bc161c8

Co-authored-by: Dr-KoKo <[email protected]>

[fix] 의존 주입할 Bean을 application.yml에서 관리할 수 있도록 수정

f393d9c

Co-authored-by: Dr-KoKo <[email protected]>

[feat] 중복 주문에 대한 Exception 생성 및 error code 갱신

24d8ccc

Co-authored-by: Dr-KoKo <[email protected]>

[test] 중복 주문 동시성 테스트에 사용할 더미데이터 만드는 메서드 추가

833fa6c

Co-authored-by: Dr-KoKo <[email protected]>

[test] 중복 주문 방지에 대한 동시성 테스트 시나리오 추가

e534e18

Co-authored-by: Dr-KoKo <[email protected]>

[refactor] 검증 부분 주석 추가

14edb4c

[fix] OneToMany 관계에서 cascade를 사용하기 위해 mappedBy 사용

8d70ebf

@joincolumn만 이용하면 CartEntity를 통해CartItemEntity를 영속화시키지 못한다.

[feat] 마이크로미터 앤드포인트 노출

04bb3a0

[feat] 마이크로미터 앤드포인트 노출

898034c

[test] 마이크로미터 앤드포인트 노출

a308456

테스트용 yml 수정

[fix] APIResponseAdvice에 @EnableConfigurationProperties 추가

9d998a5

[fix] JpaCartDao 오류 수정

93e994d

fetchOne()의 결과로 리스트가 반환되어 오류가 발생

[feat] 주문 시점에 배타락을 적용해서 문제를 해결 (#148)

a2b0da5

* [feat] 카트 조회시 X-lock 적용 * [test] TransactionTemplate를 통해서 Cart를 조회

Revert "[feat] 주문 시점에 배타락을 적용해서 문제를 해결 (#148)"

fa6980e

This reverts commit a2b0da5.

[fix] CartRepository를 트랜잭션 내에서 테스트

21f03ce

[fix] cart 저장소를 redis로 변경

1cbb09a

This was linked to issues Aug 26, 2024

[고도화] 결제시 동일한 카트에 대한 중복 결제 문제 #118

Closed

[기능] 구매자는 카트를 조회할 수 있다 #127

Closed

Hyeon-Uk marked this pull request as ready for review August 27, 2024 04:17

Hyeon-Uk added 2 commits August 27, 2024 13:27

[fix] Redisson에 의존하는 configuration 이름을 redis로 변경

7247757

[feat] conflict 해결

ef4694a

- 기존 테스트환경에서 redis connection 오류를 수정한 main 브랜치를 머지하며 생긴 conflict 해결

Hyeon-Uk mentioned this pull request Aug 28, 2024

[feat] 멱등성키 적용을 통한 중복 결제 문제 해결 #153

Closed

6 tasks

kimhyun5u approved these changes Aug 28, 2024

View reviewed changes

june-777 approved these changes Aug 28, 2024

View reviewed changes

Dr-KoKo added 6 commits August 28, 2024 23:42

[fix] 생성자 통일

d83b684

두 개로 분리되어 있던 생성자를 하나로 통일

[fix] redis 설정 통합

73bf5ef

[fix] 컴파일 오류 해결

7fba4a7

디폴트 예외 설정의 누락으로 인한 컴파일 오류 해결

[fix] 테스트시 in_memory 버전의 CartRepository를 사용하게 설정

12367ad

[refactor] build.gradle 정리

b975470

Dr-KoKo merged commit a9dd1c3 into main Aug 29, 2024
1 check passed

Hyeon-Uk mentioned this pull request Aug 29, 2024

[feat] 멱등성키 적용을 통한 중복결제 방지 #155

Merged

6 tasks

kimhyun5u deleted the feature/118_Dr-KoKo_Hyeon-Uk_고도화_결제시_동일한_카트에_대한_중복_결제_문제 branch September 19, 2024 09:30

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[feat] 결제시 동일한 카트에 대한 중복 결제 문제 #150

[feat] 결제시 동일한 카트에 대한 중복 결제 문제 #150

Dr-KoKo commented Aug 26, 2024 •

edited by Hyeon-Uk

Loading

kimhyun5u left a comment

june-777 left a comment

[feat] 결제시 동일한 카트에 대한 중복 결제 문제 #150

[feat] 결제시 동일한 카트에 대한 중복 결제 문제 #150

Conversation

Dr-KoKo commented Aug 26, 2024 • edited by Hyeon-Uk Loading

💡 다음 이슈를 해결했어요.

Issue Link - #118

분산락을 이용한 중복 주문/결제 방지

배타적 락과 비교

낙관적 락과 비교

Redis를 이용한 Cart 저장 및 조회

💡 이슈를 처리하면서 추가된 코드가 있어요.

💡 다음 자료를 참고하면 좋아요.

✅ 셀프 체크리스트

kimhyun5u left a comment

Choose a reason for hiding this comment

june-777 left a comment

Choose a reason for hiding this comment

Dr-KoKo commented Aug 26, 2024 •

edited by Hyeon-Uk

Loading