BUG/MINOR: cfgparse: make sure to include openssl-compat
authorWilly Tarreau <w@1wt.eu>
Wed, 19 Apr 2023 08:41:55 +0000 (10:41 +0200)
committerWilly Tarreau <w@1wt.eu>
Wed, 19 Apr 2023 08:46:21 +0000 (10:46 +0200)
commitbe336620b7cee3ebf06186352748e1ebac5830ae
tree32ddf5a6436e74f45ee0bc0d260f817c1fd612c5
parent89e48ff92f3ac7645a4d90fc352b88aba8af802a
BUG/MINOR: cfgparse: make sure to include openssl-compat

Commit 5003ac7fe ("MEDIUM: config: set useful ALPN defaults for HTTPS
and QUIC") revealed a build dependency bug: if QUIC is not enabled,
cfgparse doesn't have any dependency on the SSL stack, so the various
ifdefs that try to check special conditions such as rejecting an H2
config with too small a bufsize, are silently ignored. This was
detected because the default ALPN string was not set and caused the
alpn regtest to fail without QUIC support. Adding openssl-compat to
the list of includes seems to be sufficient to have what we need.

It's unclear when this dependency was broken, it seems that even 2.2
didn't have an explicit dependency on anything SSL-related, though it
could have been inherited through other files (as happens with QUIC
here). It would be safe to backport it to all stable branches. The
impact is very low anyway.
src/cfgparse.c