tdlight-team/tdlight-telegram-bot-api

Handle lost connection to telegram servers gracefully

luckydonald opened this issue · 6 comments

luckydonald commented

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:

  1. Lauch bot api
  2. Disconnect Wifi or whatever
  3. Send a request to the Bot API.
  4. ????
  5. Boom
luckydonald commented

So I think it should just return an error right away.
Just like the Internal Server Error: restart one.

penn5 commented

@luckydonald I don't get it

cavallium commented

It should work with the next update, I'm closing this issue. Reopen it if the problem exists also in the next update.

luckydonald commented

Any concrete commit fixing this?

cavallium commented

Is still not working with the current release?

luckydonald commented

No idea, didn't test yet.
I was rather curious by what it could be fixed now.

Like, did a upstream merge change that, or did one of you lovely people fix it?