-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
Feature Request: retry request when error #55
Comments
@xg-wang Currently this is not supported by default, but it's definitely a useful feature. It could be achieved by adding a |
Or we can do it on |
@xg-wang yes, that is another good approach, however it gives less flexibility to manage retries imo. If it is a part of the |
LimitRule doesn't feel like the right place for me to handle request resubmission, since it is responsible for limiting requests and not to perform requests again. I've implemented the |
I'm down for it. Looks good. |
Hi @asciimoo do you have any example using your retry solution? |
bump on any example - since it helps give users a clear idea on how you intend users to use it. |
|
@olivierpons thanks for the response. I did add retry logic once i due in a bit into the codebase. I wanted to limit the number of retries before i give up. |
Is there a way to do request retry inside colly? Right now I can only resend the request in
onError
The text was updated successfully, but these errors were encountered: