tree d2e730a8451bb84345d50f41d8683938c5cbed96
parent 6b1379fb8a6a3ecfbc7180d5085e03681e3fef87
author Willy Tarreau <w@1wt.eu> 1542552370 +0100
committer Willy Tarreau <w@1wt.eu> 1542573704 +0100
encoding latin1

MINOR: stream-int: relax the forwarding rules in stream_int_notify()

There currently is an optimization in stream_int_notify() consisting
in not trying to forward small bits of data if extra data remain to be
processed. The purpose is to avoid forwarding one chunk at a time if
multiple chunks are available to be parsed at once. It consists in
avoiding sending pending output data if there are still data to be
parsed in the channel's buffer, since process_stream() will have the
opportunity to deal with them all at once.

Not only this optimization is less useful with the new way the connections
work, but it even causes problems like lost events since WAIT_ROOM will
not be removed. And with HTX, it will never be able to update the input
buffer after the first read.

Let's relax the rules now, by always sending if we don't have the
CF_EXPECT_MORE flag (used to group writes), or if the buffer is
already full.