Mercurial > hg > nginx-tests
changeset 1938:e1059682aeef
Tests: fixed ClientHello with resending Initial QUIC packets.
Previously it was rebuilt each time using distinct ClientHello.random
resulting in different CRYPTO payload. As such, it led to TLS digest
hash and derived secrets mismatch when resending Initial packet. Now
ClientHello is built once and reused when resending Initial packets.
Additionally, this required to preserve a generated secret value used
in shared secret calculation as part of TLS key schedule. Previously
it was regenerated when receiving a Retry packet, but this won't work
with reused ClientHello as the resulting shared secrets won't match.
author | Sergey Kandaurov <pluknet@nginx.com> |
---|---|
date | Wed, 30 Aug 2023 02:22:58 +0400 |
parents | 2a0a6035a1af |
children | 24482e311749 |
files | lib/Test/Nginx/HTTP3.pm |
diffstat | 1 files changed, 2 insertions(+), 2 deletions(-) [+] |
line wrap: on
line diff
--- a/lib/Test/Nginx/HTTP3.pm Fri Aug 25 18:12:08 2023 +0400 +++ b/lib/Test/Nginx/HTTP3.pm Wed Aug 30 02:22:58 2023 +0400 @@ -59,6 +59,7 @@ $self->{buf} = ''; $self->init(); + $self->init_key_schedule(); $self->retry(%extra) or return; return $self; @@ -100,7 +101,6 @@ $self->set_traffic_keys('tls13 client in', 'SHA256', 32, 0, 'w', $prk); $self->set_traffic_keys('tls13 server in', 'SHA256', 32, 0, 'r', $prk); - $self->init_key_schedule(); $self->initial(); return $self if $extra{probe}; $self->handshake() or return; @@ -134,7 +134,7 @@ sub initial { my ($self) = @_; - $self->{tlsm}{ch} = $self->build_tls_client_hello(); + $self->{tlsm}{ch} ||= $self->build_tls_client_hello(); my $ch = $self->{tlsm}{ch}; my $crypto = build_crypto($ch); my $padding = 1200 - length($crypto);