Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Handle Telegram API response in case of failure #179

Open
rougeth opened this issue Oct 21, 2018 · 0 comments
Open

Handle Telegram API response in case of failure #179

rougeth opened this issue Oct 21, 2018 · 0 comments
Labels
bug hacktoberfest

Comments

@rougeth
Copy link
Owner

rougeth commented Oct 21, 2018

Bottery ignores errors from Telegram API. If, for example, an invalid token is used, no message is shown and, the process breaks because it tries to continue even if something is wrong with Telegram.

Traceback (most recent call last):
  File "bottery/telegram/engine.py", line 95, in polling
    tasks = [self.message_handler(msg) for msg in updates['result']]
KeyError: 'result'
ivkalita added a commit to ivkalita/bottery that referenced this issue Oct 22, 2018
@ivkalita
rougeth#179: Added general telegram exceptions handling.
bef09f9
ivkalita added a commit to ivkalita/bottery that referenced this issue Oct 22, 2018
@ivkalita
rougeth#179: Added general telegram exceptions handling.
4e28e42
@ivkalita ivkalita mentioned this issue Oct 22, 2018
Closed
ivkalita added a commit to ivkalita/bottery that referenced this issue Oct 23, 2018
@ivkalita
rougeth#179: Added a note about Bottery.stop inside the exception han…
2bead2c 
…dler.
ivkalita added a commit to ivkalita/bottery that referenced this issue Oct 23, 2018
@ivkalita
rougeth#179: Introduced PlatformError.
2cef49e
@ivkalita ivkalita mentioned this issue Oct 23, 2018
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug hacktoberfest
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

#179: Added general telegram exceptions handling. ivkalita/bottery
1 participant
@rougeth