Skip to content
This repository has been archived by the owner on Apr 4, 2021. It is now read-only.

Switch all GET requests to RETRY requests for robustness #26

Closed
tanho63 opened this issue Feb 23, 2021 · 1 comment
Closed

Switch all GET requests to RETRY requests for robustness #26

tanho63 opened this issue Feb 23, 2021 · 1 comment

Comments

@tanho63
Copy link

tanho63 commented Feb 23, 2021

Is your feature request related to a problem? Please describe.
Suggestion for best-practice - RETRY is more robust to accidental API unavailability - per https://blog.r-hub.io/2020/04/07/retry-wheel/

Describe the solution you'd like
Should be as simple as swapping out all instances of httr::GET(...) with httr::RETRY("GET",...) - is drop-in replacement

@saiemgilani
Copy link
Owner

archiving repository, this suggestion was implemented in cfbfastR per suggestion.

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

No branches or pull requests

2 participants