[issue42130] AsyncIO's wait_for can hide cancellation in a rare race condition

Sam Bull added the comment: It has been addressed, PR should be merged this week: https://github.com/python/cpython/pull/28149 Like most open source projects, it just needed someone to actually propose a fix. -- ___ Python tracker

[issue42130] AsyncIO's wait_for can hide cancellation in a rare race condition

James Ferguson added the comment: Echoing nmatravolgyi's comments - I got here after hitting this bug and I too am amazed it's been around so long and hasn't been addressed. It makes cancellation in my application very unreliable, and the reason I need well-controlled cancellation

[issue42130] AsyncIO's wait_for can hide cancellation in a rare race condition

Change by Sam Bull : -- nosy: +dreamsorcerer nosy_count: 8.0 -> 9.0 pull_requests: +26587 pull_request: https://github.com/python/cpython/pull/28149 ___ Python tracker ___

[issue42130] AsyncIO's wait_for can hide cancellation in a rare race condition

Change by Denis S. Otkidach : -- keywords: +patch pull_requests: +24737 stage: -> patch review pull_request: https://github.com/python/cpython/pull/26097 ___ Python tracker

[issue42130] AsyncIO's wait_for can hide cancellation in a rare race condition

Change by Adam Liddell : -- nosy: +aaliddell ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue42130] AsyncIO's wait_for can hide cancellation in a rare race condition

nmatravolgyi added the comment: I've also found this deficiency of asyncio.wait_for by debugging an obscure hang in an application. Back then I've quickly made an issue about it: https://bugs.python.org/issue43389 I've just closed it as duplicate, since this issue covers the same bug and

[issue42130] AsyncIO's wait_for can hide cancellation in a rare race condition

Change by Crowbar Z : -- nosy: +crowbarz ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue42130] AsyncIO's wait_for can hide cancellation in a rare race condition

Change by Denis S. Otkidach : -- nosy: +ods ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue42130] AsyncIO's wait_for can hide cancellation in a rare race condition

Taras Voinarovskyi added the comment: Hi Chris, Yes, I do believe that is the respectful change, if we look the CancelledError is not checked to be external or originate from the timer. Best regards, Taras Voinarovskyi -- ___ Python tracker

[issue42130] AsyncIO's wait_for can hide cancellation in a rare race condition

Chris Jerdonek added the comment: It looks like issue 37658 might be the relevant change rather. Here is the new logic it introduced: https://github.com/python/cpython/blob/db455296be5f792b8c12b7cd7f3962b52e4f44ee/Lib/asyncio/tasks.py#L483-L488 (via

[issue42130] AsyncIO's wait_for can hide cancellation in a rare race condition

Change by Chris Jerdonek : -- nosy: +chris.jerdonek ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue42130] AsyncIO's wait_for can hide cancellation in a rare race condition

New submission from Taras Voinarovskyi : Hi, during migration to Python 3.8.6 we encountered a behavior change from previous versions: wait_for ignored the request to cancellation and returned instead. After investigation, it seems to be related to the update in bpo-32751 and is only