http request timeout - Githubissues

askmike / bitstamp

Bitstamp REST API wrapper

80 stars 54 forks source link

http request timeout #7

Closed Removed-5an closed 10 years ago

Removed-5an commented 10 years ago

Added a 5 second timeout and a domain wrapper around the http request to make sure we have no unexpected crashes.

Removed-5an commented 10 years ago

It might be unstable, but I've been running with it for over 2 months without issues. If you still don't want it in, just leave out the domain part and just add the timeout :).

askmike commented 10 years ago

Stability is also about if Node is going to support this feature the way it does now in new versions of Node (eg. unstable means they can't promise backwards compatibility).