BUG/MINOR: ssl: do not try to use early data if not configured
authorWilly Tarreau <w@1wt.eu>
Wed, 3 Feb 2021 10:21:38 +0000 (11:21 +0100)
committerWilly Tarreau <w@1wt.eu>
Fri, 5 Feb 2021 07:04:02 +0000 (08:04 +0100)
commita84986ae4f27f5335aedd5216ee3f4db06db74d0
treecd717d820684f318144d2a8c2728fddc02ab08d1
parent23296f92f4abeaf0991c70200f3a2694142a10b3
BUG/MINOR: ssl: do not try to use early data if not configured

The CO_FL_EARLY_SSL_HS flag was inconditionally set on the connection,
resulting in SSL_read_early_data() always being used first in handshake
calculations. While this seems to work well (probably that there are
fallback paths inside openssl), it's particularly confusing and makes
the debugging quite complicated. It possibly is not optimal by the way.

This flag ought to be set only when early_data is configured on the bind
line. Apparently there used to be a good reason for doing it this way in
1.8 times, but it really does not make sense anymore. It may be OK to
backport this to 2.3 if this helps with troubleshooting, but better not
go too far as it's unlikely to fix any real issue while it could introduce
some in old versions.
src/ssl_sock.c