Skip to content

Commit f261719

Browse files
committed
chore: remove NoRetries body from high-level async MPU
1 parent 034bb89 commit f261719

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

src/main/java/software/amazon/encryption/s3/S3AsyncEncryptionClient.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,6 @@
3333
import software.amazon.awssdk.services.s3.multipart.MultipartConfiguration;
3434
import software.amazon.encryption.s3.internal.GetEncryptedObjectPipeline;
3535
import software.amazon.encryption.s3.internal.InstructionFileConfig;
36-
import software.amazon.encryption.s3.internal.NoRetriesAsyncRequestBody;
3736
import software.amazon.encryption.s3.internal.PutEncryptedObjectPipeline;
3837
import software.amazon.encryption.s3.materials.AesKeyring;
3938
import software.amazon.encryption.s3.materials.CryptographicMaterialsManager;
@@ -165,8 +164,9 @@ private CompletableFuture<PutObjectResponse> multipartPutObject(PutObjectRequest
165164
.secureRandom(_secureRandom)
166165
.build();
167166
// Ensures parts are not retried to avoid corrupting ciphertext
168-
AsyncRequestBody noRetryBody = new NoRetriesAsyncRequestBody(requestBody);
169-
return pipeline.putObject(putObjectRequest, noRetryBody);
167+
// TODO: We might not need this - double check...
168+
// AsyncRequestBody noRetryBody = new NoRetriesAsyncRequestBody(requestBody);
169+
return pipeline.putObject(putObjectRequest, requestBody);
170170
}
171171

172172
/**

0 commit comments

Comments
 (0)