Skip to content

Bugfix/handle chunked data - #29

Open
doronz88 wants to merge 2 commits into
gtsystem:masterfrom
doronz88:bugfix/handle-chunked-data
Open

Bugfix/handle chunked data#29
doronz88 wants to merge 2 commits into
gtsystem:masterfrom
doronz88:bugfix/handle-chunked-data

Conversation

@doronz88

Copy link
Copy Markdown
  • Had problems using this lib over HTTP
  • Tested with python 3.11 & 3.13 :)

@gtsystem

gtsystem commented Aug 19, 2026

Copy link
Copy Markdown
Owner

Hi, I'm very sorry for the 2 years delay in addressing this PR.
I did see it when it first come in, but I was not able to investigate the changes immediately and somehow I forgot to followup.

There are two separate changes in here:

  1. Fix for chunked encoding: urllib3 decode_content controls Content-Encoding such as gzip, but not Transfer-Encoding: chunked. Chunked responses should already work transparently through urllib3. Do you have a url for which the service fail with chunked data? We may need to create a failing test and then a proper fix for it.
  2. Removal of member position index: This change seems unrelated and while the library will work without it, the number of http calls needed will increase, removing one of the main optimization that this module offer. So I'm afraid I cannot merge this second part of the change.

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