You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hc.apache.org by ol...@apache.org on 2015/01/27 11:35:08 UTC
svn commit: r1655001 - in /httpcomponents/httpclient/trunk/httpclient/src:
main/java/org/apache/http/client/entity/EntityBuilder.java
test/java/org/apache/http/client/entity/TestEntityBuilder.java
Author: olegk
Date: Tue Jan 27 10:35:08 2015
New Revision: 1655001
URL: http://svn.apache.org/r1655001
Log:
HTTPCLIENT-1605: EntityBuilder sets incorrect content length for entities backed by InputStream
Modified:
httpcomponents/httpclient/trunk/httpclient/src/main/java/org/apache/http/client/entity/EntityBuilder.java
httpcomponents/httpclient/trunk/httpclient/src/test/java/org/apache/http/client/entity/TestEntityBuilder.java
Modified: httpcomponents/httpclient/trunk/httpclient/src/main/java/org/apache/http/client/entity/EntityBuilder.java
URL: http://svn.apache.org/viewvc/httpcomponents/httpclient/trunk/httpclient/src/main/java/org/apache/http/client/entity/EntityBuilder.java?rev=1655001&r1=1655000&r2=1655001&view=diff
==============================================================================
--- httpcomponents/httpclient/trunk/httpclient/src/main/java/org/apache/http/client/entity/EntityBuilder.java (original)
+++ httpcomponents/httpclient/trunk/httpclient/src/main/java/org/apache/http/client/entity/EntityBuilder.java Tue Jan 27 10:35:08 2015
@@ -317,7 +317,7 @@ public class EntityBuilder {
} else if (this.binary != null) {
e = new ByteArrayEntity(this.binary, getContentOrDefault(ContentType.DEFAULT_BINARY));
} else if (this.stream != null) {
- e = new InputStreamEntity(this.stream, 1, getContentOrDefault(ContentType.DEFAULT_BINARY));
+ e = new InputStreamEntity(this.stream, -1, getContentOrDefault(ContentType.DEFAULT_BINARY));
} else if (this.parameters != null) {
e = new UrlEncodedFormEntity(this.parameters,
this.contentType != null ? this.contentType.getCharset() : null);
Modified: httpcomponents/httpclient/trunk/httpclient/src/test/java/org/apache/http/client/entity/TestEntityBuilder.java
URL: http://svn.apache.org/viewvc/httpcomponents/httpclient/trunk/httpclient/src/test/java/org/apache/http/client/entity/TestEntityBuilder.java?rev=1655001&r1=1655000&r2=1655001&view=diff
==============================================================================
--- httpcomponents/httpclient/trunk/httpclient/src/test/java/org/apache/http/client/entity/TestEntityBuilder.java (original)
+++ httpcomponents/httpclient/trunk/httpclient/src/test/java/org/apache/http/client/entity/TestEntityBuilder.java Tue Jan 27 10:35:08 2015
@@ -71,6 +71,7 @@ public class TestEntityBuilder {
Assert.assertNotNull(entity);
Assert.assertNotNull(entity.getContent());
Assert.assertNotNull(entity.getContentType());
+ Assert.assertEquals(-1, entity.getContentLength());
Assert.assertEquals("application/octet-stream", entity.getContentType().getValue());
}