On Wed, Apr 01, 2020 at 09:24:27AM +0200, Ruediger Pluem wrote: > I have checked socket, pipe and cgi buckets and none of them seem to return > EOF. In case they hit EOF they replace themselves with > an immortal bucket of length 0. So above seems just an additional safety if a > (future) morphing bucket behaves differently and > returns EOF, but with the current code that path should not be hit really.
Yeah, the "read till EOF" is an implementation detail for those bucket types, I would very strongly argue if they ever exposed EOF on read() it would be a bucket type bug. It could quite possibly obscure a bug elsewhere if filters ignored EOF on read() so I don't think that should be recommended.