Skip to content

[9.2.0] Verify the whole blob when downloading chunks (#29435) (#29593)#29615

Open
iancha1992 wants to merge 1 commit into
bazelbuild:release-9.2.0from
iancha1992:cp29514
Open

[9.2.0] Verify the whole blob when downloading chunks (#29435) (#29593)#29615
iancha1992 wants to merge 1 commit into
bazelbuild:release-9.2.0from
iancha1992:cp29514

Conversation

@iancha1992
Copy link
Copy Markdown
Member

#29588 fails CI. This fixes it.

This is a bug fix for the chunking implementation, where we should verify the hash of the full blob if verify downloads is specified, so we don't blindly trust SplitBlob responses.

Description

Motivation

Build API Changes

No

Checklist

  • I have added tests for the new use cases (if any).
  • I have updated the documentation (if applicable).

Release Notes

RELNOTES: None

Commit 37237ad

…) (bazelbuild#29593)

bazelbuild#29588 fails CI. This fixes it.

This is a bug fix for the chunking implementation, where we should
verify the hash of the full blob if verify downloads is specified, so we
don't blindly trust SplitBlob responses.
@iancha1992 iancha1992 requested review from Wyverald and tjgq May 21, 2026 19:10
@iancha1992 iancha1992 requested a review from a team as a code owner May 21, 2026 19:10
@iancha1992 iancha1992 added team-Remote-Exec Issues and PRs for the Execution (Remote) team awaiting-review PR is awaiting review from an assigned reviewer labels May 21, 2026
@iancha1992 iancha1992 enabled auto-merge May 21, 2026 19:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting-review PR is awaiting review from an assigned reviewer team-Remote-Exec Issues and PRs for the Execution (Remote) team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants