FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

fix: update PCU request building logic to properly clear crc32c and md5 by BenWhitehead · Pull Request #3323 · googleapis/java-storage · GitHub

fix: update PCU request building logic to properly clear crc32c and md5 - #3323

Merged
BenWhitehead merged 1 commit into
mainfrom
pcu/checksum-fix
Oct 3, 2025
Merged

fix: update PCU request building logic to properly clear crc32c and md5#3323
BenWhitehead merged 1 commit into
mainfrom
pcu/checksum-fix

Conversation

Copy link
Copy Markdown
Collaborator

If a PCU is initiated with a BlobInfo from the current generation of an object, the crc32c and md5 will be included in all compose requests. Update the code to properly clear when necessary, and to only apply a provided value to the final compose if crc32cMatch or md5Match were specified.

If a PCU is initiated with a BlobInfo from the current generation of an object, the crc32c and md5 will be included in all compose requests. Update the code to properly clear when necessary, and to only apply a provided value to the final compose if crc32cMatch or md5Match were specified.
BenWhitehead requested a review from a team October 2, 2025 22:49
product-auto-label Bot added size: s Pull request size is small. api: storage Issues related to the googleapis/java-storage API. labels Oct 2, 2025
BenWhitehead merged commit 4da9f31 into main Oct 3, 2025
25 checks passed
BenWhitehead deleted the pcu/checksum-fix branch October 3, 2025 17:03
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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 googleapis/java-storage API. size: s Pull request size is small.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants


Back | FazBrowse Home | New Git URL