MagicStack / MagicStack/uvloop
sys.exit(...) exception swallowed by uvloop
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- Cython
- Estrellas
- 11.9k
- Forks
- 616
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Descripción
-
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.
Guía de contribución
No hay ninguna guía de contribución indexada para este repositorio
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Línea de trabajo
Comienza en uvloop/handles/basetransport.pyx, alrededor de la implementación enlazada de _fatal_error, y reproduce el comportamiento reportado de SystemExit bajo uvloop. Revisa las otras implementaciones de _fatal_error mencionadas en el issue y, después, verifica que SystemExit se propaga a la aplicación en lugar de registrarse y ser ignorado.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- python
- Área
- networking
- Tipo de issue
- Error
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 38/100