Skip to content

feat(storage): Set CRC32C as the default checksum option#7547

Open
thiyaguk09 wants to merge 2 commits intogoogleapis:mainfrom
thiyaguk09:feat/default-crc32c
Open

feat(storage): Set CRC32C as the default checksum option#7547
thiyaguk09 wants to merge 2 commits intogoogleapis:mainfrom
thiyaguk09:feat/default-crc32c

Conversation

@thiyaguk09
Copy link
Contributor

@thiyaguk09 thiyaguk09 commented Mar 10, 2026

This pull request significantly enhances the data integrity features of the transfer manager by introducing CRC32C checksum validation. It updates the system to fully support CRC32C as a validation option and configures it as the default for file uploads, thereby strengthening data consistency checks during transfer operations.

Highlights

  • New Checksum Option: The crc32c checksum option has been added as a valid validation method for file uploads.
  • Default Validation: CRC32C is now set as the default checksum validation for TransferManager uploads when no specific validation option is provided.
  • CRC32C Implementation: Logic was implemented to calculate and include the x-goog-hash header with the crc32c value during multipart uploads.
  • Enhanced Testing: New unit tests were added to ensure that both explicit and default crc32c validation behaviors function as expected.

This is the relevant ticket: b/428117019

thiyaguk09 and others added 2 commits March 6, 2026 15:36
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
@product-auto-label product-auto-label bot added the api: storage Issues related to the Cloud Storage API. label Mar 10, 2026
@thiyaguk09 thiyaguk09 marked this pull request as ready for review March 10, 2026 08:34
@thiyaguk09 thiyaguk09 requested a review from a team as a code owner March 10, 2026 08:34
@quirogas quirogas added the priority: p2 Moderately-important priority. Fix may not be included in next release. label Mar 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: storage Issues related to the Cloud Storage API. priority: p2 Moderately-important priority. Fix may not be included in next release.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants