Cite an RFC over 9000 (draft-ietf-quic-tls is now RFC 9001). Also now that it's finalized, flip the default for SSL_set_quic_use_legacy_codepoint. Update-Note: QUIC APIs now default to the standard code point rather than the draft one. QUICHE has already been calling SSL_set_quic_use_legacy_codepoint, so this should not affect them. Once callers implementing the draft versions cycle out, we can then drop SSL_set_quic_use_legacy_codepoint altogether. I've also bumped BORINGSSL_API_VERSION in case we end up needing an ifdef. Change-Id: Id2cab66215f4ad4c1e31503d329c0febfdb4603e Reviewed-on: https://boringssl-review.googlesource.com/c/boringssl/+/47864 Reviewed-by: David Schinazi <dschinazi@google.com> Reviewed-by: Adam Langley <agl@google.com>
diff --git a/ssl/t1_lib.cc b/ssl/t1_lib.cc index 67f18fd..420b6ee 100644 --- a/ssl/t1_lib.cc +++ b/ssl/t1_lib.cc
@@ -2646,7 +2646,7 @@ return true; } - uint16_t extension_type = TLSEXT_TYPE_quic_transport_parameters_standard; + uint16_t extension_type = TLSEXT_TYPE_quic_transport_parameters; if (hs->config->quic_use_legacy_codepoint) { extension_type = TLSEXT_TYPE_quic_transport_parameters_legacy; } @@ -2782,7 +2782,7 @@ return true; } - uint16_t extension_type = TLSEXT_TYPE_quic_transport_parameters_standard; + uint16_t extension_type = TLSEXT_TYPE_quic_transport_parameters; if (hs->config->quic_use_legacy_codepoint) { extension_type = TLSEXT_TYPE_quic_transport_parameters_legacy; } @@ -3251,7 +3251,7 @@ dont_add_serverhello, }, { - TLSEXT_TYPE_quic_transport_parameters_standard, + TLSEXT_TYPE_quic_transport_parameters, NULL, ext_quic_transport_params_add_clienthello, ext_quic_transport_params_parse_serverhello,