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

Replaced HTTPX to Requests module #1622

Merged
merged 1 commit into from
Sep 20, 2023
Merged

Replaced HTTPX to Requests module #1622

merged 1 commit into from
Sep 20, 2023

Conversation

CastagnaIT
Copy link
Owner

@CastagnaIT CastagnaIT commented Sep 17, 2023

Check if this PR fulfills these requirements:

Types of changes

  • New feature (non-breaking change which adds functionality)
  • Feature change (non-breaking change which changes behaviour of an existing functionality)
  • Improvement (non-breaking change which improves functionality)
  • Bug fix (non-breaking change which fixes an issue)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Refactor (non-breaking performance or readability improvements)

Description

Our local old httpx version have problems with new pyhton versions e.g. 3.11
by raising error AttributeError: module 'httpcore' has no attribute 'NetworkBackend'

and updating httpx to last version (0.25) cause folowing errors httpcore.RemoteProtocolError: Server disconnected
at least while testing on android (#1621), that make almost unusuable the addon,
so for now has been restored Requests module

With this will be lost http2 compatibility but since the login is broken, should be not a problem.
NOTE: KeepAlive fix has been not reintroduced intentionally (ref #1065).

fix #1611

In case of Feature change / Breaking change:

Describe the current behavior

Describe the new behavior

Screenshots (if appropriate):

Our local old httpx version have problems with new pyhton versions e.g. 3.11
update the httpx to last version (0.25) cause folowing errors
httpcore.RemoteProtocolError: Server disconnected
at least while testing on android, that make almost unusuable the addon
so for now has been reverted to Requests module.
With this will be lost http2 compatibility but since the login is broken, should be not a problem.
NOTE: KeepAlive fix has been not reintroduced intentionally.
@CastagnaIT CastagnaIT merged commit c56dc6d into master Sep 20, 2023
12 checks passed
@CastagnaIT CastagnaIT deleted the revert_httpx branch September 20, 2023 07:15
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.

AttributeError: module 'httpcore' has no attribute 'NetworkBackend'
1 participant