Merge pull request #3649 from aws/zoewang/addressFeedback

Minor fixes to address feedback and update changelog file

Author: zoewangg

.changes/2.30.17.json

@@ -14,6 +14,24 @@
             "contributor": "",
             "description": "Remove unnecessary invocation of `ContentStreamProvider#newStream` when content-length is known for requests that use AWS chunked encoding."
         },
+        {
+            "type": "feature",
+            "category": "AWS SDK for Java v2",
+            "contributor": "",
+            "description": "The SDK now does not buffer input data from `RequestBody#fromInputStream` in cases where the InputStream does not support mark and reset."
+        },
+        {
+            "type": "feature",
+            "category": "AWS SDK for Java v2",
+            "contributor": "",
+            "description": "The SDK now throws exception for input streaming operation if the stream has fewer bytes (i.e. reaches EOF) before the expected length is reached."
+        },
+        {
+            "type": "feature",
+            "category": "AWS SDK for Java v2",
+            "contributor": "",
+            "description": "The SDK now does not buffer input data from ContentStreamProvider in cases where content length is known."
+        },
         {
             "type": "feature",
             "category": "AWS Database Migration Service",

.changes/next-release/feature-AWSSDKforJavav2-02883e2.json

@@ -11,6 +11,9 @@
 ## __AWS SDK for Java v2__
   - ### Features
     - Updated endpoint and partition metadata.
+    - The SDK now does not buffer input data from `RequestBody#fromInputStream` in cases where the InputStream does not support mark and reset.
+    - The SDK now throws exception for input streaming operation if the stream has fewer bytes (i.e. reaches EOF) before the expected length is reached.
+    - The SDK now does not buffer input data from ContentStreamProvider in cases where content length is known.
 
   - ### Bugfixes
     - Remove unnecessary invocation of `ContentStreamProvider#newStream` when content-length is known for requests that use AWS chunked encoding.

.changes/next-release/feature-AWSSDKforJavav2-9dffbaf.json

@@ -52,7 +52,10 @@ public int read() throws IOException {
 
         if (read != -1) {
             remaining--;
-        } else if (remaining != 0) { // EOF, ensure we've read the number of expected bytes
+        }
+
+        // EOF, ensure we've read the number of expected bytes
+        if (read == -1 && remaining > 0) {
             throw new IllegalStateException("The request content has fewer bytes than the "
                                             + "specified "
                                             + "content-length: " + length + " bytes.");
@@ -75,7 +78,7 @@ public int read(byte[] b, int off, int len) throws IOException {
         }
 
         // EOF, ensure we've read the number of expected bytes
-        if (read == -1 && remaining != 0) {
+        if (read == -1 && remaining > 0) {
             throw new IllegalStateException("The request content has fewer bytes than the "
                                             + "specified "
                                             + "content-length: " + length + " bytes.");