MINOR: channel: don't unset CF_SHUTR_NOW after shutting down.
authorWilly Tarreau <w@1wt.eu>
Mon, 25 Mar 2019 17:35:05 +0000 (18:35 +0100)
committerWilly Tarreau <w@1wt.eu>
Mon, 25 Mar 2019 17:35:05 +0000 (18:35 +0100)
commit65e04eb2bb68f508b778ca6b8a8e934307c6537d
treeeca4aad3af1a6e6c2f330acec21609c20bef67e1
parentf882a85da4d56fba7db1c8b118df87225cd272e8
MINOR: channel: don't unset CF_SHUTR_NOW after shutting down.

This flag is set by the stream layer to request an abort, and results in
CF_SHUTR being set once the abort is performed. However by analogy with
the send side, the flag was removed once the CF_SHUTR flag was set, thus
we lose the information about the cause of the shutr. This is what creates
the confusion that sometimes arises between client and server aborts.

This patch makes sure we don't remove this flag anymore in this case.
All call places only use it to perform the shutr and already check it
against CF_SHUTR. So no condition needs to be updated to take this into
account.

Some later, more careful changes may consist in refining the conditions
where we report a client reset or a server reset to ignore SHUTR when
SHUTR_NOW is set so that we don't report such misleading information
anymore.
src/stream_interface.c