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());
     }