Skip to content

Commit ee58639

Browse files
committed
Merge branch 'download-checksums' of https://github.com/aws/aws-sdk-cpp into download-checksums
2 parents f78bd13 + e81102f commit ee58639

File tree

2 files changed

+0
-11
lines changed

2 files changed

+0
-11
lines changed

src/aws-cpp-sdk-transfer/include/aws/transfer/TransferHandle.h

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -442,7 +442,6 @@ namespace Aws
442442
mutable std::mutex m_getterSetterLock;
443443
Aws::String m_checksum;
444444
// Map of part number to Hash instance for multipart download checksum validation
445-
// TODO: Add CRT checksum combining utility when available
446445
Aws::Map<int, std::shared_ptr<Aws::Utils::Crypto::Hash>> m_partChecksums;
447446
};
448447

src/aws-cpp-sdk-transfer/source/transfer/TransferManager.cpp

Lines changed: 0 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -1281,16 +1281,6 @@ namespace Aws
12811281
Aws::IOStream* bufferStream = partState->GetDownloadPartStream();
12821282
assert(bufferStream);
12831283

1284-
// checksum for this part if validation is enabled
1285-
if (m_transferConfig.validateChecksums)
1286-
{
1287-
auto hash = handle->GetPartChecksum(partState->GetPartId());
1288-
if (hash && partState->GetDownloadBuffer())
1289-
{
1290-
hash->Update(partState->GetDownloadBuffer(), static_cast<size_t>(partState->GetSizeInBytes()));
1291-
}
1292-
}
1293-
12941284
Aws::String errMsg{handle->WritePartToDownloadStream(bufferStream, partState->GetRangeBegin())};
12951285
if (errMsg.empty()) {
12961286
handle->ChangePartToCompleted(partState, outcome.GetResult().GetETag());

0 commit comments

Comments
 (0)