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

Fix: send part's checksum for XML MPU part upload by chandra-siri · Pull Request #1529 · googleapis/python-storage · GitHub

This repository was archived by the owner on Mar 31, 2026. It is now read-only.
/ python-storage Public archive

Fix: send part's checksum for XML MPU part upload - #1529

Merged
chandra-siri merged 18 commits into
googleapis:mainfrom
chandra-siri:fix/send_part_checksum_for_XML_MPU
Aug 22, 2025
ConversationCommits (18)ChecksFiles changed
Merged

Fix: send part's checksum for XML MPU part upload#1529
chandra-siri merged 18 commits into
googleapis:mainfrom
chandra-siri:fix/send_part_checksum_for_XML_MPU

Conversation

chandra-siri commented Aug 21, 2025 •
edited
Loading

Copy link
Copy Markdown
Collaborator
  • For XML MPU part upload checksum of the part should be sent to the server using x-goog-hash header . Docs
  • The server will throw 400 InvalidDigest/BadDigest/CrcMismatch in difference scenarios; but all of them essentially mean the same - "The checksum provided by user didn't match the one calculated by us"

product-auto-label Bot added size: l Pull request size is large. api: storage Issues related to the googleapis/python-storage API. labels Aug 21, 2025
product-auto-label Bot added size: m Pull request size is medium. and removed size: l Pull request size is large. labels Aug 22, 2025
chandra-siri marked this pull request as ready for review August 22, 2025 08:50
chandra-siri requested review from a team August 22, 2025 08:50
shubham-up-47 reviewed Aug 22, 2025
View reviewed changes
Comment thread tests/resumable_media/unit/test__upload.py Outdated
Comment thread tests/resumable_media/unit/requests/test_upload.py
Comment thread google/cloud/storage/_media/requests/upload.py
chandra-siri added the owlbot:run Add this label to trigger the Owlbot post processor. label Aug 22, 2025
gcf-owl-bot Bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Aug 22, 2025
chandra-siri merged commit 2ad77c7 into googleapis:main Aug 22, 2025
15 checks passed
release-please Bot mentioned this pull request Aug 22, 2025
Merged
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
Show hidden characters
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

1 more reviewer
shubham-up-47 shubham-up-47 approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

v-pratap

shubham-up-47

Labels

api: storage Issues related to the googleapis/python-storage API. size: m Pull request size is medium.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

Back | FazBrowse Home | New Git URL