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

Make chunked transfer-encoding optional #270

Merged
merged 1 commit into from
May 3, 2016
Merged

Make chunked transfer-encoding optional #270

merged 1 commit into from
May 3, 2016

Conversation

gitblit
Copy link
Collaborator

@gitblit gitblit commented May 3, 2016

By flushing the response buffer we force chunked transfer-encoding. When we do this we lose the option to relay Content-Length to the client for HEAD requests or for setting-up client-side deterministic download progress indicators on GET requests.

This PR makes chunked transfer encoding optional and improves the HEAD response of the DirectoryHandler.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.03%) to 11.023% when pulling bafb890 on chunked into c024a56 on master.

@decebals decebals merged commit 6c6d0d7 into master May 3, 2016
@decebals
Copy link
Member

decebals commented May 3, 2016

Thanks!

@gitblit gitblit deleted the chunked branch May 3, 2016 18:19
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.

3 participants