Fix in transfer manager

Author: jencymaryjoseph

services-custom/s3-transfer-manager/src/it/java/software/amazon/awssdk/transfer/s3/S3TransferManagerPresignedUrlDownloadIntegrationTest.java

@@ -128,7 +128,7 @@ void downloadFileWithPresignedUrl_progressTracking(String tmType, S3TransferMana
         PresignedUrlDownloadRequest.Builder requestBuilder = PresignedUrlDownloadRequest.builder()
                                                                                         .presignedUrl(createPresignedRequest(key).url());
         if (range != null) {
-            requestBuilder.range(range);
+            requestBuilder.putHeader("Range", range);
         }
 
         PresignedFileDownload download = tm.downloadFileWithPresignedUrl(
@@ -159,7 +159,7 @@ void downloadWithPresignedUrl_toBytes_progressTracking(String tmType, S3Transfer
         PresignedUrlDownloadRequest.Builder requestBuilder = PresignedUrlDownloadRequest.builder()
                                                                                         .presignedUrl(createPresignedRequest(key).url());
         if (range != null) {
-            requestBuilder.range(range);
+            requestBuilder.putHeader("Range", range);
         }
 
         Download<ResponseBytes<GetObjectResponse>> download = tm.downloadWithPresignedUrl(

services-custom/s3-transfer-manager/src/main/java/software/amazon/awssdk/transfer/s3/internal/GenericS3TransferManager.java

@@ -615,7 +615,7 @@ public final PresignedFileDownload downloadFileWithPresignedUrl(PresignedDownloa
         progressUpdater.transferInitiated();
 
         responseTransformer = isS3ClientMultipartEnabled()
-                              && presignedDownloadFileRequest.presignedUrlDownloadRequest().range() == null
+                              && !presignedDownloadFileRequest.presignedUrlDownloadRequest().headers().containsKey("Range")
                               ? progressUpdater.wrapForNonSerialFileDownload(
                                   responseTransformer, GetObjectRequest.builder().build())
                               : progressUpdater.wrapResponseTransformer(responseTransformer);
@@ -651,7 +651,7 @@ public final <ResultT> Download<ResultT> downloadWithPresignedUrl(
         progressUpdater.transferInitiated();
 
         responseTransformer = isS3ClientMultipartEnabled()
-                              && presignedDownloadRequest.presignedUrlDownloadRequest().range() == null
+                              && !presignedDownloadRequest.presignedUrlDownloadRequest().headers().containsKey("Range")
                               ? progressUpdater.wrapForNonSerialFileDownload(
                                   responseTransformer, GetObjectRequest.builder().build())
                               : progressUpdater.wrapResponseTransformer(responseTransformer);

services-custom/s3-transfer-manager/src/test/java/software/amazon/awssdk/transfer/s3/internal/S3TransferManagerPresignedUrlListenerWiremockTest.java

@@ -119,7 +119,7 @@ void presignedUrlDownload_shouldInvokeListener(boolean multipartEnabled, String
         PresignedUrlDownloadRequest.Builder requestBuilder = PresignedUrlDownloadRequest.builder()
                                                                                        .presignedUrl(presignedUrl);
         if (range != null) {
-            requestBuilder.range(range);
+            requestBuilder.putHeader("Range", range);
         }
 
         if ("toFile".equals(type)) {