DescriptionAdd missing plumbing of ProofSource::Details in some QUIC codepaths
This CL adds a bit of code for moving ProofSource::Details objects
through the codepaths for evaluating CHLOs. Such an object is produced
by ProofSource::GetProof calls in three places:
QuicCryptoServerConfig::{ValidateClientHello, ProcessClientHello,
BuildServerConfigUpdateMessage}. When a CHLO is evaluated, both
ValidateClientHello and ProcessClientHello are called, but the code is
structured so that only one of these will make the call to GetProof.
But it is possible for the first call not to call GetProof, and for the
second to call it instead. In this case, the ProofSource::Details will
get dropped on the floor.
This issue was not detected in testing because in practice, apparently
the first call always invokes GetProof. This change is therefore mostly
for completeness and future-proofing.
Additional plumbing for stats propagation in QUIC code. Mostly
protected by existing flag --enable_async_get_proof.
Merge internal change: 137148536
R=rch@chromium.org
BUG=
Patch Set 1 #
Depends on Patchset: Dependent Patchsets: Messages
Total messages: 2 (0 generated)
|