-
-
Notifications
You must be signed in to change notification settings - Fork 735
Disable transparent ungzip and add decompress interceptor #36
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
Disable transparent ungzip and add decompress interceptor #36
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: newline at EOF
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
kk
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: whitespace
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
kk
79d0f96
to
6802178
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Comment somewhere why we set Content-Length
and totalSize
to -1?
Small nits, squash and LGTM |
6802178
to
01a30d5
Compare
…interceptor_when_we_have_external_interceptors Disable transparent ungzip and add decompress interceptor
@wangmengyan95 updated the pull request. |
In order to expose the raw response(ungziped response) to developers, we need to disable the transparent ungzip for ApacheHttpClient and URLConnection and add a decompress interceptor at the beginning of the interceptor list manually.
For OkHttp, since we can not disable the transparent ungzip, we will handle it separately.