This is an automated email from the ASF dual-hosted git repository. juergbi pushed a commit to branch juerg/test-proxy-server in repository https://gitbox.apache.org/repos/asf/buildstream.git
commit 156ec1a280a9c7d4f4a294a6d070be17195f63b1 Author: Jürg Billeter <[email protected]> AuthorDate: Fri Mar 7 15:10:49 2025 +0100 casserver.py: Fix error handling in `ByteStream.Read()` This proxy server is only used in tests, so this doesn't affect any real use of BuildStream. --- src/buildstream/_cas/casserver.py | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/src/buildstream/_cas/casserver.py b/src/buildstream/_cas/casserver.py index 6570df75f..72340ad7c 100644 --- a/src/buildstream/_cas/casserver.py +++ b/src/buildstream/_cas/casserver.py @@ -135,10 +135,9 @@ class _ByteStreamServicer(bytestream_pb2_grpc.ByteStreamServicer): def Read(self, request, context): self.logger.debug("Reading %s", request.resource_name) try: - ret = self.bytestream.Read(request) + yield from self.bytestream.Read(request) except grpc.RpcError as err: context.abort(err.code(), err.details()) - return ret def Write(self, request_iterator, context): # Note that we can't easily give more information because the
