diff options
author | Florian Klink <flokli@flokli.de> | 2024-01-31T17·03+0200 |
---|---|---|
committer | clbot <clbot@tvl.fyi> | 2024-02-02T16·26+0000 |
commit | 9504015031a7299f22a9827ff0eded74a95c66f8 (patch) | |
tree | 279d5e62003b31731c531336b076be8cdeeab229 /tvix/castore/src/blobservice | |
parent | 5ad5a0da00622beb713fb85520821212416b02f9 (diff) |
feat(tvix/castore/blobsvc): validate StatBlobResponse r/7470
All chunks must have valid blake3 digests. It is allowed to send an empty list, if no more granular chunking is available. Change-Id: I7ecb53579cdf40fd938bb68a85685751b4d3626f Reviewed-on: https://cl.tvl.fyi/c/depot/+/10726 Tested-by: BuildkiteCI Reviewed-by: Connor Brewster <cbrewster@hey.com> Autosubmit: flokli <flokli@flokli.de>
Diffstat (limited to 'tvix/castore/src/blobservice')
-rw-r--r-- | tvix/castore/src/blobservice/grpc.rs | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/tvix/castore/src/blobservice/grpc.rs b/tvix/castore/src/blobservice/grpc.rs index acc0125c82ed..d98a9b517724 100644 --- a/tvix/castore/src/blobservice/grpc.rs +++ b/tvix/castore/src/blobservice/grpc.rs @@ -129,6 +129,10 @@ impl BlobService for GRPCBlobService { Err(e) => Err(io::Error::new(io::ErrorKind::Other, e)), Ok(resp) => { let resp = resp.into_inner(); + + resp.validate() + .map_err(|e| std::io::Error::new(io::ErrorKind::InvalidData, e))?; + if resp.chunks.is_empty() { warn!("chunk list is empty"); } |