-
Notifications
You must be signed in to change notification settings - Fork 547
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
reconnect on failure ? #6
Comments
I haven't had a need. I'm open for patches though. |
I'd gladly do so, but any idea how this could be tested ? long story short : I started logging the activity on some keywords and my app crashed after 20 hours. Fiddled and try to add some reconnect after a couple of seconds… Crashed after 24 hours or so without reconnecting. Anyways, the thing is : I obviously can't afford to wait a day or so until a crash to see if my handling of a disconnect is working… Of course, I could switch on/off my laptop's wifi, but then again, just wondering if you had any ideas of a more… elegant way of dealing with it. |
What's causing the crashes? If it's just a loss of connection between Twitter and your script, write a stupid simple node server that emits random json, and run local tests with it dying. I've talked to some people that ran into this, and they just had some code that would immediately catch and reopen the connection. I switched jobs after writing this, and have no real use for it anymore. All I can suggest is try some stuff out in a personal fork, and submit a pull request to get some in the community to look at it. If it works, I'll gladly take the patch. Also, try and be polite to Twitter's servers: http://dev.twitter.com/pages/user_streams_suggestions#Failure :) |
I've submitted a patch for this issue. Please let me know if you have any questions. |
Fix for Issue #6 "reconnect on failure ?"
any plans to implement auto-reconnect on failure ?
The text was updated successfully, but these errors were encountered: