BUG/MINOR: stream: Fix test on channels flags to set clientfin/serverfin touts
authorChristopher Faulet <cfaulet@haproxy.com>
Tue, 4 Apr 2023 08:16:57 +0000 (10:16 +0200)
committerChristopher Faulet <cfaulet@haproxy.com>
Wed, 5 Apr 2023 06:46:06 +0000 (08:46 +0200)
commit947b2e59223b53110c435ecc4d1afd75f97f1129
treef4463fdeed2fe4b9cc13b0c4ff7d7d5ed4409613
parentc665bb5637959be282226bd55ce372e1d2244db3
BUG/MINOR: stream: Fix test on channels flags to set clientfin/serverfin touts

There is a bug in a way the channels flags are checked to set clientfin or
serverfin timeout. Indeed, to set the clientfin timeout, the request channel
must be shut for reads (CF_SHUTR) or the response channel must be shut for
writes (CF_SHUTW). As the opposite, the serverfin timeout must be set when
the request channel is shut for writes (CF_SHUTW) or the response channel is
shut for reads (CF_SHUTR).

It is a 2.8-dev specific issue. No backport needed.
src/stream.c