Skip to content

Don't call done_callback if Promise raises an exception.#2544

Merged
Bibo-Joshi merged 3 commits into
masterfrom
fix-promise
Jun 5, 2021
Merged

Don't call done_callback if Promise raises an exception.#2544
Bibo-Joshi merged 3 commits into
masterfrom
fix-promise

Conversation

@starry-shivam

Copy link
Copy Markdown
Member

Closes #2542

Comment thread telegram/ext/filters.py Outdated
Comment thread telegram/ext/utils/promise.py Outdated
starry-shivam and others added 2 commits June 5, 2021 18:20

@Bibo-Joshi Bibo-Joshi left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Rest looks good 🙂

@Bibo-Joshi Bibo-Joshi merged commit 46cdeb4 into master Jun 5, 2021
@Bibo-Joshi Bibo-Joshi deleted the fix-promise branch June 5, 2021 15:08
sakibguy added a commit to sakibguy/python-telegram-bot that referenced this pull request Jun 6, 2021
@github-actions github-actions Bot locked and limited conversation to collaborators Jun 6, 2021
@Bibo-Joshi Bibo-Joshi added 🔌 bug pr description: bug and removed bug 🐛 labels Nov 3, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

🔌 bug pr description: bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] promise mistreats pooled_function exceptions as done_callback exceptions

3 participants