Skip to content

migrate from aiohttp to httpx#83

Merged
DBoyara merged 2 commits into
DBoyara:masterfrom
a-popovkin:master
Feb 6, 2026
Merged

migrate from aiohttp to httpx#83
DBoyara merged 2 commits into
DBoyara:masterfrom
a-popovkin:master

Conversation

@a-popovkin

@a-popovkin a-popovkin commented Jan 16, 2026

Copy link
Copy Markdown
Contributor

It seems finam api forces to use HTTP/2.

curl -sI --http2 https://api.finam.ru/v1/assets -o /dev/null -w '%{http_version}\n'
2

So on request https://api.finam.ru/v1/assets we are facing error 500. As aiohttp does not support HTTP/2, I suggest migrating to httpx lib.
CleanShot 2026-01-16 at 12 20 10@2x

@DBoyara

DBoyara commented Jan 23, 2026

Copy link
Copy Markdown
Owner

Я в целом не против, но на aiohttp запрос работает) Проверю еще все запросы с httpx.

@a-popovkin

a-popovkin commented Jan 23, 2026

Copy link
Copy Markdown
Contributor Author

Точно ли работает? У меня с разных провайдеров выкидывает в 500ую ошибку =(

Причем, это единственный запрос, который падает. Там 3mb ответ. Вот видимо, не влезает

python /Users/popovkin/Code/3dparty/FinamTradeApiPy/examples/assets.py Traceback (most recent call last): File "/Users/popovkin/Code/3dparty/FinamTradeApiPy/examples/assets.py", line 31, in <module> asyncio.run(main()) File "/Users/popovkin/.pyenv/versions/3.11.13/lib/python3.11/asyncio/runners.py", line 190, in run return runner.run(main) ^^^^^^^^^^^^^^^^ File "/Users/popovkin/.pyenv/versions/3.11.13/lib/python3.11/asyncio/runners.py", line 118, in run return self._loop.run_until_complete(task) ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ File "/Users/popovkin/.pyenv/versions/3.11.13/lib/python3.11/asyncio/base_events.py", line 654, in run_until_complete return future.result() ^^^^^^^^^^^^^^^ File "/Users/popovkin/Code/3dparty/FinamTradeApiPy/examples/assets.py", line 15, in main pprint(await client.assets.get_assets()) ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ File "/Users/popovkin/Code/3dparty/FinamTradeApiPy/finam_trade_api/assets/assets.py", line 65, in get_assets response, ok = await self._exec_request( ^^^^^^^^^^^^^^^^^^^^^^^^^ File "/Users/popovkin/Code/3dparty/FinamTradeApiPy/finam_trade_api/base_client/base.py", line 71, in _exec_request response.raise_for_status() File "/Users/popovkin/Library/Caches/pypoetry/virtualenvs/finam-trade-api-d2aXmdeK-py3.11/lib/python3.11/site-packages/aiohttp/client_reqrep.py", line 636, in raise_for_status raise ClientResponseError( aiohttp.client_exceptions.ClientResponseError: 500, message='Internal Server Error', url='https://api.finam.ru/v1/assets'

@DBoyara DBoyara added the bug Something isn't working label Feb 6, 2026
@DBoyara

DBoyara commented Feb 6, 2026

Copy link
Copy Markdown
Owner

надо как-то создать бранчь от PR) Сделаешь? Я разрулю lock файл и добавлю версию. А то я так не могу подтянуть себе PR

@DBoyara

DBoyara commented Feb 6, 2026

Copy link
Copy Markdown
Owner

Хотя, лан, я ща сделаю

@DBoyara DBoyara merged commit 3f9d5ba into DBoyara:master Feb 6, 2026
0 of 7 checks passed
@DBoyara

DBoyara commented Feb 6, 2026

Copy link
Copy Markdown
Owner

Готово https://pypi.org/project/finam-trade-api/4.2.5/ Спасибо

@a-popovkin

Copy link
Copy Markdown
Contributor Author

Тебе спасибо)
У тебя тоже повторилась проблема? Или только у меня была?

Кстати, сорри, забыл еще в readme подправить aiohttp -> httpx

@DBoyara

DBoyara commented Feb 6, 2026

Copy link
Copy Markdown
Owner

Да, у меня тоже повторилась, хотя в декабре работало) Рандомная штука какая-то. Ридми я тоже упустил, потом поправлю

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants