sys.exit(...) exception swallowed by uvloop
まだ誰も着手していません。
評価
- 難易度
- 4/5
- 見積もり時間
- 3〜5日
- 初心者へのやさしさ
- 38/100
- issue の種類
- バグ
- 明瞭さ
- おおむね明確
- 活発さ
- 停滞
- 技術スタック
- python
- 領域
- networking
調査の方向性
uvloop/handles/basetransport.pyx の、リンクされている _fatal_error 実装付近から始め、uvloop で報告されている SystemExit の動作を再現します。Issue で言及されている他の _fatal_error 実装を確認し、その後、SystemExit がログに記録されて握りつぶされるのではなく、アプリケーションまで伝播することを検証します。
索引モデルが issue の本文から書いたものです。
説明
-
uvloop version:
0.19.0 -
Python version:
3.11.9 -
Platform:
MacOS 14.5 (Sonoma)
mamba 1.5.8
conda 24.5.0 -
Can you reproduce the bug with
PYTHONASYNCIODEBUGin env?:
Not yet -
Does uvloop behave differently from vanilla asyncio? How?:
I have never had this bug with vanilla asyncio
If a sys.exit(...) call is made while uvloop is being used in coroutines (in my case, through aiohttp crawling of web URLs), uvloop may catch the SystemExit exception, and not let it bubble as a SystemExit to the top of the application. Instead, it logs this at the ERROR level:
2024-07-26 06:25:06,802 ERROR asyncio Fatal error on transport TCPTransport
protocol: <uvloop.loop.SSLProtocol object at 0x16b54de80>
transport: <TCPTransport closed=False reading=False 0x1693bfff0>
Traceback (most recent call last):
File "uvloop/handles/stream.pyx", line 1007, in uvloop.loop.__uv_stream_buffered_on_read
File "uvloop/loop.pyx", line 109, in uvloop.loop.run_in_context1
File "uvloop/sslproto.pyx", line 382, in uvloop.loop.SSLProtocol.buffer_updated
File "uvloop/sslproto.pyx", line 711, in uvloop.loop.SSLProtocol._do_read
File "uvloop/sslproto.pyx", line 776, in uvloop.loop.SSLProtocol._do_read__copied
File "/opt/homebrew/Caskroom/miniforge/base/envs/work/lib/python3.11/ssl.py", line 912, in read
def read(self, len=1024, buffer=None):
File "/Users/dmoklaf/Local/metax/python_libraries/application.py", line 171, in _exit
sys.exit(code)
SystemExit: 1
Indeed, a quick look at the uvloop code shows that it logs the SystemExit exception, as SystemExit does not subclass OSError:
https://github.com/MagicStack/uvloop/blob/6c770dc3fbdd281d15c2ad46588c139696f9269c/uvloop/handles/basetransport.pyx#L40
PS: there also seems to be other _fatal_error implementations in uvloop, I am not completely sure if these other places do not suffer from the same issue
And it also swallows the SystemExit exception, instead of propagating it upward as requested by the Python specification.
- 主要言語
- Cython
- スター
- 11.9k
- フォーク
- 615
- PR マージ指標
- 30日以内にマージされた PR はありません
コントリビューションガイド
このリポジトリのコントリビューションガイドは索引されていません
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
MagicStack/uvloop のほかの issue
-
License not clear オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
MagicStack/uvloop#759 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
MagicStack/uvloop#741 · リアクション 2 件 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
MagicStack/uvloop#702 · コメント 8 件 · リアクション 9 件 ·
-
難易度 4/5 3〜5日 初心者へのやさしさ 25/100
MagicStack/uvloop#766 ·
-
難易度 4/5 3〜5日 初心者へのやさしさ 52/100
MagicStack/uvloop#765 ·
MagicStack/uvloop の issue をすべて見る
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
sonic-net/sonic-mgmt#28113 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
canonical/postgresql-watcher-operator#110 · コメント 1 件 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
EleutherAI/lm-evaluation-harness#4201 ·