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

An extra delay seems to be required between requests #5

Closed
Riches opened this issue Jul 4, 2016 · 3 comments
Closed

An extra delay seems to be required between requests #5

Riches opened this issue Jul 4, 2016 · 3 comments

Comments

@Riches
Copy link

Riches commented Jul 4, 2016

The README tells us that "Only one request can be done at a time by each client. You'll have to wait for the first request to be processed before sending another one."

I've implemented a queue that processes the next request only after the previous one has returned its promise. However, requests seem to hang unless an additional 700ms delay is added after the GC returns and the promise is satisfied.

Is there a programmatic way to identify when the GC is ready for a new request? Or is this a bug with this implementation?

Many thanks for any assistance.

@balthazar
Copy link
Owner

Unfortunately no, or at least not to my knowledge, and that's why it's not integrated by default. Still looking for solutions atm, but you're welcome to keep me up-to-date if you find anything ;)

@Riches
Copy link
Author

Riches commented Jul 4, 2016

Ah right, I'll keep playing around for a bit looking around for some kind of solution. Cheers.

@Maarethyu
Copy link
Contributor

I am pretty sure you can only do 1 request per second, but I don't remember where this info comes from so I may be wrong.

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

No branches or pull requests

3 participants