BUG/MINOR: ssl: default settings for ssl server options are not used
authorJerome Magnin <jmagnin@haproxy.com>
Wed, 22 Apr 2020 09:40:18 +0000 (11:40 +0200)
committerWilly Tarreau <w@1wt.eu>
Fri, 1 May 2020 10:11:13 +0000 (12:11 +0200)
commitbd0ace790d81fcbd3331f34db500bedb41fc4a89
tree195651eaf4a4d925f337dd5264b66666ad5f29d6
parent5a8526222f2a905d5fae66586e83b8b7d53640ae
BUG/MINOR: ssl: default settings for ssl server options are not used

Documentation states that default settings for ssl server options can be set
using either ssl-default-server-options or default-server directives. In practice,
not all ssl server options can have default values, such as ssl-min-ver, ssl-max-ver,
etc..

This patch adds the missing ssl options in srv_ssl_settings_cpy() and srv_parse_ssl(),
making it possible to write configurations like the following examples, and have them
behave as expected.

   global
     ssl-default-server-options ssl-max-ver TLSv1.2

   defaults
     mode http

   listen l1
     bind 1.2.3.4:80
     default-server ssl verify none
     server s1 1.2.3.5:443

   listen l2
     bind 2.2.3.4:80
     default-server ssl verify none ssl-max-ver TLSv1.3 ssl-min-ver TLSv1.2
     server s1 1.2.3.6:443

This should be backported as far as 1.8.
This fixes issue #595.

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