OpenSource/python-telegram-bot
OpenSource/python-telegram-bot
1
0
Fork 0
mirror of https://github.com/python-telegram-bot/python-telegram-bot.git synced 2024-10-24 01:46:22 +02:00
Code Issues Projects Releases Packages Wiki Activity

Make ConversationHandler and run_async work together properly (#1176)

Browse source 
* #1175

* fix issue with timeout

* exception in promise resolve is treated as None result

* removed useless variables
This commit is contained in:
Kirill Vasin 2018-09-10 21:05:45 +03:00 committed by Eldinnie
parent bbe633e571
commit b8c288ff4a
1 changed files with 11 additions and 12 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

23
telegram/ext/conversationhandler.py
View file

@ -230,18 +230,17 @@ class ConversationHandler(Handler):
self.logger.debug('waiting for promise...')
old_state, new_state = state
error = False
try:
res = new_state.result(timeout=self.run_async_timeout)
except Exception as exc:
self.logger.exception("Promise function raised exception")
self.logger.exception("{}".format(exc))
error = True
if not error and new_state.done.is_set():
self.update_state(res, key)
state = self.conversations.get(key)
if new_state.done.wait(timeout=self.run_async_timeout):
try:
res = new_state.result(timeout=0)
res = res if res is not None else old_state
except Exception as exc:
self.logger.exception("Promise function raised exception")
self.logger.exception("{}".format(exc))
res = old_state
finally:
self.update_state(res, key)
state = self.conversations.get(key)
else:
for candidate in (self.timed_out_behavior or []):
if candidate.check_update(update):