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

Add Content-Length header as part of building the request. #82

Merged
merged 1 commit into from Jan 3, 2014
Merged

Add Content-Length header as part of building the request. #82

merged 1 commit into from Jan 3, 2014

Conversation

@BenWhitehead
Copy link
Contributor

@BenWhitehead BenWhitehead commented Jan 1, 2014

Also add organization to build.sbt to allow for local publishing of snapshot artifact for testing.

Add organization to build.sbt to allow for local publishing of snapshot artifact for testing.
capotej added a commit that referenced this pull request Jan 3, 2014
Add Content-Length header as part of building the request.
@capotej capotej merged commit 4f1eede into twitter:sbt Jan 3, 2014
1 check passed
1 check passed
default The Travis CI build passed
Details
@capotej
Copy link
Contributor

@capotej capotej commented Jan 3, 2014

Awesome! Was under the impression finagle-http did this for us already.

@BenWhitehead
Copy link
Contributor Author

@BenWhitehead BenWhitehead commented Jan 3, 2014

I figured it did it as well, but ended up noticing it not being there when doing some response header inspection for a service I'm building on finatra.

@BenWhitehead BenWhitehead deleted the BenWhitehead:content-length branch Jan 7, 2014
cacoco pushed a commit that referenced this pull request May 13, 2015
Add Content-Length header as part of building the request.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants