Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

import name 'CeilTimeout' from 'aiohttp.helpers' #6

Closed
minamotorin opened this issue Feb 6, 2022 · 2 comments
Closed

import name 'CeilTimeout' from 'aiohttp.helpers' #6

minamotorin opened this issue Feb 6, 2022 · 2 comments
Labels
bug Something isn't working

Comments

@minamotorin
Copy link
Owner

minamotorin commented Feb 6, 2022 •
edited

Some users report that the error import name 'CeilTimeout' from 'aiohttp.helpers' happend with recent aiohttp.
And they said that the solution is using old aiohttp (v3.7.0).

However, the problem doesn't occur for me.
Surely, it seems that aiohttp.helpers.CeliTimeout was removed since aiohttp v3.8.0, but other codes of aiohttp were also updated to stop using aiohttp.helpers.CeliTimeout.
aiohttp.helpers.CeliTimeout might inner code of aiohttp and it might have to be called by only from aiohttp.
Twint doesn't call aiohttp.helpers.CeliTimeout directly.

So I think this is a wrong installation problem of users.
Maybe also python -c 'import aiohttp' causes the error.
Try pip install --force-reinstall aiohttp.
In my environment, twint works with also both aiohttp v3.8.0 and v3.8.1.
@minamotorin minamotorin added the wontfix This will not be worked on label Feb 6, 2022
@minamotorin minamotorin added invalid This doesn't seem right and removed wontfix This will not be worked on labels Feb 6, 2022
@minamotorin
Copy link
Owner Author

minamotorin commented Feb 6, 2022 •
edited

Related

@minamotorin minamotorin mentioned this issue Mar 7, 2022
Open
minamotorin added a commit that referenced this issue Mar 8, 2022
@minamotorin
Fix import name 'CeilTimeout' from 'aiohttp.helpers'
0c63886 
Fix #6
Solution is from twintproject#1360 (comment)

I'm not sure about twintproject#831, but it seems work fine with also aiohttp_socks 0.4.1 or later.
Version limitation of aiohttp_socks was added by commit twintproject@fef94a5
@minamotorin minamotorin added bug Something isn't working and removed invalid This doesn't seem right labels Mar 8, 2022
@minamotorin
Copy link
Owner Author

This was a bug. Fixed in 0c63886.

Solution and following description is from twintproject#1360 (comment) by @tperka.

Just remove aiohttp-socks version limitation in requirements.txt because it is enforced to be old version (0.4.1) while aiohttp package is not so it just tries to use functions which has gotten deleted. Newest version of aiohttp-socks package makes this work again :)

@minamotorin minamotorin pinned this issue Mar 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@minamotorin