René's URL Explorer Experiment

{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"asyncio queue: get/put will not awaken next if interrupted","articleBody":"### Checklist\n\n- [X] I am confident this is a bug in CPython, not a bug in a third-party project\n- [X] I have searched the [CPython issue tracker](https://github.com/python/cpython/issues?q=is%3Aissue+sort%3Acreated-desc),\nand am confident this bug has not been reported before\n\n\n### CPython versions tested on:\n\nCPython main branch\n\n### Operating systems tested on:\n\nLinux\n\n### Output from running 'python -VV' on the command line:\n\nPython 3.11.4 (main, Jun  9 2023, 07:59:55) [GCC 12.3.0]\n\n### A clear and concise description of the bug:\n\nIf the current putter has an exception thrown (such as when the task awaiting put() has an exception thrown) and the queue isn't full (which can happen if another task put()s before the current task is rewoken), then the queue must wake up the next putter, because otherwise nothing else will wake up the next putter. This line is responsible for this:\r\nhttps://github.com/python/cpython/blob/42429d3b9adb8af1eadcfa155f6e8422a254ec67/Lib/asyncio/queues.py#L130\r\n\r\nUnfortunately that condition - specifically `not putter.cancelled()` - will never be met because just before, this:\r\nhttps://github.com/python/cpython/blob/42429d3b9adb8af1eadcfa155f6e8422a254ec67/Lib/asyncio/queues.py#L122\r\n\r\nThis affects both `put()` and `get()`. I propose moving `cancel()` to just before the `raise`.\r\n\r\nThere is no test case - this is something I noticed while reading `asyncio/queue.py`.\n\n\u003c!-- gh-linked-prs --\u003e\n### Linked PRs\n* gh-108066\n\u003c!-- /gh-linked-prs --\u003e\n","author":{"url":"https://github.com/orbisvicis","@type":"Person","name":"orbisvicis"},"datePublished":"2023-08-16T16:33:33.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":6},"url":"https://github.com/108042/cpython/issues/108042"}