-
Notifications
You must be signed in to change notification settings - Fork 3.2k
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
Throttling Requests #716
Comments
Is your entire app in one node.js process or is it balanced across a few? |
@mikeal Just one. |
https://npmjs.org/package/throttle-function looks good. |
Closing this - not a |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
One of the API's I use most frequently (the Shopify API). Recently put out a new call limit, I'm seeing errors like:
I've already gotten an upgrade, but regardless of how much bandwidth I get I have to account for this. A large majority of the requests to the Shopify API are within async.map() functions, which loop asynchronous requests, and gather the bodies.
I'm looking for any help, perhaps a library that already exists, that would wrap around the request module and actually block, sleep, throttle, allocate, manage, the many simultaneous requests that are firing off asynchronously and limit them to say
6
requests at a time. I have no problem with working on such a project if it doesn't exist. I just don't know how to handle this kind of situation, and I'm hoping for some kind of standard.The text was updated successfully, but these errors were encountered: