-
Notifications
You must be signed in to change notification settings - Fork 234
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
How to set timeout? #67
Comments
OT: How long have you tried to wait?.. For me, all queries return a response (or raise a |
Sorry, my bad. I get an
Is it just me or isn't it possible right now to create an order at Kraken via API? Sorry for the mess, i do realize that this has nothing todo with my initial question... |
No problem! ATM, Kraken is indeed a mess; |
BTW, careful with |
Please don't close the issue, though: the topic question is still valid. |
Tonight I had a situation where quite likely my internet connection was dropped while krakenex was waiting for a reply. That ended up waiting for that reply endlessly (hours), so adding a (reasonable) timeout would still be a good idea, even if this only occurs rarely. |
I sometimes have the same problem as @TheSeven. I run a script that queries the public endpoint for the current price at regular intervals, and occasionally I’ll get a response that never comes back. Currently I have to manually inspect and reset, but I’d really like to handle it with a timeout to just give up and move on. |
This is proposed solution to veox#67. The newly added timeout parameter is None by default, so the default behavior remains unchanged, but gives the option to add a timeout to those who wish to use one.
PR #77 has been merged - introducing a The value passed should be a number ( For now, there is no way to set a default timeout (for all queries and/or connection opening). |
Versions
What are you trying to achieve?
Get a response from Kraken
What do you expect to happen?
Request should wait till response is returned
What happens instead?
I get no response
The text was updated successfully, but these errors were encountered: