aio-libs / aio-libs/aiozmq

Bug: TimeoutError in default error_table in aiozmq.rpc not handled

Aberta
#93 0 comentários 0 reações 0 responsáveis Ver no GitHub
Linguagem predominante
Python
Estrelas
431
Forks
57
Métricas de merge de PRs
Nenhum PR com merge em 30d

Descrição

`asyncio.TimeoutError`'s `__module__` property is "concurrent.futures._base" instead of "asyncio" because `asyncio` imports the `TimeoutError` exception class from `concurrent.futures`.

Since the current RPC server is using `__module__` property, as shown in [`process_call_result()`](https://github.com/aio-libs/aiozmq/blob/007a296/aiozmq/rpc/rpc.py#L288-L289), while the client-side default error table is constructed using manual module name composition in utils.py (`"asyncio." + name`), `TimeoutError` from server-side codes is not handled as intended.

So, `_fill_error_table()` in utils.py should be fixed to use the same mechanism to generate the key strings.

Guia de contribuição

Abrir o guia de contribuição

Avaliação

Esta issue ainda não foi avaliada.

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.