Skip to content
This repository was archived by the owner on Oct 18, 2022. It is now read-only.

Remove strict check for Content-Length header #41

Merged
merged 1 commit into from
May 14, 2018

Conversation

faulesocke
Copy link

This header is automatically removed from the response by reqwest on
compressed responses (and also does not make any sense there).

This closes #40

@faulesocke
Copy link
Author

rustfmt also did some cleanups to the code, do you want those as well?

@jonas-schievink
Copy link
Owner

Please update the doc comment, too. I think it's best if it doesn't mention the exact checks it performs.

I'd prefer to hold off on running rustfmt for now. I should probably take a look at rustfmt and run it on everything if it makes it look better.

This header is automatically removed from the response by reqwest on
compressed responses (and also does not make any sense there).
@jonas-schievink
Copy link
Owner

Thanks!

@jonas-schievink jonas-schievink merged commit 33df1e4 into jonas-schievink:master May 14, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

"Expected Content-Length header, but none was found" on gzip compressed data
2 participants