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

set Content-Length on static file responses #102

Merged
merged 1 commit into from Jan 24, 2014

Conversation

2 participants
@zuercher

zuercher commented Jan 24, 2014

I also removed the extraneous call to InputStream.read -- IOUtils.toByteArray will already have consumed the entire InputStream. Also, if there was anything left, the read call would pave over the start of the byte array.

capotej added a commit that referenced this pull request Jan 24, 2014

Merge pull request #102 from zuercher/master
set Content-Length on static file responses

@capotej capotej merged commit 43b244a into twitter:master Jan 24, 2014

1 check passed

default The Travis CI build passed
Details
@capotej

This comment has been minimized.

Contributor

capotej commented Jan 24, 2014

Thanks!

cacoco pushed a commit that referenced this pull request May 13, 2015

Merge pull request #102 from zuercher/master
set Content-Length on static file responses
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment