From 156ec1a280a9c7d4f4a294a6d070be17195f63b1 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?J=C3=BCrg=20Billeter?= Date: Fri, 7 Mar 2025 15:10:49 +0100 Subject: [PATCH] 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 @@ def __init__(self, casd, *, enable_push): 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