Le correctif est une pull request. Polylane l'écrit, tu la fusionnes.
Rejoindre la liste d'attenteCap retries on the checkout webhook worker #491
polylanemain from polylane/autofix/chat/k3x9f2-4e7d21a 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.
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.