commit 009726eb6321e09a4a845e5ea5c293c55f56f51d
author wub <wub@google.com> Mon Mar 01 11:01:03 2021 -0800
committer Copybara-Service <copybara-worker@google.com> Mon Mar 01 11:02:02 2021 -0800
tree b8c9887b9d0159796e544bcf4ad97063ee157d18
parent 8c5ac4c0e8dc5cfd9525180d471f63ce2bc2bca7

Fix gQUIC remote handshake by postpone certificate validation to after GetProof is done.

Also modify standalone_handshaker_end_to_end_test to verify the HTTP response content, instead of response_complete(), which is implemented incorrectly in quic client.

Protected by FLAGS_quic_reloadable_flag_quic_crypto_postpone_cert_validate_for_server.

PiperOrigin-RevId: 360229986
Change-Id: I1d7afa111daec727901c1e6a8c1c95ca4c76e731

quic/test_tools/quic_test_client.cc

diff --git a/quic/test_tools/quic_test_client.cc b/quic/test_tools/quic_test_client.cc
index 4133950..d1c1272 100644
--- a/quic/test_tools/quic_test_client.cc
+++ b/quic/test_tools/quic_test_client.cc
@@ -796,7 +796,8 @@
   closed_stream_states_.insert(std::make_pair(
       id,
       PerStreamState(
-          client_stream->stream_error(), true,
+          // Set response_complete to true iff stream is closed while connected.
+          client_stream->stream_error(), connected(),
           client_stream->headers_decompressed(),
           client_stream->response_headers(),
           client_stream->preliminary_headers(),