Dashboard

Le correctif est une pull request. Polylane l'écrit, tu la fusionnes.

Rejoindre la liste d'attente
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

Un correctif que tu n'as pas eu à écrire. Et des raisons de lui faire confiance.

  • Écrit comme le ferait un coéquipier

    Dans un bac à sable, sur une branche, en suivant tes conventions de commit et les instructions de ton dépôt.

  • Vérifié avant d'être ouvert

    Une revue de sécurité traque les façons dont un diff correct casse quand même un deploy : migrations bloquantes, ruptures entre versions mixtes.

  • S'arrête à ta revue

    Ta revue et ta CI décident de la fusion, point final.

  • Il apprend d'un non

    Ferme la pull request sans la fusionner et Polylane comprend pourquoi, et ne recommence pas.

Ta revue est le bouton de fusion. Polylane ne fusionne jamais de lui-même.