Skip to content

Commit

Permalink
[debops] defaults: Adopt NCSC-NL recommended curves.
Browse files Browse the repository at this point in the history
There's some concern over the rigidity of prime256v1 because
it's a NIST curve. See here for the rabbit hole:
nodejs/node#1495

secp521r1 was erroneously copied.

Since Mozilla gives no explanation why they recommend certain
curves and it included prime256v1, the curves are replaced with
NCSC-NL recommendation.
  • Loading branch information
Ellen Papsch committed Aug 6, 2020
1 parent 7cb65e1 commit 6792e91
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion ansible/roles/nginx/defaults/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -523,7 +523,7 @@ nginx_default_tls_protocols: '{{ [ "TLSv1.2", "TLSv1.3" ]
#
# See also: https://security.stackexchange.com/questions/31772/
# Set to ``False`` to disable ECC.
nginx_default_ssl_curve: 'secp521r1:secp384r1:prime256v1'
nginx_default_ssl_curve: 'secp384r1:secp256r1:x448:x25519'

# ]]]
# .. envvar:: nginx_default_ssl_verify_client [[[
Expand Down

0 comments on commit 6792e91

Please sign in to comment.