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:06:56 +0000 (08:06 +0100)
commit5930fb23ef9a3dc148c5ea70c260357f1f93ecf6
tree15c974487271c9ed5bb86a158de885e8cf4165bc
parent59ad20e080aa9dd9a197c074b18850b99c94b050
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.

(cherry picked from commit a84986ae4f27f5335aedd5216ee3f4db06db74d0)
Signed-off-by: Willy Tarreau <w@1wt.eu>
src/ssl_sock.c