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

gh-102780: Fix uncancel() call in asyncio timeouts #102815

Merged
merged 13 commits into from
Mar 22, 2023
Merged

gh-102780: Fix uncancel() call in asyncio timeouts #102815

Changes from 1 commit
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
9f96cea
document use of uncancel() when suppressing CancelledError
kristjanvalur Mar 18, 2023
07517b7
Create unittest
kristjanvalur Mar 18, 2023
1150db0
record previous cancel state in timeout context manager
kristjanvalur Mar 18, 2023
d207fb1
explicitly raise from cancelled error
kristjanvalur Mar 18, 2023
912f26d
Update Doc/library/asyncio-task.rst
kristjanvalur Mar 20, 2023
ee574fc
Clarify how code should "ignore" CancelledError
kristjanvalur Mar 20, 2023
c91dae7
Apply suggestions from code review
kristjanvalur Mar 20, 2023
902691e
Add unittest for asyncio.TimeoutError.__cause__
kristjanvalur Mar 20, 2023
0f7bde5
Merge branch 'main' into kristjan/uncancel
kristjanvalur Mar 20, 2023
07c40cc
add news
kristjanvalur Mar 22, 2023
2a8914e
Merge branch 'main' into kristjan/uncancel
kristjanvalur Mar 22, 2023
a6f3114
Rename news file
kristjanvalur Mar 22, 2023
6e299fa
fix news entry
kristjanvalur Mar 22, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 3 additions & 0 deletions Misc/NEWS.d/next/Library/2023-03-22-16-15-18.bpo-102780.NEcljy.rst
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,3 @@
The `asyncio.timeout()` now works reliably even when performing cleanup due
to task cancellation. Previously the context manager could raise a
`asyncio.CancelledError` instead of an `asyncio.TimeoutError` in such cases.
gvanrossum marked this conversation as resolved.
Show resolved Hide resolved