Skip to content

Conversation

@kyleknap
Copy link
Contributor

For uploads, the CRT S3 client will add CRC32 trailing checksums. For downloads, the CRT S3 client will validate checksums associated to the object when possible.

This is a port from this CLI v2 PR: aws/aws-cli#8298

For uploads, the CRT S3 client will add CRC32 trailing checksums.
For downloads, the CRT S3 client will validate checksums
associated to the object when possible.
@kyleknap kyleknap requested a review from nateprewitt November 13, 2023 22:01
@codecov-commenter
Copy link

Codecov Report

Attention: 13 lines in your changes are missing coverage. Please review.

Comparison is base (b8906b3) 84.97% compared to head (d85a0fa) 84.57%.

Files Patch % Lines
s3transfer/crt.py 0.00% 13 Missing ⚠️

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@                 Coverage Diff                  @@
##           crt-enhancements     #280      +/-   ##
====================================================
- Coverage             84.97%   84.57%   -0.41%     
====================================================
  Files                    16       16              
  Lines                  2736     2749      +13     
====================================================
  Hits                   2325     2325              
- Misses                  411      424      +13     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Contributor

@nateprewitt nateprewitt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@kyleknap kyleknap merged commit db20da4 into boto:crt-enhancements Nov 13, 2023
@kyleknap kyleknap deleted the crt-checksums branch November 13, 2023 22:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants