A correção é uma pull request. O Polylane escreve, você faz o merge.
Entrar na lista de esperaCap 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.
Uma correção que você não precisou escrever. E motivos para confiar nela.
-
Escrita como um colega escreveria
Em um sandbox, em uma branch, seguindo suas convenções de commit e as instruções do seu repositório.
-
Conferida antes de abrir
Uma revisão de segurança caça as formas pelas quais um diff correto ainda quebra um deploy: migrações que travam, quebras entre versões misturadas.
-
Para na sua revisão
Sua revisão e seu CI decidem o merge, ponto final.
-
Aprende com um não
Feche a pull request sem merge e o Polylane descobre o motivo, e não repete.