-
Notifications
You must be signed in to change notification settings - Fork 27
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
Handle lost connection to telegram servers gracefully #54
Comments
So I think it should just return an error right away. |
@luckydonald I don't get it |
It should work with the next update, I'm closing this issue. Reopen it if the problem exists also in the next update. |
Any concrete commit fixing this? |
Is still not working with the current release? |
No idea, didn't test yet. Like, did a upstream merge change that, or did one of you lovely people fix it? |
Currently, if telegram isn't working, requests are simply left open, stalling forever.
That's not great as that means basically DDoS potential.
To Reproduce:
The text was updated successfully, but these errors were encountered: