แดชบอร์ด

การแก้ไขคือ pull request Polylane เขียน คุณ merge

ลงชื่อเข้า waitlist
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

การแก้ไขที่คุณไม่ต้องเขียนเอง และเหตุผลที่จะไว้ใจมัน

  • เขียนเหมือนเพื่อนร่วมทีมเขียน

    ใน sandbox บน branch ตามธรรมเนียม commit ของคุณและคำแนะนำของ repo ของคุณ

  • ตรวจก่อนเปิด

    การตรวจสอบความปลอดภัยไล่หาทางที่ diff ซึ่งถูกต้องยังทำให้ deploy พังได้: migration ที่ล็อกตาราง ความไม่เข้ากันระหว่างเวอร์ชันที่รันปนกัน

  • หยุดที่การรีวิวของคุณ

    การรีวิวและ CI ของคุณเป็นผู้ตัดสินการ merge จบ

  • มันเรียนรู้จากคำว่าไม่

    ปิด pull request โดยไม่ merge แล้ว Polylane จะหาว่าทำไม และไม่ทำซ้ำ

การรีวิวของคุณคือปุ่ม merge Polylane ไม่เคย merge ด้วยตัวเอง

ลงชื่อเข้า waitlist