수정은 풀 리퀘스트입니다. Polylane이 작성하고, 여러분이 병합합니다.
대기자 명단 등록 github.com/coreplane/payments-api/pull/491
Cap retries on the checkout webhook worker #491
polylane Open polylane wants to merge 1 commit into
main from polylane/autofix/chat/k3x9f2-4e7d21a Conversation 1 Commits 1 Checks 1 Files changed 2
polylane bot commented 6 minutes ago ···
Retries on the checkout webhook worker were unbounded: a failing delivery re-queued itself forever and amplified load on payments-api. This caps delivery at 5 attempts with exponential backoff and dead-letters the payload after the last one.
What changed
worker/deliver.ts gains MAX_DELIVERY_ATTEMPTS = 5 and backoff between attempts; exhausted payloads land in checkout-webhooks-dlq instead of re-queueing.
Validation
npm test — 214 passed. A forced failing delivery stopped after 5 attempts and appeared in the dead-letter queue.
Root cause · Why it's safe · Out of scope
polylane added commit 4e7d21a Verified
Review required At least 1 approving review is required
ci / test Successful in 3m 12s Details
Review required Waiting on your review: Polylane never merges on its own
Merging is blocked
직접 쓰지 않아도 되는 수정. 그리고 그것을 믿을 이유.
-
동료가 쓰듯 작성
샌드박스에서, 브랜치 위에서, 여러분의 커밋 규칙과 리포지토리 지침을 따릅니다.
-
열기 전에 검사
안전 리뷰가 올바른 diff라도 배포를 망가뜨릴 수 있는 방식을 찾아냅니다. 잠금을 거는 마이그레이션, 버전 혼재로 인한 장애.
-
여러분의 리뷰에서 멈춥니다
병합은 여러분의 리뷰와 CI가 결정합니다. 그것으로 끝입니다.
-
거절에서 배웁니다
풀 리퀘스트를 병합하지 않고 닫으면 Polylane이 그 이유를 파악하고, 같은 일을 반복하지 않습니다.