Skip to content

Commit af437c6

Browse files
alexjfisherehelms
authored andcommitted
Add TLS1.3 ciphers to default server_cipher_suites
In 2835ba2 the default `server_ssl_protocols` was expanded to include TLS 1.3. This had no affect for users not overriding `server_cipher_suites` though as no TLS 1.3 cipher suites were included by default. This commit adds `TLS_AES_128_GCM_SHA256` and `TLS_AES_256_GCM_SHA384` which are both NIST approved.
1 parent 041a646 commit af437c6

File tree

2 files changed

+6
-2
lines changed

2 files changed

+6
-2
lines changed

manifests/init.pp

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -400,8 +400,10 @@
400400
# Defaults to "${ssl_dir}/ca/ca_crt.pem"
401401
#
402402
# $server_cipher_suites:: List of SSL ciphers to use in negotiation
403-
# Defaults to [ 'TLS_RSA_WITH_AES_256_CBC_SHA256', 'TLS_RSA_WITH_AES_256_CBC_SHA',
404-
# 'TLS_RSA_WITH_AES_128_CBC_SHA256', 'TLS_RSA_WITH_AES_128_CBC_SHA', ]
403+
# Defaults to ['TLS_AES_128_GCM_SHA256', 'TLS_AES_256_GCM_SHA384',
404+
# 'TLS_DHE_RSA_WITH_AES_128_GCM_SHA256', 'TLS_DHE_RSA_WITH_AES_256_GCM_SHA384',
405+
# 'TLS_ECDHE_ECDSA_WITH_AES_128_GCM_SHA256', 'TLS_ECDHE_ECDSA_WITH_AES_256_GCM_SHA384',
406+
# 'TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256', 'TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384']
405407
#
406408
# $server_ruby_load_paths:: List of ruby paths
407409
#

manifests/params.pp

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -386,6 +386,8 @@
386386
$server_admin_api_whitelist = ['localhost', $lower_fqdn]
387387
$server_ca_client_whitelist = ['localhost', $lower_fqdn]
388388
$server_cipher_suites = [
389+
'TLS_AES_128_GCM_SHA256',
390+
'TLS_AES_256_GCM_SHA384',
389391
'TLS_DHE_RSA_WITH_AES_128_GCM_SHA256',
390392
'TLS_DHE_RSA_WITH_AES_256_GCM_SHA384',
391393
'TLS_ECDHE_ECDSA_WITH_AES_128_GCM_SHA256',

0 commit comments

Comments
 (0)