diff options
author | Maxim Dounin <mdounin@mdounin.ru> | 2018-08-07 02:15:28 +0300 |
---|---|---|
committer | Maxim Dounin <mdounin@mdounin.ru> | 2018-08-07 02:15:28 +0300 |
commit | 9f30fda1c2e24058e91a8c637c0717b32be399da (patch) | |
tree | 091374f7e7b2e79d7a037118fab763187f42d9a8 /src/http/modules/ngx_http_ssl_module.h | |
parent | a6baf5e508cad9d3561ed71a637127deb1e3f40d (diff) | |
download | nginx-9f30fda1c2e24058e91a8c637c0717b32be399da.tar.gz nginx-9f30fda1c2e24058e91a8c637c0717b32be399da.zip |
SSL: enabled TLSv1.3 with BoringSSL.
BoringSSL currently requires SSL_CTX_set_max_proto_version(TLS1_3_VERSION)
to be able to enable TLS 1.3. This is because by default max protocol
version is set to TLS 1.2, and the SSL_OP_NO_* options are merely used
as a blacklist within the version range specified using the
SSL_CTX_set_min_proto_version() and SSL_CTX_set_max_proto_version()
functions.
With this change, we now call SSL_CTX_set_max_proto_version() with an
explicit maximum version set. This enables TLS 1.3 with BoringSSL.
As a side effect, this change also limits maximum protocol version to
the newest protocol we know about, TLS 1.3. This seems to be a good
change, as enabling unknown protocols might have unexpected results.
Additionally, we now explicitly call SSL_CTX_set_min_proto_version()
with 0. This is expected to help with Debian system-wide default
of MinProtocol set to TLSv1.2, see
http://mailman.nginx.org/pipermail/nginx-ru/2017-October/060411.html.
Note that there is no SSL_CTX_set_min_proto_version macro in BoringSSL,
so we call SSL_CTX_set_min_proto_version() and SSL_CTX_set_max_proto_version()
as long as the TLS1_3_VERSION macro is defined.
Diffstat (limited to 'src/http/modules/ngx_http_ssl_module.h')
0 files changed, 0 insertions, 0 deletions