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.

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

Automatically retry on network errors #33

Open
sathoro opened this issue Mar 22, 2023 · 2 comments
Open

Automatically retry on network errors #33

sathoro opened this issue Mar 22, 2023 · 2 comments

Comments

@sathoro
Copy link

sathoro commented Mar 22, 2023

Hey, thanks for this library. I was able to reduce our DO code by like 90%

We are using it in production and now that we are getting some significant request volume there are some transient DO network errors popping up.

For example, we have several Cannot resolve Durable Object due to transient issue on remote node. and Network connection lost. in Sentry. These are on routes that do nothing but connect to DO and return some data so even though there is no stack trace I am pretty confident both of these issues are DO related (well obviously the first one is, but no clue where else that second one would be coming from).

It was suggested in the CF Discord to do retrying but that will be quite messy to add in the user code.

Would it be possible for this library to automatically do retrying with backoff on these types of errors?

Thanks a lot!

@sathoro
Copy link
Author

sathoro commented Mar 22, 2023

Related fix: dabblewriter/durable-apis@dad78ba

@sathoro
Copy link
Author

sathoro commented Apr 24, 2023

I forked it and added the auto retrying in: v1.x...sathoro:itty-durable:v1.x

Can't do a PR since I add to change some other code though

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant