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.

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

Fix a source of the 'Unknown API error.' #47

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

aminsdo
Copy link

@aminsdo aminsdo commented Jun 8, 2023

Hello dear Proton team and community,

Since November 2020 the parameter strict was removed from urllib3 for HTTPSConnectionPool class in connectionpool.py. Leaving the parameter strict in the call is shifting the next ones and timeout is taken as a boolean leading to the error 'Timeout cannot be a boolean, it must be an...' raising an UnknownConnectionError in api.py and finally the famous UnknownAPIError in the cli_wrapper.py.

This explains why so many people on forums have their proton cli not working after updating their packages with urllib3 among them.

Since November 2020 the parameter 'strict' was removed from urllib3
HTTPSConnectionPool and HTTPConnectionPool classes' init in connectionpool.py.
Leaving the parameter 'strict' in the call is shifting the next ones and
timeout is taken as a boolean leading to the error 'Timeout cannot be a
boolean, it must be an...' raising an UnknownConnectionError in api.py
and finally the famous UnknowAPIError in the cli_wrapper.py.

This explains why so many people have their proton cli not working after
updating their packages with urllib3 among them.
@waltercool
Copy link

Can confirm this works.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants