Skip to content

Commit bb4f6ef

Browse files
QUICHE teamcopybara-github
authored andcommitted
No public description
Startblock: PiperOrigin-RevId: 874798072
1 parent 8aa2871 commit bb4f6ef

File tree

5 files changed

+19
-12
lines changed

5 files changed

+19
-12
lines changed

quiche/quic/core/crypto/proof_source.h

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -418,7 +418,8 @@ class QUICHE_EXPORT ProofSourceHandle {
418418
std::optional<std::string> alps,
419419
const std::vector<uint8_t>& quic_transport_params,
420420
const std::optional<std::vector<uint8_t>>& early_data_context,
421-
const QuicSSLConfig& ssl_config) = 0;
421+
const QuicSSLConfig& ssl_config,
422+
bool disable_alps_explicit_codepoint) = 0;
422423

423424
// Starts a compute signature operation. If the operation is not cancelled
424425
// when it completes, callback()->OnComputeSignatureDone will be invoked.

quiche/quic/core/tls_server_handshaker.cc

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -107,7 +107,8 @@ TlsServerHandshaker::DefaultProofSourceHandle::SelectCertificate(
107107
std::optional<std::string> /*alps*/,
108108
const std::vector<uint8_t>& /*quic_transport_params*/,
109109
const std::optional<std::vector<uint8_t>>& /*early_data_context*/,
110-
const QuicSSLConfig& /*ssl_config*/) {
110+
const QuicSSLConfig& /*ssl_config*/,
111+
bool /*disable_alps_explicit_codepoint*/) {
111112
if (!handshaker_ || !proof_source_) {
112113
QUIC_BUG(quic_bug_10341_1)
113114
<< "SelectCertificate called on a detached handle";
@@ -1084,7 +1085,7 @@ ssl_select_cert_result_t TlsServerHandshaker::EarlySelectCertCallback(
10841085
AlpnForVersion(session()->version()), std::move(alps_result.alps_buffer),
10851086
set_transport_params_result.quic_transport_params,
10861087
set_transport_params_result.early_data_context,
1087-
tls_connection_.ssl_config());
1088+
tls_connection_.ssl_config(), /*disable_alps_explicit_codepoint=*/false);
10881089

10891090
QUICHE_DCHECK_EQ(status, *select_cert_status());
10901091

quiche/quic/core/tls_server_handshaker.h

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -275,7 +275,8 @@ class QUICHE_EXPORT TlsServerHandshaker : public TlsHandshaker,
275275
std::optional<std::string> alps,
276276
const std::vector<uint8_t>& quic_transport_params,
277277
const std::optional<std::vector<uint8_t>>& early_data_context,
278-
const QuicSSLConfig& ssl_config) override;
278+
const QuicSSLConfig& ssl_config,
279+
bool disable_alps_explicit_codepoint) override;
279280

280281
// Delegates to proof_source_->ComputeTlsSignature.
281282
// Returns QUIC_SUCCESS, QUIC_FAILURE or QUIC_PENDING.

quiche/quic/test_tools/fake_proof_source_handle.cc

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -95,14 +95,14 @@ QuicAsyncStatus FakeProofSourceHandle::SelectCertificate(
9595
std::optional<std::string> alps,
9696
const std::vector<uint8_t>& quic_transport_params,
9797
const std::optional<std::vector<uint8_t>>& early_data_context,
98-
const QuicSSLConfig& ssl_config) {
98+
const QuicSSLConfig& ssl_config, bool disable_alps_explicit_codepoint) {
9999
if (select_cert_action_ != Action::FAIL_SYNC_DO_NOT_CHECK_CLOSED) {
100100
QUICHE_CHECK(!closed_);
101101
}
102-
all_select_cert_args_.push_back(
103-
SelectCertArgs(server_address, client_address, original_connection_id,
104-
ssl_capabilities, hostname, alpn, alps,
105-
quic_transport_params, early_data_context, ssl_config));
102+
all_select_cert_args_.push_back(SelectCertArgs(
103+
server_address, client_address, original_connection_id, ssl_capabilities,
104+
hostname, alpn, alps, quic_transport_params, early_data_context,
105+
ssl_config, disable_alps_explicit_codepoint));
106106

107107
if (select_cert_action_ == Action::DELEGATE_ASYNC ||
108108
select_cert_action_ == Action::FAIL_ASYNC) {

quiche/quic/test_tools/fake_proof_source_handle.h

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -62,7 +62,8 @@ class FakeProofSourceHandle : public ProofSourceHandle {
6262
std::optional<std::string> alps,
6363
const std::vector<uint8_t>& quic_transport_params,
6464
const std::optional<std::vector<uint8_t>>& early_data_context,
65-
const QuicSSLConfig& ssl_config) override;
65+
const QuicSSLConfig& ssl_config,
66+
bool disable_alps_explicit_codepoint) override;
6667

6768
QuicAsyncStatus ComputeSignature(const QuicSocketAddress& server_address,
6869
const QuicSocketAddress& client_address,
@@ -85,7 +86,8 @@ class FakeProofSourceHandle : public ProofSourceHandle {
8586
std::string alpn, std::optional<std::string> alps,
8687
std::vector<uint8_t> quic_transport_params,
8788
std::optional<std::vector<uint8_t>> early_data_context,
88-
QuicSSLConfig ssl_config)
89+
QuicSSLConfig ssl_config,
90+
bool disable_alps_explicit_codepoint)
8991
: server_address(server_address),
9092
client_address(client_address),
9193
original_connection_id(original_connection_id),
@@ -95,7 +97,8 @@ class FakeProofSourceHandle : public ProofSourceHandle {
9597
alps(alps),
9698
quic_transport_params(quic_transport_params),
9799
early_data_context(early_data_context),
98-
ssl_config(ssl_config) {}
100+
ssl_config(ssl_config),
101+
disable_alps_explicit_codepoint(disable_alps_explicit_codepoint) {}
99102

100103
QuicSocketAddress server_address;
101104
QuicSocketAddress client_address;
@@ -107,6 +110,7 @@ class FakeProofSourceHandle : public ProofSourceHandle {
107110
std::vector<uint8_t> quic_transport_params;
108111
std::optional<std::vector<uint8_t>> early_data_context;
109112
QuicSSLConfig ssl_config;
113+
bool disable_alps_explicit_codepoint;
110114
};
111115

112116
struct ComputeSignatureArgs {

0 commit comments

Comments
 (0)