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

Only skip decompression if the Content-Length is 0. #189

Merged
merged 1 commit into from
Sep 20, 2018

Conversation

visualphoenix
Copy link
Contributor

Resolves #187

Skip decompression if the Content-Length is 0 specifically. Chunked gzipped responses can return a Content-Length of -1 to specify an unknown size.

Skip decompression if the Content-Length is 0 specifically. Chunked
gzipped responses can return a Content-Length of -1 to specify an
unknown size.
@codecov
Copy link

codecov bot commented Sep 20, 2018

Codecov Report

Merging #189 into master will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #189   +/-   ##
=======================================
  Coverage   96.31%   96.31%           
=======================================
  Files           9        9           
  Lines        1086     1086           
=======================================
  Hits         1046     1046           
  Misses         22       22           
  Partials       18       18
Impacted Files Coverage Δ
client.go 97.02% <100%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 31a5d93...02a8706. Read the comment docs.

Copy link
Member

@jeevatkm jeevatkm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@visualphoenix Thank you for the PR.

@jeevatkm jeevatkm merged commit 466835f into go-resty:master Sep 20, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

2 participants