search

mloesch / sickle

Sickle: OAI-PMH for Humans
Other
106 stars 42 forks source link

Retry all 5xx status codes #36

Closed bibliotechy closed 4 years ago

bibliotechy commented 4 years ago

This PR rebases existing PR #21 on top of the current master, ensuring that all required Travis tests pass. Since that PR has not seen any movement in a year, I though it might be easier to start clean.

I found the original PR useful, as I have also encountered multiple OAI servers that will return 500 errors instead of the 503 that is indicated in the specification. Expanding retry behavior to more 5xx status codes would be extremely useful for fully automated harvests taking place in a distributed process over a large number of diverse server implementations.

And thanks for working on sickle. It's great.

bibliotechy commented 4 years ago

@mloesch Is there any interest in getting this change in? Would love to keep using the main package instead of forking. But if you have objections to how this is implemented, I'd love to try to work towards getting this functionality in. Thanks

mloesch commented 4 years ago

Superseded by #41

mloesch commented 4 years ago

fix released with 0.7.0