BUG/MINOR: cache: also consider CF_SHUTR to abort delivery
The cache runs in an applet, so it delivers data into the input side
of the channel's buffer. Thus it must also abort feeding the buffer
as soon as CF_SHUTR is present, not just CF_SHUTW*, since these last
ones may only appear later. There doesn't seem to be an observable
side effect of this bug, the fix probably doesn't even need to be
backported.
diff --git a/src/cache.c b/src/cache.c
index c3d44f5..0322552 100644
--- a/src/cache.c
+++ b/src/cache.c
@@ -961,7 +961,7 @@
goto out;
}
- if (res->flags & (CF_SHUTW|CF_SHUTW_NOW))
+ if (res->flags & (CF_SHUTW|CF_SHUTR|CF_SHUTW_NOW))
appctx->st0 = HTX_CACHE_END;
if (appctx->st0 == HTX_CACHE_INIT) {
@@ -1190,7 +1190,7 @@
goto out;
}
- if (res->flags & (CF_SHUTW|CF_SHUTW_NOW))
+ if (res->flags & (CF_SHUTW|CF_SHUTW_NOW|CF_SHUTR))
appctx->st0 = HTTP_CACHE_END;
/* buffer are aligned there, should be fine */