Skip to content

fix: 결제 검증의 중복 외부 호출 제거 - #421

Merged
soyeon1806 merged 1 commit into
devfrom
fix/payment-remove-double-verify
Jul 15, 2026
Merged

fix: 결제 검증의 중복 외부 호출 제거#421
soyeon1806 merged 1 commit into
devfrom
fix/payment-remove-double-verify

Conversation

@soyeon1806

@soyeon1806 soyeon1806 commented Jul 15, 2026

Copy link
Copy Markdown
Collaborator

✅ PR 설명

기존엔 verifyPayment가 Toss confirm으로 승인한 뒤, 같은 결제를 다시 GET /payments/{key}로 조회해 status와 금액을 재확인하는 이중 호출 구조였습니다.
그러나 confirm 응답 자체가 승인 사실의 신뢰 원천이므로 재조회는 지연을 2배로 만들고, 두 호출 사이 상태 변화와 두 번째 호출 실패로 승인됐으나 미완결 될 수 있는 위험만 키웁니다.
이에 confirm 응답의 totalAmount로 금액을 그 자리에서 검증하고 두 번째 GET 호출을 제거해서 승인 검증을 외부 왕복 1회로 단순화했습니다.

🏗 작업 내용

  • verifyPayment의 confirm 응답에서 totalAmount를 읽어 저장 금액과 대조(불일치 시 PAYMENT_AMOUNT_MISMATCH)
  • TossPaymentClient.verifyPayment 메서드 삭제
  • 삭제된 메서드와 관련된 단위 테스트 5개 제거
  • verifyPayment 해피패스 및 금액불일치 테스트 케이스 추가

📸 테스트 결과 (선택)

  • PaymentServiceTest 25/25
  • TossPaymentClientTest 2/2

🔗 관련 이슈 (선택)

관련된 Issue가 있다면 #이슈번호 형식으로 작성해주세요.

🚨 참고 사항 (선택)

리뷰어가 참고해야 할 사항이 있다면 작성해주세요.

@soyeon1806 soyeon1806 self-assigned this Jul 15, 2026
@soyeon1806
soyeon1806 merged commit 2222694 into dev Jul 15, 2026
2 checks passed
@soyeon1806
soyeon1806 deleted the fix/payment-remove-double-verify branch July 15, 2026 14:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant