Skip to content

Conversation

@SinJeongEun
Copy link
Collaborator

@SinJeongEun SinJeongEun commented Dec 1, 2025

  1. Map<PgProvider, Map<PayMethod, PgStrategy>> pgStrategies
    에서 target PG사 + 결제유형 서비스 빈을 추출한다.
  2. PgStrategy 는 승인/취소프로세스.. 결제를 위한 프로세스를 담당하고, pg사별 구현하게 된다.
  3. 각 pg사마다 OOPgStrategy를 생성하고 (2)번을 구현한다.
    • 해당 pg사 연동방식을 분석하고, pg사 내부적으로 결제유형에 따라 달라지는 구조는 각 OO[Card/Phone]Service서비스에서 구체화 하도록 함.
  • 토스 : 결제유형별(카드,간편,가상계좌) 요청 동일 / 결제유형별 응답 파싱 필요
  • nhn : 카드,간편결제 요청 동일 / 결제유형별 응답 개별 파싱 필요

1. db 에서는 카드사별 프로모션 데이터를 json으로 관리한다.
2. CardBinPromotion 매핑 시 payProvier 에 따라 해당카드사 프로모션 객체로 변환한다.
fix: pg수수료 redis zset 캐싱 -> 로컬캐싱
pg health check 는 redis 유지
애플리케이션 실행 전 로컬케시 초기화 검증
scheduling refreshPgCache 매일 0시
토스 : 결제유형별(카드,간편,가상계좌) 요청/응답 호출 동일, 파라미터 구조는 동일하지만 결제유형에 따라 nullable
nhn : 카드,간편결제  요청/응답 호출 동일하다.  (구현중)
@SinJeongEun SinJeongEun self-assigned this Dec 1, 2025
@SinJeongEun SinJeongEun changed the title [#29] TOSS, NHN 결제요청 구체화 [#29] TOSS 구체화 TossStrategy 에서 통합처리 Dec 1, 2025
SinJeongEun and others added 9 commits December 6, 2025 16:54
[#26]feature: 카드사별 프로모션 관리
[#23]  pg수수료  ApplicationStartedEvent 로컬캐싱 전환
토스 : 결제유형별(카드,간편,가상계좌) 요청/응답 호출 동일, 파라미터 구조는 동일하지만 결제유형에 따라 nullable
nhn : 카드,간편결제  요청/응답 호출 동일하다.  (구현중)
# Conflicts:
#	platform/src/main/java/com/commerce/platform/core/application/in/PaymentUseCaseImpl.java
#	platform/src/main/java/com/commerce/platform/core/application/out/PgStrategy.java
#	platform/src/main/java/com/commerce/platform/infrastructure/adaptor/PgCacheService.java
#	platform/src/main/java/com/commerce/platform/infrastructure/pg/DanalStrategy.java
#	platform/src/main/java/com/commerce/platform/infrastructure/pg/nhn/NHNStrategy.java
#	platform/src/main/java/com/commerce/platform/infrastructure/pg/toss/TossStrategy.java
#	platform/src/test/java/com/commerce/platform/core/application/in/PaymentUseCaseImplTest.java
#	platform/src/test/java/com/commerce/platform/core/domain/service/PaymentPgRouterTest.java
TossPgStrategy .. : PgStrategy 메소드 구현 및 해당 pg사의 특성에 맞게 내부적으로 결제유형별 추상화
@SinJeongEun SinJeongEun merged commit 3d65d0a into main Jan 4, 2026
1 check passed
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.

2 participants