Skip to content

[9.2.0] Verify the whole blob when downloading chunks (https://github.com/bazelbuild/bazel/pull/29435)#29514

Closed
bazel-io wants to merge 2 commits into
bazelbuild:release-9.2.0from
bazel-io:cp29435-9.2.0-180701
Closed

[9.2.0] Verify the whole blob when downloading chunks (https://github.com/bazelbuild/bazel/pull/29435)#29514
bazel-io wants to merge 2 commits into
bazelbuild:release-9.2.0from
bazel-io:cp29435-9.2.0-180701

Conversation

@bazel-io
Copy link
Copy Markdown
Member

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.

Closes #29435.

PiperOrigin-RevId: 913555501
Change-Id: I7ffd600e53e585ad946f15c89201382343992d7e

Commit e87f54e

@bazel-io bazel-io added team-Remote-Exec Issues and PRs for the Execution (Remote) team awaiting-review PR is awaiting review from an assigned reviewer labels May 11, 2026
@bazel-io bazel-io requested a review from a team as a code owner May 11, 2026 18:07
@bazel-io bazel-io requested review from fmeum and tjgq May 11, 2026 18:07
@iancha1992 iancha1992 removed the request for review from fmeum May 11, 2026 18:09
@iancha1992 iancha1992 enabled auto-merge May 11, 2026 18:09
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.

Closes bazelbuild#29435.

PiperOrigin-RevId: 913555501
Change-Id: I7ffd600e53e585ad946f15c89201382343992d7e
@iancha1992 iancha1992 force-pushed the cp29435-9.2.0-180701 branch from 02f2453 to c2bcf3a Compare May 19, 2026 19:13
@iancha1992
Copy link
Copy Markdown
Member

Closing in favor of: #29615

@iancha1992 iancha1992 closed this May 21, 2026
auto-merge was automatically disabled May 21, 2026 19:10

Pull request was closed

@github-actions github-actions Bot removed the awaiting-review PR is awaiting review from an assigned reviewer label May 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

team-Remote-Exec Issues and PRs for the Execution (Remote) team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants