Interesting that we hadn't bumped into this previously, but yeah, we were resubmitting the exact same request every time. This PR changes the retryOnStatus function to re-run the request builder on retry.
[x] Fix
[ ] Test
Closes #549
Still writing tests, but it would be good to have this working in the meantime. The CLI changes are extremely straightforward, and continue to work in the non-timeout scenario that I can test locally. Writing tests for the 502 case, which I'll put in a separate PR
Interesting that we hadn't bumped into this previously, but yeah, we were resubmitting the exact same request every time. This PR changes the
retryOnStatusfunction to re-run the request builder on retry.Closes #549
Still writing tests, but it would be good to have this working in the meantime. The CLI changes are extremely straightforward, and continue to work in the non-timeout scenario that I can test locally. Writing tests for the 502 case, which I'll put in a separate PR