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

Updated httpx module to v0.25.0 and relative dependencies #1621

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

CastagnaIT
Copy link
Owner

@CastagnaIT CastagnaIT commented Sep 13, 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

Updated httpx module to v0.25.0 and relative dependencies to their last github branch/tag versions

atm not tested,
tested on android
"BlockingIOError" error raised with httpx version 0.23 (#1485) dont happens anymore
but now httpcore.RemoteProtocolError: Server disconnected is raised always after some time, that prevent addon to working well (usually this error is raised after waiting few minutes or while/after watching a video)

httpcore.RemoteProtocolError: Server disconnected is a custom error of httpcore and there are no specifics info on how to fix/workaround this problem, testing Requests module seem dont raise similar problems

Screenshots (if appropriate):

Currently there is no way to enable TCP KeepAlive directly on httpx
module

ref. PR:#1172
The new httpx version has disabled it by default
@CastagnaIT CastagnaIT added the WIP PR that is still being worked on label Sep 14, 2023
@CastagnaIT CastagnaIT mentioned this pull request Sep 17, 2023
10 tasks
@CastagnaIT CastagnaIT added the On hold Will be processed later label Sep 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
On hold Will be processed later WIP PR that is still being worked on
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant