search

varnishcache / varnish-cache

Varnish Cache source code repository
https://www.varnish-cache.org
Other
3.56k stars 366 forks source link

Consistently abort ESI transactions also for empty includes #4071

Closed nigoroll closed 4 months ago

nigoroll commented 4 months ago

When we introduced checking ESI includes for status 200 or 204, we would only (potentially, depending on the onerror attribute and esi_onerror_continue parameter) abort ESI processing for includes with a non-zero length.

This patch makes behavior consistent to not depend on whether or not an include is empty.

Fixes #4070