From 99cb130c0969688ad20a8af81ab125107568342e Mon Sep 17 00:00:00 2001 From: Piotr Sikora Date: Wed, 11 Nov 2020 18:02:18 -0800 Subject: [PATCH] Always process HTTP headers before processing HTTP body. (#95) Signed-off-by: Piotr Sikora --- src/context.cc | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/src/context.cc b/src/context.cc index 89daa97a..c33027e1 100644 --- a/src/context.cc +++ b/src/context.cc @@ -629,6 +629,12 @@ FilterHeadersStatus ContextBase::convertVmCallResultToFilterHeadersStatus(uint64 result > static_cast(FilterHeadersStatus::StopAllIterationAndWatermark)) { return FilterHeadersStatus::StopAllIterationAndWatermark; } + if (result == static_cast(FilterHeadersStatus::StopIteration)) { + // Always convert StopIteration (pause processing headers, but continue processing body) + // to StopAllIterationAndWatermark (pause all processing), since the former breaks all + // assumptions about HTTP processing. + return FilterHeadersStatus::StopAllIterationAndWatermark; + } return static_cast(result); }