Freegle / Iznik / 28965 / 28965

73%

master: 73%

Pull #1195

circleci

fix(retry): stop 504s triggering ten stacked retries of expensive requests

A 504 means the gateway gave up on a request the server was probably
still executing - usually an endpoint that is slow for everyone, not a
transient blip. The retry loop's ten attempts each stacked another full
copy of that work on the server (seen live on the sysadmin rippling
analytics tab: the retry loop kept ~10 heavyweight query sets running
against the DB while the user only ever saw one failure).

Retry a 504 once, after a 10s backoff instead of the usual attempt*1s,
then surface the failure.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

572 of 2839 branches covered (20.15%)

Branch coverage included in aggregate %.

343 of 1311 relevant lines covered (26.16%)

10.21 hits per line