Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Retrying indefinitely on 503 error for Azul summary and index endpoints #203

Closed
NoopDog opened this issue Sep 23, 2024 · 1 comment
Closed

Retrying indefinitely on 503 error for Azul summary and index endpoints #203

NoopDog opened this issue Sep 23, 2024 · 1 comment
Assignees
@hunterckx

Comments

@NoopDog
Copy link
Collaborator

NoopDog commented Sep 23, 2024 •
edited
Loading

image

I thought we had limited retries to three and had a small delay between them.
What broke this?
Let's add a unit test here as well, please.
@NoopDog NoopDog assigned frano-m and hunterckx and unassigned frano-m Sep 23, 2024
@NoopDog NoopDog mentioned this issue Sep 24, 2024
Closed
hunterckx added a commit that referenced this issue Oct 10, 2024
@hunterckx
feat: limit retries and use exponential backoff for api requests (#203)
93cfff1
hunterckx added a commit that referenced this issue Oct 11, 2024
@hunterckx
test: add tests for api retry (#203)
14969dd
hunterckx added a commit that referenced this issue Oct 17, 2024
@hunterckx
refactor: use ky instead of axios (#203)
b0ade6a
hunterckx added a commit that referenced this issue Oct 17, 2024
@hunterckx
test: refactor tests to work with esm (#203)
d0b2b97
hunterckx added a commit that referenced this issue Oct 17, 2024
@hunterckx
test: finish updating fetchApi tests for ky (#203)
839f62a
hunterckx added a commit that referenced this issue Oct 17, 2024
@hunterckx
chore: rename fetchApi tests (#203)
beb7618
hunterckx added a commit that referenced this issue Oct 17, 2024
@hunterckx
test: use async imports directly in variable declarations (#203)
f0c913a
hunterckx added a commit that referenced this issue Oct 17, 2024
@hunterckx
chore: remove entityURL (#203)
3efbee7
hunterckx added a commit that referenced this issue Oct 17, 2024
@hunterckx
feat: limit retries and use exponential backoff for api requests (#203)
522b3b0
hunterckx added a commit that referenced this issue Oct 17, 2024
@hunterckx
test: add tests for api retry (#203)
75d00bf
hunterckx added a commit that referenced this issue Oct 17, 2024
@hunterckx
refactor: use ky instead of axios (#203)
6b6f8bd
hunterckx added a commit that referenced this issue Oct 17, 2024
@hunterckx
test: refactor tests to work with esm (#203)
dddcde0
hunterckx added a commit that referenced this issue Oct 17, 2024
@hunterckx
test: finish updating fetchApi tests for ky (#203)
2183670
hunterckx added a commit that referenced this issue Oct 17, 2024
@hunterckx
chore: rename fetchApi tests (#203)
9d3ea62
hunterckx added a commit that referenced this issue Oct 17, 2024
@hunterckx
test: use async imports directly in variable declarations (#203)
7d373e3
hunterckx added a commit that referenced this issue Oct 17, 2024
@hunterckx
chore: remove entityURL (#203)
ab1fd60
NoopDog pushed a commit that referenced this issue Oct 18, 2024
@hunterckx
feat!: use ky with limited retries instead of axios (#203) (#216)
ec18080
This was referenced Oct 18, 2024
Merged
Closed
@NoopDog
Copy link
Collaborator Author

NoopDog commented Oct 18, 2024

Complete! Thanks @hunterckx !

@NoopDog NoopDog closed this as completed Oct 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@hunterckx hunterckx

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@NoopDog @frano-m @hunterckx