BUG/MINOR: cache: Remove incomplete entries from the cache when stream is closed
authorRemi Tricot-Le Breton <rlebreton@haproxy.com>
Tue, 28 Nov 2023 16:08:56 +0000 (17:08 +0100)
committerWilly Tarreau <w@1wt.eu>
Tue, 28 Nov 2023 16:18:48 +0000 (17:18 +0100)
commit23c810d04244166a1a205de510b898cf8bba2a5c
treefd0890c72e26aa3ad59add5709f0a9d38c44e19d
parentad61a5dde34cab4fb05da005b10e772f4ea8899f
BUG/MINOR: cache: Remove incomplete entries from the cache when stream is closed

When a stream is interrupted by the client before the full answer is
stored in the cache, we end up with an incomplete entry in the cache
that cannot be overwritten until it "naturally" expires. In such a case,
we call the cache filter's cache_store_strm_deinit callback without ever
calling cache_store_http_end which means that the 'complete' flag is
never set on the concerned cache_entry.
This patch adds a check on the 'complete' flag in the strm_deinit
callback and removes the entry from the cache if it is incomplete.

A way to exhibit this bug is to try to get the same "big" response on
multiple clients at the same time thanks to h2load for instance, and to
interrupt the client side before the answer can be fully stored in the
cache.

This patch can be backported up to 2.4 but it will need some rework
starting with branch 2.8 because of the latest cache changes.
src/cache.c