Use "Connection: close" instead of chunked encoding on HTTP/1.0 #1722

Merged
merged 2 commits into from Mar 24, 2017

Conversation

Projects
None yet
1 participant
@s-ludwig
Member

s-ludwig commented Mar 24, 2017

For variable body lengths, the server erroneously used chunked transfer encoding on HTTP/1.0 connections. Since there are still proxy servers that don't support HTTP/1.1, this can still be a practical issue. This instead switched to "Connection: close" without a "Content-Length" header.

Fixes #1721.

s-ludwig added some commits Mar 24, 2017

Use "Connection: close" instead of chunked encoding on HTTP/1.0. Fixes
…#1721.

For variable body lengths, the server erroneously used chunked transfer encoding on HTTP/1.0 connections. Since there are still proxy servers that don't support HTTP/1.1, this can still be a practical issue. This instead switched to "Connection: close" without a "Content-Length" header.

@s-ludwig s-ludwig merged commit 96ad591 into 0.7.x Mar 24, 2017

2 of 4 checks passed

continuous-integration/appveyor/branch AppVeyor build failed
Details
continuous-integration/appveyor/pr AppVeyor build failed
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details

@s-ludwig s-ludwig deleted the issue1721-http10-chunked branch Mar 24, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment