[email protected] | dd3fd0e | 2012-11-04 05:14:40 | [diff] [blame] | 1 | // Copyright (c) 2012 The Chromium Authors. All rights reserved. |
| 2 | // Use of this source code is governed by a BSD-style license that can be |
| 3 | // found in the LICENSE file. |
| 4 | |
Ryan Hamilton | a3ee93a7 | 2018-08-01 22:03:08 | [diff] [blame] | 5 | #include "net/quic/quic_chromium_client_session.h" |
[email protected] | dd3fd0e | 2012-11-04 05:14:40 | [diff] [blame] | 6 | |
[email protected] | 5db45220 | 2014-08-19 05:22:15 | [diff] [blame] | 7 | #include "base/base64.h" |
Sebastien Marchand | 6d0558fd | 2019-01-25 16:49:37 | [diff] [blame] | 8 | #include "base/bind.h" |
[email protected] | f21ec37 | 2014-07-02 07:15:12 | [diff] [blame] | 9 | #include "base/files/file_path.h" |
danakj | ad1777e | 2016-04-16 00:56:42 | [diff] [blame] | 10 | #include "base/memory/ptr_util.h" |
jri | 9f30371 | 2016-09-13 01:10:22 | [diff] [blame] | 11 | #include "base/run_loop.h" |
Ryan Sleevi | b8d7ea0 | 2018-05-07 20:01:01 | [diff] [blame] | 12 | #include "base/stl_util.h" |
Devlin Cronin | e4bcb40e | 2018-06-05 18:02:47 | [diff] [blame] | 13 | #include "base/test/metrics/histogram_tester.h" |
Matt Menke | 26e4154 | 2019-06-05 01:09:51 | [diff] [blame] | 14 | #include "base/test/scoped_feature_list.h" |
gab | f767595f | 2016-05-11 18:50:35 | [diff] [blame] | 15 | #include "base/threading/thread_task_runner_handle.h" |
Zhongyi Shi | c16b410 | 2019-02-12 00:37:40 | [diff] [blame] | 16 | #include "base/time/default_tick_clock.h" |
Ryan Hamilton | a3ee93a7 | 2018-08-01 22:03:08 | [diff] [blame] | 17 | #include "build/build_config.h" |
Matt Menke | 26e4154 | 2019-06-05 01:09:51 | [diff] [blame] | 18 | #include "net/base/features.h" |
| 19 | #include "net/base/network_isolation_key.h" |
[email protected] | 8ee611b | 2012-11-20 01:48:12 | [diff] [blame] | 20 | #include "net/base/test_completion_callback.h" |
[email protected] | f21ec37 | 2014-07-02 07:15:12 | [diff] [blame] | 21 | #include "net/cert/cert_verify_result.h" |
[email protected] | 5db45220 | 2014-08-19 05:22:15 | [diff] [blame] | 22 | #include "net/http/transport_security_state.h" |
Matt Mueller | 230996f1 | 2018-10-22 19:39:44 | [diff] [blame] | 23 | #include "net/http/transport_security_state_test_util.h" |
mikecirone | f22f981 | 2016-10-04 03:40:19 | [diff] [blame] | 24 | #include "net/log/net_log_source.h" |
vishal.b | 62985ca9 | 2015-04-17 08:45:51 | [diff] [blame] | 25 | #include "net/log/test_net_log.h" |
Victor Vasiliev | 4f6fb89 | 2019-05-31 16:58:31 | [diff] [blame] | 26 | #include "net/quic/address_utils.h" |
Ryan Hamilton | a3ee93a7 | 2018-08-01 22:03:08 | [diff] [blame] | 27 | #include "net/quic/crypto/proof_verifier_chromium.h" |
| 28 | #include "net/quic/mock_crypto_client_stream_factory.h" |
| 29 | #include "net/quic/mock_quic_data.h" |
| 30 | #include "net/quic/quic_chromium_alarm_factory.h" |
| 31 | #include "net/quic/quic_chromium_client_session_peer.h" |
| 32 | #include "net/quic/quic_chromium_connection_helper.h" |
| 33 | #include "net/quic/quic_chromium_packet_reader.h" |
| 34 | #include "net/quic/quic_chromium_packet_writer.h" |
| 35 | #include "net/quic/quic_crypto_client_stream_factory.h" |
| 36 | #include "net/quic/quic_http_utils.h" |
| 37 | #include "net/quic/quic_server_info.h" |
| 38 | #include "net/quic/quic_test_packet_maker.h" |
tfarina | 5dd13c2 | 2016-11-16 12:08:26 | [diff] [blame] | 39 | #include "net/socket/datagram_client_socket.h" |
[email protected] | 4d283b3 | 2013-10-17 12:57:27 | [diff] [blame] | 40 | #include "net/socket/socket_test_util.h" |
Bence Béky | 94658bf | 2018-05-11 19:22:58 | [diff] [blame] | 41 | #include "net/spdy/spdy_test_util_common.h" |
[email protected] | f21ec37 | 2014-07-02 07:15:12 | [diff] [blame] | 42 | #include "net/test/cert_test_util.h" |
robpercival | 214763f | 2016-07-01 23:27:01 | [diff] [blame] | 43 | #include "net/test/gtest_util.h" |
rsleevi | a69c79a | 2016-06-22 03:28:43 | [diff] [blame] | 44 | #include "net/test/test_data_directory.h" |
Gabriel Charette | c710874 | 2019-08-23 03:31:40 | [diff] [blame] | 45 | #include "net/test/test_with_task_environment.h" |
Victor Vasiliev | 6bb59d2 | 2019-03-08 21:34:51 | [diff] [blame] | 46 | #include "net/third_party/quiche/src/quic/core/crypto/aes_128_gcm_12_encrypter.h" |
| 47 | #include "net/third_party/quiche/src/quic/core/crypto/crypto_protocol.h" |
| 48 | #include "net/third_party/quiche/src/quic/core/crypto/quic_decrypter.h" |
| 49 | #include "net/third_party/quiche/src/quic/core/crypto/quic_encrypter.h" |
| 50 | #include "net/third_party/quiche/src/quic/core/http/quic_client_promised_info.h" |
| 51 | #include "net/third_party/quiche/src/quic/core/quic_connection_id.h" |
| 52 | #include "net/third_party/quiche/src/quic/core/quic_packet_writer.h" |
| 53 | #include "net/third_party/quiche/src/quic/core/quic_utils.h" |
Victor Vasiliev | 6bb59d2 | 2019-03-08 21:34:51 | [diff] [blame] | 54 | #include "net/third_party/quiche/src/quic/platform/api/quic_flags.h" |
| 55 | #include "net/third_party/quiche/src/quic/platform/api/quic_test.h" |
| 56 | #include "net/third_party/quiche/src/quic/test_tools/crypto_test_utils.h" |
| 57 | #include "net/third_party/quiche/src/quic/test_tools/quic_client_promised_info_peer.h" |
| 58 | #include "net/third_party/quiche/src/quic/test_tools/quic_connection_peer.h" |
| 59 | #include "net/third_party/quiche/src/quic/test_tools/quic_session_peer.h" |
| 60 | #include "net/third_party/quiche/src/quic/test_tools/quic_stream_peer.h" |
| 61 | #include "net/third_party/quiche/src/quic/test_tools/quic_test_utils.h" |
| 62 | #include "net/third_party/quiche/src/quic/test_tools/simple_quic_framer.h" |
Victor Vasiliev | 27cc771 | 2019-01-24 11:50:14 | [diff] [blame] | 63 | #include "net/third_party/quiche/src/spdy/core/spdy_test_utils.h" |
Ramin Halavati | 683bcaa9 | 2018-02-14 08:42:39 | [diff] [blame] | 64 | #include "net/traffic_annotation/network_traffic_annotation_test_helper.h" |
robpercival | 214763f | 2016-07-01 23:27:01 | [diff] [blame] | 65 | #include "testing/gmock/include/gmock/gmock.h" |
[email protected] | dd3fd0e | 2012-11-04 05:14:40 | [diff] [blame] | 66 | |
| 67 | using testing::_; |
| 68 | |
| 69 | namespace net { |
| 70 | namespace test { |
| 71 | namespace { |
| 72 | |
eroman | 36d84e5443 | 2016-03-17 03:23:02 | [diff] [blame] | 73 | const IPEndPoint kIpEndPoint = IPEndPoint(IPAddress::IPv4AllZeros(), 0); |
rch | 1fe2eeb | 2015-10-26 14:45:57 | [diff] [blame] | 74 | const char kServerHostname[] = "test.example.com"; |
Avi Drissman | 13fc893 | 2015-12-20 04:40:46 | [diff] [blame] | 75 | const uint16_t kServerPort = 443; |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 76 | const size_t kMaxReadersPerQuicSession = 5; |
| 77 | |
David Benjamin | 3cc5977 | 2019-04-02 07:33:31 | [diff] [blame] | 78 | // A subclass of QuicChromiumClientSession that allows OnPathDegrading to be |
| 79 | // mocked. |
Nick Harper | e45fe4fc | 2017-07-13 00:34:58 | [diff] [blame] | 80 | class TestingQuicChromiumClientSession : public QuicChromiumClientSession { |
| 81 | public: |
| 82 | using QuicChromiumClientSession::QuicChromiumClientSession; |
| 83 | |
Zhongyi Shi | 5068bb0 | 2018-08-03 02:44:09 | [diff] [blame] | 84 | MOCK_METHOD0(OnPathDegrading, void()); |
Nick Harper | e45fe4fc | 2017-07-13 00:34:58 | [diff] [blame] | 85 | }; |
| 86 | |
ckrasic | 4f9d88d | 2015-07-22 22:23:16 | [diff] [blame] | 87 | class QuicChromiumClientSessionTest |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 88 | : public ::testing::TestWithParam< |
Nick Harper | 23290b8 | 2019-05-02 00:02:56 | [diff] [blame] | 89 | std::tuple<quic::ParsedQuicVersion, bool>>, |
Gabriel Charette | 694c3c33 | 2019-08-19 14:53:05 | [diff] [blame] | 90 | public WithTaskEnvironment { |
Ryan Hamilton | 9883ff6 | 2017-08-01 06:07:05 | [diff] [blame] | 91 | public: |
ckrasic | 4f9d88d | 2015-07-22 22:23:16 | [diff] [blame] | 92 | QuicChromiumClientSessionTest() |
Yixin Wang | 079ad54 | 2018-01-11 04:06:05 | [diff] [blame] | 93 | : version_(std::get<0>(GetParam())), |
| 94 | client_headers_include_h2_stream_dependency_(std::get<1>(GetParam())), |
Nick Harper | a598fc5f | 2019-06-21 08:46:50 | [diff] [blame] | 95 | crypto_config_( |
| 96 | quic::test::crypto_test_utils::ProofVerifierForTesting()), |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 97 | default_read_(new MockRead(SYNCHRONOUS, ERR_IO_PENDING, 0)), |
| 98 | socket_data_( |
Ryan Sleevi | b8d7ea0 | 2018-05-07 20:01:01 | [diff] [blame] | 99 | new SequencedSocketData(base::make_span(default_read_.get(), 1), |
| 100 | base::span<MockWrite>())), |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 101 | random_(0), |
rch | 16c74d1d | 2016-04-22 06:14:07 | [diff] [blame] | 102 | helper_(&clock_, &random_), |
Paul Jensen | 8e3c5d3 | 2018-02-19 17:06:33 | [diff] [blame] | 103 | session_key_(kServerHostname, |
| 104 | kServerPort, |
| 105 | PRIVACY_MODE_DISABLED, |
| 106 | SocketTag()), |
Ryan Hamilton | 6c2a2a8 | 2017-12-15 02:06:28 | [diff] [blame] | 107 | destination_(kServerHostname, kServerPort), |
Yixin Wang | 079ad54 | 2018-01-11 04:06:05 | [diff] [blame] | 108 | client_maker_(version_, |
David Schinazi | c828105 | 2019-01-24 06:14:17 | [diff] [blame] | 109 | quic::QuicUtils::CreateRandomConnectionId(&random_), |
alyssar | 2adf3ac | 2016-05-03 17:12:58 | [diff] [blame] | 110 | &clock_, |
| 111 | kServerHostname, |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 112 | quic::Perspective::IS_CLIENT, |
Yixin Wang | 079ad54 | 2018-01-11 04:06:05 | [diff] [blame] | 113 | client_headers_include_h2_stream_dependency_), |
| 114 | server_maker_(version_, |
David Schinazi | c828105 | 2019-01-24 06:14:17 | [diff] [blame] | 115 | quic::QuicUtils::CreateRandomConnectionId(&random_), |
alyssar | 2adf3ac | 2016-05-03 17:12:58 | [diff] [blame] | 116 | &clock_, |
| 117 | kServerHostname, |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 118 | quic::Perspective::IS_SERVER, |
Yixin Wang | f65c4b6d | 2018-03-08 22:37:40 | [diff] [blame] | 119 | false), |
| 120 | migrate_session_early_v2_(false) { |
Nick Harper | 057264a8 | 2019-09-12 23:33:49 | [diff] [blame^] | 121 | SetQuicReloadableFlag(quic_supports_tls_handshake, true); |
xunjieli | c70cc86 | 2016-02-19 15:29:26 | [diff] [blame] | 122 | // Advance the time, because timers do not like uninitialized times. |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 123 | clock_.AdvanceTime(quic::QuicTime::Delta::FromSeconds(1)); |
xunjieli | c70cc86 | 2016-02-19 15:29:26 | [diff] [blame] | 124 | } |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 125 | |
Ryan Hamilton | 9883ff6 | 2017-08-01 06:07:05 | [diff] [blame] | 126 | void ResetHandleOnError( |
| 127 | std::unique_ptr<QuicChromiumClientSession::Handle>* handle, |
| 128 | int net_error) { |
| 129 | EXPECT_NE(OK, net_error); |
| 130 | handle->reset(); |
| 131 | } |
| 132 | |
| 133 | protected: |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 134 | void Initialize() { |
rch | 1baa747 | 2017-04-26 05:43:58 | [diff] [blame] | 135 | if (socket_data_) |
| 136 | socket_factory_.AddSocketDataProvider(socket_data_.get()); |
danakj | ad1777e | 2016-04-16 00:56:42 | [diff] [blame] | 137 | std::unique_ptr<DatagramClientSocket> socket = |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 138 | socket_factory_.CreateDatagramClientSocket(DatagramSocket::DEFAULT_BIND, |
mikecirone | f22f981 | 2016-10-04 03:40:19 | [diff] [blame] | 139 | &net_log_, NetLogSource()); |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 140 | socket->Connect(kIpEndPoint); |
Ryan Hamilton | 9edcf1a | 2017-11-22 05:55:17 | [diff] [blame] | 141 | QuicChromiumPacketWriter* writer = new net::QuicChromiumPacketWriter( |
| 142 | socket.get(), base::ThreadTaskRunnerHandle::Get().get()); |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 143 | quic::QuicConnection* connection = new quic::QuicConnection( |
David Schinazi | c828105 | 2019-01-24 06:14:17 | [diff] [blame] | 144 | quic::QuicUtils::CreateRandomConnectionId(&random_), |
Victor Vasiliev | 4f6fb89 | 2019-05-31 16:58:31 | [diff] [blame] | 145 | ToQuicSocketAddress(kIpEndPoint), &helper_, &alarm_factory_, writer, |
| 146 | true, quic::Perspective::IS_CLIENT, |
Nick Harper | 23290b8 | 2019-05-02 00:02:56 | [diff] [blame] | 147 | quic::test::SupportedVersions(version_)); |
Nick Harper | e45fe4fc | 2017-07-13 00:34:58 | [diff] [blame] | 148 | session_.reset(new TestingQuicChromiumClientSession( |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 149 | connection, std::move(socket), |
| 150 | /*stream_factory=*/nullptr, &crypto_client_stream_factory_, &clock_, |
Nick Harper | 89bc721 | 2018-07-31 19:07:57 | [diff] [blame] | 151 | &transport_security_state_, /*ssl_config_service=*/nullptr, |
Paul Jensen | 8e3c5d3 | 2018-02-19 17:06:33 | [diff] [blame] | 152 | base::WrapUnique(static_cast<QuicServerInfo*>(nullptr)), session_key_, |
Bence Béky | 1ceba55 | 2019-07-19 17:11:05 | [diff] [blame] | 153 | /*require_confirmation=*/false, |
| 154 | /*max_allowed_push_id=*/0, migrate_session_early_v2_, |
Yixin Wang | f65c4b6d | 2018-03-08 22:37:40 | [diff] [blame] | 155 | /*migrate_session_on_network_change_v2=*/false, |
Zhongyi Shi | d672f754 | 2018-06-08 01:03:26 | [diff] [blame] | 156 | /*defaulet_network=*/NetworkChangeNotifier::kInvalidNetworkHandle, |
Zhongyi Shi | e01f2db | 2019-02-22 19:53:23 | [diff] [blame] | 157 | quic::QuicTime::Delta::FromMilliseconds( |
Ryan Sleevi | 2e8255b | 2019-07-17 21:02:21 | [diff] [blame] | 158 | kDefaultRetransmittableOnWireTimeout.InMilliseconds()), |
Zhongyi Shi | af38c4e4 | 2019-08-29 22:49:05 | [diff] [blame] | 159 | /*migrate_idle_session=*/false, /*allow_port_migration=*/false, |
| 160 | kDefaultIdleSessionMigrationPeriod, kMaxTimeOnNonDefaultNetwork, |
Zhongyi Shi | ee76076 | 2018-08-01 00:54:29 | [diff] [blame] | 161 | kMaxMigrationsToNonDefaultNetworkOnWriteError, |
Zhongyi Shi | 8b1e43f | 2017-12-13 20:46:30 | [diff] [blame] | 162 | kMaxMigrationsToNonDefaultNetworkOnPathDegrading, |
Zhongyi Shi | 5f587cc | 2017-11-21 23:24:17 | [diff] [blame] | 163 | kQuicYieldAfterPacketsRead, |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 164 | quic::QuicTime::Delta::FromMilliseconds( |
| 165 | kQuicYieldAfterDurationMilliseconds), |
Zhongyi Shi | dbce7f41 | 2019-02-01 23:16:29 | [diff] [blame] | 166 | /*cert_verify_flags=*/0, /*go_away_on_path_degrading*/ false, |
| 167 | client_headers_include_h2_stream_dependency_, |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 168 | quic::test::DefaultQuicConfig(), &crypto_config_, "CONNECTION_UNKNOWN", |
Yixin Wang | 079ad54 | 2018-01-11 04:06:05 | [diff] [blame] | 169 | base::TimeTicks::Now(), base::TimeTicks::Now(), &push_promise_index_, |
Zhongyi Shi | c16b410 | 2019-02-12 00:37:40 | [diff] [blame] | 170 | &test_push_delegate_, base::DefaultTickClock::GetInstance(), |
| 171 | base::ThreadTaskRunnerHandle::Get().get(), |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 172 | /*socket_performance_watcher=*/nullptr, &net_log_)); |
| 173 | |
| 174 | scoped_refptr<X509Certificate> cert( |
| 175 | ImportCertFromFile(GetTestCertsDirectory(), "spdy_pooling.pem")); |
| 176 | verify_details_.cert_verify_result.verified_cert = cert; |
| 177 | verify_details_.cert_verify_result.is_issued_by_known_root = true; |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 178 | session_->Initialize(); |
| 179 | session_->StartReading(); |
jri | bce3eec | 2016-09-09 05:41:08 | [diff] [blame] | 180 | writer->set_delegate(session_.get()); |
[email protected] | dd3fd0e | 2012-11-04 05:14:40 | [diff] [blame] | 181 | } |
| 182 | |
rtenneti | 85dcfac2 | 2015-03-27 20:22:19 | [diff] [blame] | 183 | void TearDown() override { |
rch | f0b18c8a | 2017-05-05 19:31:57 | [diff] [blame] | 184 | if (session_) |
Renjie | ba55fae | 2018-09-20 03:05:16 | [diff] [blame] | 185 | session_->CloseSessionOnError( |
| 186 | ERR_ABORTED, quic::QUIC_INTERNAL_ERROR, |
| 187 | quic::ConnectionCloseBehavior::SILENT_CLOSE); |
[email protected] | 4d283b3 | 2013-10-17 12:57:27 | [diff] [blame] | 188 | } |
| 189 | |
[email protected] | ed3fc15d | 2013-03-08 18:37:44 | [diff] [blame] | 190 | void CompleteCryptoHandshake() { |
rch | 433bf5f | 2017-02-14 04:10:47 | [diff] [blame] | 191 | ASSERT_THAT(session_->CryptoConnect(callback_.callback()), IsOk()); |
[email protected] | ed3fc15d | 2013-03-08 18:37:44 | [diff] [blame] | 192 | } |
| 193 | |
jri | ed79618b | 2016-07-02 03:18:52 | [diff] [blame] | 194 | QuicChromiumPacketWriter* CreateQuicChromiumPacketWriter( |
| 195 | DatagramClientSocket* socket, |
| 196 | QuicChromiumClientSession* session) const { |
danakj | ad1777e | 2016-04-16 00:56:42 | [diff] [blame] | 197 | std::unique_ptr<QuicChromiumPacketWriter> writer( |
Ryan Hamilton | 9edcf1a | 2017-11-22 05:55:17 | [diff] [blame] | 198 | new QuicChromiumPacketWriter( |
| 199 | socket, base::ThreadTaskRunnerHandle::Get().get())); |
jri | bce3eec | 2016-09-09 05:41:08 | [diff] [blame] | 200 | writer->set_delegate(session); |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 201 | return writer.release(); |
| 202 | } |
| 203 | |
Fan Yang | 32c5a11 | 2018-12-10 20:06:33 | [diff] [blame] | 204 | quic::QuicStreamId GetNthClientInitiatedBidirectionalStreamId(int n) { |
Nick Harper | 23290b8 | 2019-05-02 00:02:56 | [diff] [blame] | 205 | return quic::test::GetNthClientInitiatedBidirectionalStreamId( |
| 206 | version_.transport_version, n); |
ckrasic | bf2f59c | 2017-05-04 23:54:36 | [diff] [blame] | 207 | } |
| 208 | |
Fan Yang | 32c5a11 | 2018-12-10 20:06:33 | [diff] [blame] | 209 | quic::QuicStreamId GetNthServerInitiatedUnidirectionalStreamId(int n) { |
Nick Harper | 23290b8 | 2019-05-02 00:02:56 | [diff] [blame] | 210 | return quic::test::GetNthServerInitiatedUnidirectionalStreamId( |
| 211 | version_.transport_version, n); |
ckrasic | bf2f59c | 2017-05-04 23:54:36 | [diff] [blame] | 212 | } |
| 213 | |
Fan Yang | 32c5a11 | 2018-12-10 20:06:33 | [diff] [blame] | 214 | size_t GetMaxAllowedOutgoingBidirectionalStreams() { |
Frank Kastenholz | c9b9bea | 2018-12-03 20:13:47 | [diff] [blame] | 215 | quic::QuicSession* quic_session = |
| 216 | dynamic_cast<quic::QuicSession*>(&*session_); |
Nick Harper | 23290b8 | 2019-05-02 00:02:56 | [diff] [blame] | 217 | if (version_.transport_version != quic::QUIC_VERSION_99) { |
Frank Kastenholz | c9b9bea | 2018-12-03 20:13:47 | [diff] [blame] | 218 | return quic::test::QuicSessionPeer::GetStreamIdManager(quic_session) |
| 219 | ->max_open_outgoing_streams(); |
| 220 | } |
David Schinazi | cc1bc59 | 2019-04-24 19:40:31 | [diff] [blame] | 221 | // For version99, the count will include both static and dynamic streams. |
| 222 | // These tests are only concerned with dynamic streams (that is, the number |
Ryan Hamilton | 4aeec56 | 2019-05-17 21:22:52 | [diff] [blame] | 223 | // of streams that they can create), so back out the static header stream. |
Frank Kastenholz | c9b9bea | 2018-12-03 20:13:47 | [diff] [blame] | 224 | return quic::test::QuicSessionPeer::v99_streamid_manager(quic_session) |
Fan Yang | 2330d18 | 2019-08-05 14:50:50 | [diff] [blame] | 225 | ->max_allowed_outgoing_bidirectional_streams(); |
Frank Kastenholz | c9b9bea | 2018-12-03 20:13:47 | [diff] [blame] | 226 | } |
| 227 | |
Nick Harper | 23290b8 | 2019-05-02 00:02:56 | [diff] [blame] | 228 | const quic::ParsedQuicVersion version_; |
Yixin Wang | 079ad54 | 2018-01-11 04:06:05 | [diff] [blame] | 229 | const bool client_headers_include_h2_stream_dependency_; |
Jana Iyengar | f6b13d8 | 2017-09-04 02:09:10 | [diff] [blame] | 230 | QuicFlagSaver flags_; // Save/restore all QUIC flag values. |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 231 | quic::QuicCryptoClientConfig crypto_config_; |
vishal.b | 62985ca9 | 2015-04-17 08:45:51 | [diff] [blame] | 232 | TestNetLog net_log_; |
tfarina | 4283411 | 2016-09-22 13:38:20 | [diff] [blame] | 233 | BoundTestNetLog bound_test_net_log_; |
[email protected] | 4d283b3 | 2013-10-17 12:57:27 | [diff] [blame] | 234 | MockClientSocketFactory socket_factory_; |
danakj | ad1777e | 2016-04-16 00:56:42 | [diff] [blame] | 235 | std::unique_ptr<MockRead> default_read_; |
| 236 | std::unique_ptr<SequencedSocketData> socket_data_; |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 237 | quic::MockClock clock_; |
| 238 | quic::test::MockRandom random_; |
rch | 12fef55 | 2016-01-15 16:26:31 | [diff] [blame] | 239 | QuicChromiumConnectionHelper helper_; |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 240 | quic::test::MockAlarmFactory alarm_factory_; |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 241 | TransportSecurityState transport_security_state_; |
| 242 | MockCryptoClientStreamFactory crypto_client_stream_factory_; |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 243 | quic::QuicClientPushPromiseIndex push_promise_index_; |
Paul Jensen | 8e3c5d3 | 2018-02-19 17:06:33 | [diff] [blame] | 244 | QuicSessionKey session_key_; |
Ryan Hamilton | 6c2a2a8 | 2017-12-15 02:06:28 | [diff] [blame] | 245 | HostPortPair destination_; |
Nick Harper | e45fe4fc | 2017-07-13 00:34:58 | [diff] [blame] | 246 | std::unique_ptr<TestingQuicChromiumClientSession> session_; |
zhongyi | 0009f3e | 2016-11-11 19:47:50 | [diff] [blame] | 247 | TestServerPushDelegate test_push_delegate_; |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 248 | quic::QuicConnectionVisitorInterface* visitor_; |
[email protected] | 8ee611b | 2012-11-20 01:48:12 | [diff] [blame] | 249 | TestCompletionCallback callback_; |
alyssar | 2adf3ac | 2016-05-03 17:12:58 | [diff] [blame] | 250 | QuicTestPacketMaker client_maker_; |
| 251 | QuicTestPacketMaker server_maker_; |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 252 | ProofVerifyDetailsChromium verify_details_; |
Yixin Wang | f65c4b6d | 2018-03-08 22:37:40 | [diff] [blame] | 253 | bool migrate_session_early_v2_; |
[email protected] | dd3fd0e | 2012-11-04 05:14:40 | [diff] [blame] | 254 | }; |
| 255 | |
Victor Costan | e635086f | 2019-01-27 05:20:30 | [diff] [blame] | 256 | INSTANTIATE_TEST_SUITE_P( |
Bence Béky | ce380cb | 2018-04-26 23:39:55 | [diff] [blame] | 257 | VersionIncludeStreamDependencySequence, |
Yixin Wang | 079ad54 | 2018-01-11 04:06:05 | [diff] [blame] | 258 | QuicChromiumClientSessionTest, |
Ryan Hamilton | 93424eb8 | 2019-08-23 04:28:40 | [diff] [blame] | 259 | ::testing::Combine(::testing::ValuesIn(quic::AllSupportedVersions()), |
Nick Harper | 23290b8 | 2019-05-02 00:02:56 | [diff] [blame] | 260 | ::testing::Bool())); |
[email protected] | 4d64079 | 2013-12-18 22:21:08 | [diff] [blame] | 261 | |
Shivani Sharma | 8ae506c | 2019-07-21 21:08:27 | [diff] [blame] | 262 | // TODO(950069): Add testing for frame_origin in NetworkIsolationKey using |
| 263 | // kAppendInitiatingFrameOriginToNetworkIsolationKey. |
| 264 | |
Carlos IL | 8113338 | 2017-12-06 17:18:45 | [diff] [blame] | 265 | TEST_P(QuicChromiumClientSessionTest, IsFatalErrorNotSetForNonFatalError) { |
Renjie Tang | aadb84b | 2019-08-31 01:00:23 | [diff] [blame] | 266 | MockQuicData quic_data(version_); |
| 267 | if (VersionUsesQpack(version_.transport_version)) |
| 268 | quic_data.AddWrite(SYNCHRONOUS, client_maker_.MakeInitialSettingsPacket(1)); |
| 269 | quic_data.AddRead(ASYNC, ERR_IO_PENDING); |
| 270 | quic_data.AddRead(ASYNC, OK); // EOF |
| 271 | quic_data.AddSocketDataToFactory(&socket_factory_); |
| 272 | |
Carlos IL | 8113338 | 2017-12-06 17:18:45 | [diff] [blame] | 273 | Initialize(); |
| 274 | |
| 275 | SSLInfo ssl_info; |
| 276 | ProofVerifyDetailsChromium details; |
| 277 | details.cert_verify_result.verified_cert = |
| 278 | ImportCertFromFile(GetTestCertsDirectory(), "spdy_pooling.pem"); |
| 279 | details.cert_verify_result.cert_status = |
| 280 | MapNetErrorToCertStatus(ERR_CERT_DATE_INVALID); |
| 281 | details.is_fatal_cert_error = false; |
| 282 | CompleteCryptoHandshake(); |
| 283 | session_->OnProofVerifyDetailsAvailable(details); |
| 284 | |
| 285 | ASSERT_TRUE(session_->GetSSLInfo(&ssl_info)); |
| 286 | EXPECT_FALSE(ssl_info.is_fatal_cert_error); |
| 287 | } |
| 288 | |
| 289 | TEST_P(QuicChromiumClientSessionTest, IsFatalErrorSetForFatalError) { |
Renjie Tang | aadb84b | 2019-08-31 01:00:23 | [diff] [blame] | 290 | MockQuicData quic_data(version_); |
| 291 | if (VersionUsesQpack(version_.transport_version)) |
| 292 | quic_data.AddWrite(SYNCHRONOUS, client_maker_.MakeInitialSettingsPacket(1)); |
| 293 | quic_data.AddRead(ASYNC, ERR_IO_PENDING); |
| 294 | quic_data.AddRead(ASYNC, OK); // EOF |
| 295 | quic_data.AddSocketDataToFactory(&socket_factory_); |
Carlos IL | 8113338 | 2017-12-06 17:18:45 | [diff] [blame] | 296 | Initialize(); |
| 297 | |
| 298 | SSLInfo ssl_info; |
| 299 | ProofVerifyDetailsChromium details; |
| 300 | details.cert_verify_result.verified_cert = |
| 301 | ImportCertFromFile(GetTestCertsDirectory(), "spdy_pooling.pem"); |
| 302 | details.cert_verify_result.cert_status = |
| 303 | MapNetErrorToCertStatus(ERR_CERT_DATE_INVALID); |
| 304 | details.is_fatal_cert_error = true; |
| 305 | CompleteCryptoHandshake(); |
| 306 | session_->OnProofVerifyDetailsAvailable(details); |
| 307 | ASSERT_TRUE(session_->GetSSLInfo(&ssl_info)); |
| 308 | EXPECT_TRUE(ssl_info.is_fatal_cert_error); |
| 309 | } |
| 310 | |
ckrasic | 4f9d88d | 2015-07-22 22:23:16 | [diff] [blame] | 311 | TEST_P(QuicChromiumClientSessionTest, CryptoConnect) { |
Renjie Tang | aadb84b | 2019-08-31 01:00:23 | [diff] [blame] | 312 | MockQuicData quic_data(version_); |
| 313 | if (VersionUsesQpack(version_.transport_version)) |
| 314 | quic_data.AddWrite(SYNCHRONOUS, client_maker_.MakeInitialSettingsPacket(1)); |
| 315 | quic_data.AddRead(ASYNC, ERR_IO_PENDING); |
| 316 | quic_data.AddRead(ASYNC, OK); // EOF |
| 317 | quic_data.AddSocketDataToFactory(&socket_factory_); |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 318 | Initialize(); |
[email protected] | ed3fc15d | 2013-03-08 18:37:44 | [diff] [blame] | 319 | CompleteCryptoHandshake(); |
[email protected] | 8ee611b | 2012-11-20 01:48:12 | [diff] [blame] | 320 | } |
| 321 | |
rch | f0b18c8a | 2017-05-05 19:31:57 | [diff] [blame] | 322 | TEST_P(QuicChromiumClientSessionTest, Handle) { |
Ryan Hamilton | abad59e | 2019-06-06 04:02:59 | [diff] [blame] | 323 | MockQuicData quic_data(version_); |
Renjie Tang | aadb84b | 2019-08-31 01:00:23 | [diff] [blame] | 324 | if (VersionUsesQpack(version_.transport_version)) |
| 325 | quic_data.AddWrite(SYNCHRONOUS, client_maker_.MakeInitialSettingsPacket(1)); |
rch | f0b18c8a | 2017-05-05 19:31:57 | [diff] [blame] | 326 | quic_data.AddRead(ASYNC, ERR_IO_PENDING); |
| 327 | quic_data.AddRead(ASYNC, OK); // EOF |
| 328 | quic_data.AddSocketDataToFactory(&socket_factory_); |
| 329 | |
| 330 | Initialize(); |
| 331 | |
| 332 | NetLogWithSource session_net_log = session_->net_log(); |
| 333 | EXPECT_EQ(NetLogSourceType::QUIC_SESSION, session_net_log.source().type); |
| 334 | EXPECT_EQ(&net_log_, session_net_log.net_log()); |
| 335 | |
| 336 | std::unique_ptr<QuicChromiumClientSession::Handle> handle = |
Ryan Hamilton | 6c2a2a8 | 2017-12-15 02:06:28 | [diff] [blame] | 337 | session_->CreateHandle(destination_); |
rch | f0b18c8a | 2017-05-05 19:31:57 | [diff] [blame] | 338 | EXPECT_TRUE(handle->IsConnected()); |
| 339 | EXPECT_FALSE(handle->IsCryptoHandshakeConfirmed()); |
Nick Harper | 23290b8 | 2019-05-02 00:02:56 | [diff] [blame] | 340 | EXPECT_EQ(version_.transport_version, handle->GetQuicVersion()); |
Paul Jensen | 8e3c5d3 | 2018-02-19 17:06:33 | [diff] [blame] | 341 | EXPECT_EQ(session_key_.server_id(), handle->server_id()); |
rch | f0b18c8a | 2017-05-05 19:31:57 | [diff] [blame] | 342 | EXPECT_EQ(session_net_log.source().type, handle->net_log().source().type); |
| 343 | EXPECT_EQ(session_net_log.source().id, handle->net_log().source().id); |
| 344 | EXPECT_EQ(session_net_log.net_log(), handle->net_log().net_log()); |
| 345 | IPEndPoint address; |
| 346 | EXPECT_EQ(OK, handle->GetPeerAddress(&address)); |
| 347 | EXPECT_EQ(kIpEndPoint, address); |
Bin Wu | ae74d8b | 2019-06-19 23:55:07 | [diff] [blame] | 348 | EXPECT_TRUE(handle->CreatePacketBundler().get() != nullptr); |
rch | f0b18c8a | 2017-05-05 19:31:57 | [diff] [blame] | 349 | |
| 350 | CompleteCryptoHandshake(); |
| 351 | |
| 352 | EXPECT_TRUE(handle->IsCryptoHandshakeConfirmed()); |
| 353 | |
| 354 | // Request a stream and verify that a stream was created. |
| 355 | TestCompletionCallback callback; |
| 356 | ASSERT_EQ(OK, handle->RequestStream(/*requires_confirmation=*/false, |
Ramin Halavati | 683bcaa9 | 2018-02-14 08:42:39 | [diff] [blame] | 357 | callback.callback(), |
| 358 | TRAFFIC_ANNOTATION_FOR_TESTS)); |
rch | 1bcfddf2 | 2017-06-03 00:26:29 | [diff] [blame] | 359 | EXPECT_TRUE(handle->ReleaseStream() != nullptr); |
rch | f0b18c8a | 2017-05-05 19:31:57 | [diff] [blame] | 360 | |
| 361 | quic_data.Resume(); |
| 362 | EXPECT_TRUE(quic_data.AllReadDataConsumed()); |
| 363 | EXPECT_TRUE(quic_data.AllWriteDataConsumed()); |
| 364 | |
| 365 | // Veirfy that the handle works correctly after the session is closed. |
| 366 | EXPECT_FALSE(handle->IsConnected()); |
| 367 | EXPECT_TRUE(handle->IsCryptoHandshakeConfirmed()); |
Nick Harper | 23290b8 | 2019-05-02 00:02:56 | [diff] [blame] | 368 | EXPECT_EQ(version_.transport_version, handle->GetQuicVersion()); |
Paul Jensen | 8e3c5d3 | 2018-02-19 17:06:33 | [diff] [blame] | 369 | EXPECT_EQ(session_key_.server_id(), handle->server_id()); |
rch | f0b18c8a | 2017-05-05 19:31:57 | [diff] [blame] | 370 | EXPECT_EQ(session_net_log.source().type, handle->net_log().source().type); |
| 371 | EXPECT_EQ(session_net_log.source().id, handle->net_log().source().id); |
| 372 | EXPECT_EQ(session_net_log.net_log(), handle->net_log().net_log()); |
| 373 | EXPECT_EQ(ERR_CONNECTION_CLOSED, handle->GetPeerAddress(&address)); |
Bin Wu | ae74d8b | 2019-06-19 23:55:07 | [diff] [blame] | 374 | EXPECT_TRUE(handle->CreatePacketBundler().get() == nullptr); |
rch | f0b18c8a | 2017-05-05 19:31:57 | [diff] [blame] | 375 | { |
| 376 | // Verify that CreateHandle() works even after the session is closed. |
| 377 | std::unique_ptr<QuicChromiumClientSession::Handle> handle2 = |
Ryan Hamilton | 6c2a2a8 | 2017-12-15 02:06:28 | [diff] [blame] | 378 | session_->CreateHandle(destination_); |
rch | f0b18c8a | 2017-05-05 19:31:57 | [diff] [blame] | 379 | EXPECT_FALSE(handle2->IsConnected()); |
| 380 | EXPECT_TRUE(handle2->IsCryptoHandshakeConfirmed()); |
| 381 | ASSERT_EQ(ERR_CONNECTION_CLOSED, |
| 382 | handle2->RequestStream(/*requires_confirmation=*/false, |
Ramin Halavati | 683bcaa9 | 2018-02-14 08:42:39 | [diff] [blame] | 383 | callback.callback(), |
| 384 | TRAFFIC_ANNOTATION_FOR_TESTS)); |
rch | f0b18c8a | 2017-05-05 19:31:57 | [diff] [blame] | 385 | } |
| 386 | |
| 387 | session_.reset(); |
| 388 | |
| 389 | // Veirfy that the handle works correctly after the session is deleted. |
| 390 | EXPECT_FALSE(handle->IsConnected()); |
| 391 | EXPECT_TRUE(handle->IsCryptoHandshakeConfirmed()); |
Nick Harper | 23290b8 | 2019-05-02 00:02:56 | [diff] [blame] | 392 | EXPECT_EQ(version_.transport_version, handle->GetQuicVersion()); |
Paul Jensen | 8e3c5d3 | 2018-02-19 17:06:33 | [diff] [blame] | 393 | EXPECT_EQ(session_key_.server_id(), handle->server_id()); |
rch | f0b18c8a | 2017-05-05 19:31:57 | [diff] [blame] | 394 | EXPECT_EQ(session_net_log.source().type, handle->net_log().source().type); |
| 395 | EXPECT_EQ(session_net_log.source().id, handle->net_log().source().id); |
| 396 | EXPECT_EQ(session_net_log.net_log(), handle->net_log().net_log()); |
| 397 | EXPECT_EQ(ERR_CONNECTION_CLOSED, handle->GetPeerAddress(&address)); |
Bin Wu | ae74d8b | 2019-06-19 23:55:07 | [diff] [blame] | 398 | EXPECT_TRUE(handle->CreatePacketBundler().get() == nullptr); |
Ramin Halavati | 683bcaa9 | 2018-02-14 08:42:39 | [diff] [blame] | 399 | ASSERT_EQ( |
| 400 | ERR_CONNECTION_CLOSED, |
| 401 | handle->RequestStream(/*requires_confirmation=*/false, |
| 402 | callback.callback(), TRAFFIC_ANNOTATION_FOR_TESTS)); |
rch | f0b18c8a | 2017-05-05 19:31:57 | [diff] [blame] | 403 | } |
| 404 | |
rch | 1baa747 | 2017-04-26 05:43:58 | [diff] [blame] | 405 | TEST_P(QuicChromiumClientSessionTest, StreamRequest) { |
Ryan Hamilton | abad59e | 2019-06-06 04:02:59 | [diff] [blame] | 406 | MockQuicData quic_data(version_); |
Renjie Tang | aadb84b | 2019-08-31 01:00:23 | [diff] [blame] | 407 | if (VersionUsesQpack(version_.transport_version)) |
| 408 | quic_data.AddWrite(SYNCHRONOUS, client_maker_.MakeInitialSettingsPacket(1)); |
rch | 1baa747 | 2017-04-26 05:43:58 | [diff] [blame] | 409 | quic_data.AddRead(ASYNC, ERR_IO_PENDING); |
| 410 | quic_data.AddRead(ASYNC, OK); // EOF |
| 411 | quic_data.AddSocketDataToFactory(&socket_factory_); |
| 412 | |
| 413 | Initialize(); |
| 414 | CompleteCryptoHandshake(); |
| 415 | |
| 416 | // Request a stream and verify that a stream was created. |
rch | f0b18c8a | 2017-05-05 19:31:57 | [diff] [blame] | 417 | std::unique_ptr<QuicChromiumClientSession::Handle> handle = |
Ryan Hamilton | 6c2a2a8 | 2017-12-15 02:06:28 | [diff] [blame] | 418 | session_->CreateHandle(destination_); |
rch | 1baa747 | 2017-04-26 05:43:58 | [diff] [blame] | 419 | TestCompletionCallback callback; |
rch | f0b18c8a | 2017-05-05 19:31:57 | [diff] [blame] | 420 | ASSERT_EQ(OK, handle->RequestStream(/*requires_confirmation=*/false, |
Ramin Halavati | 683bcaa9 | 2018-02-14 08:42:39 | [diff] [blame] | 421 | callback.callback(), |
| 422 | TRAFFIC_ANNOTATION_FOR_TESTS)); |
rch | 1bcfddf2 | 2017-06-03 00:26:29 | [diff] [blame] | 423 | EXPECT_TRUE(handle->ReleaseStream() != nullptr); |
rch | 1baa747 | 2017-04-26 05:43:58 | [diff] [blame] | 424 | |
| 425 | quic_data.Resume(); |
| 426 | EXPECT_TRUE(quic_data.AllReadDataConsumed()); |
| 427 | EXPECT_TRUE(quic_data.AllWriteDataConsumed()); |
| 428 | } |
| 429 | |
rch | d606b6c3 | 2017-05-02 17:32:57 | [diff] [blame] | 430 | TEST_P(QuicChromiumClientSessionTest, ConfirmationRequiredStreamRequest) { |
Ryan Hamilton | abad59e | 2019-06-06 04:02:59 | [diff] [blame] | 431 | MockQuicData quic_data(version_); |
Renjie Tang | aadb84b | 2019-08-31 01:00:23 | [diff] [blame] | 432 | if (VersionUsesQpack(version_.transport_version)) |
| 433 | quic_data.AddWrite(SYNCHRONOUS, client_maker_.MakeInitialSettingsPacket(1)); |
rch | d606b6c3 | 2017-05-02 17:32:57 | [diff] [blame] | 434 | quic_data.AddRead(ASYNC, ERR_IO_PENDING); |
| 435 | quic_data.AddRead(ASYNC, OK); // EOF |
| 436 | quic_data.AddSocketDataToFactory(&socket_factory_); |
| 437 | |
| 438 | Initialize(); |
| 439 | CompleteCryptoHandshake(); |
| 440 | |
| 441 | // Request a stream and verify that a stream was created. |
rch | f0b18c8a | 2017-05-05 19:31:57 | [diff] [blame] | 442 | std::unique_ptr<QuicChromiumClientSession::Handle> handle = |
Ryan Hamilton | 6c2a2a8 | 2017-12-15 02:06:28 | [diff] [blame] | 443 | session_->CreateHandle(destination_); |
rch | d606b6c3 | 2017-05-02 17:32:57 | [diff] [blame] | 444 | TestCompletionCallback callback; |
rch | f0b18c8a | 2017-05-05 19:31:57 | [diff] [blame] | 445 | ASSERT_EQ(OK, handle->RequestStream(/*requires_confirmation=*/true, |
Ramin Halavati | 683bcaa9 | 2018-02-14 08:42:39 | [diff] [blame] | 446 | callback.callback(), |
| 447 | TRAFFIC_ANNOTATION_FOR_TESTS)); |
rch | 1bcfddf2 | 2017-06-03 00:26:29 | [diff] [blame] | 448 | EXPECT_TRUE(handle->ReleaseStream() != nullptr); |
rch | d606b6c3 | 2017-05-02 17:32:57 | [diff] [blame] | 449 | |
| 450 | quic_data.Resume(); |
| 451 | EXPECT_TRUE(quic_data.AllReadDataConsumed()); |
| 452 | EXPECT_TRUE(quic_data.AllWriteDataConsumed()); |
| 453 | } |
| 454 | |
| 455 | TEST_P(QuicChromiumClientSessionTest, StreamRequestBeforeConfirmation) { |
Ryan Hamilton | abad59e | 2019-06-06 04:02:59 | [diff] [blame] | 456 | MockQuicData quic_data(version_); |
Renjie Tang | aadb84b | 2019-08-31 01:00:23 | [diff] [blame] | 457 | if (VersionUsesQpack(version_.transport_version)) |
| 458 | quic_data.AddWrite(SYNCHRONOUS, client_maker_.MakeInitialSettingsPacket(1)); |
rch | d606b6c3 | 2017-05-02 17:32:57 | [diff] [blame] | 459 | quic_data.AddRead(ASYNC, ERR_IO_PENDING); |
| 460 | quic_data.AddRead(ASYNC, OK); // EOF |
| 461 | quic_data.AddSocketDataToFactory(&socket_factory_); |
| 462 | |
| 463 | Initialize(); |
| 464 | |
| 465 | // Request a stream and verify that a stream was created. |
rch | f0b18c8a | 2017-05-05 19:31:57 | [diff] [blame] | 466 | std::unique_ptr<QuicChromiumClientSession::Handle> handle = |
Ryan Hamilton | 6c2a2a8 | 2017-12-15 02:06:28 | [diff] [blame] | 467 | session_->CreateHandle(destination_); |
rch | d606b6c3 | 2017-05-02 17:32:57 | [diff] [blame] | 468 | TestCompletionCallback callback; |
Ramin Halavati | 683bcaa9 | 2018-02-14 08:42:39 | [diff] [blame] | 469 | ASSERT_EQ( |
| 470 | ERR_IO_PENDING, |
| 471 | handle->RequestStream(/*requires_confirmation=*/true, callback.callback(), |
| 472 | TRAFFIC_ANNOTATION_FOR_TESTS)); |
rch | d606b6c3 | 2017-05-02 17:32:57 | [diff] [blame] | 473 | |
| 474 | CompleteCryptoHandshake(); |
| 475 | |
| 476 | EXPECT_THAT(callback.WaitForResult(), IsOk()); |
| 477 | |
rch | 1bcfddf2 | 2017-06-03 00:26:29 | [diff] [blame] | 478 | EXPECT_TRUE(handle->ReleaseStream() != nullptr); |
rch | d606b6c3 | 2017-05-02 17:32:57 | [diff] [blame] | 479 | |
| 480 | quic_data.Resume(); |
| 481 | EXPECT_TRUE(quic_data.AllReadDataConsumed()); |
| 482 | EXPECT_TRUE(quic_data.AllWriteDataConsumed()); |
| 483 | } |
| 484 | |
rch | 1baa747 | 2017-04-26 05:43:58 | [diff] [blame] | 485 | TEST_P(QuicChromiumClientSessionTest, CancelStreamRequestBeforeRelease) { |
Ryan Hamilton | abad59e | 2019-06-06 04:02:59 | [diff] [blame] | 486 | MockQuicData quic_data(version_); |
Renjie Tang | aadb84b | 2019-08-31 01:00:23 | [diff] [blame] | 487 | int packet_num = 1; |
| 488 | if (VersionUsesQpack(version_.transport_version)) { |
| 489 | quic_data.AddWrite(SYNCHRONOUS, |
| 490 | client_maker_.MakeInitialSettingsPacket(packet_num++)); |
| 491 | } |
| 492 | quic_data.AddWrite( |
| 493 | SYNCHRONOUS, |
| 494 | client_maker_.MakeRstPacket(packet_num++, true, |
| 495 | GetNthClientInitiatedBidirectionalStreamId(0), |
| 496 | quic::QUIC_STREAM_CANCELLED)); |
rch | 1baa747 | 2017-04-26 05:43:58 | [diff] [blame] | 497 | quic_data.AddRead(ASYNC, ERR_IO_PENDING); |
| 498 | quic_data.AddRead(ASYNC, OK); // EOF |
| 499 | quic_data.AddSocketDataToFactory(&socket_factory_); |
| 500 | |
| 501 | Initialize(); |
| 502 | CompleteCryptoHandshake(); |
| 503 | |
| 504 | // Request a stream and cancel it without releasing the stream. |
rch | f0b18c8a | 2017-05-05 19:31:57 | [diff] [blame] | 505 | std::unique_ptr<QuicChromiumClientSession::Handle> handle = |
Ryan Hamilton | 6c2a2a8 | 2017-12-15 02:06:28 | [diff] [blame] | 506 | session_->CreateHandle(destination_); |
rch | 1baa747 | 2017-04-26 05:43:58 | [diff] [blame] | 507 | TestCompletionCallback callback; |
rch | f0b18c8a | 2017-05-05 19:31:57 | [diff] [blame] | 508 | ASSERT_EQ(OK, handle->RequestStream(/*requires_confirmation=*/false, |
Ramin Halavati | 683bcaa9 | 2018-02-14 08:42:39 | [diff] [blame] | 509 | callback.callback(), |
| 510 | TRAFFIC_ANNOTATION_FOR_TESTS)); |
rch | f0b18c8a | 2017-05-05 19:31:57 | [diff] [blame] | 511 | handle.reset(); |
rch | 1baa747 | 2017-04-26 05:43:58 | [diff] [blame] | 512 | |
| 513 | quic_data.Resume(); |
| 514 | EXPECT_TRUE(quic_data.AllReadDataConsumed()); |
| 515 | EXPECT_TRUE(quic_data.AllWriteDataConsumed()); |
| 516 | } |
| 517 | |
| 518 | TEST_P(QuicChromiumClientSessionTest, AsyncStreamRequest) { |
Ryan Hamilton | abad59e | 2019-06-06 04:02:59 | [diff] [blame] | 519 | MockQuicData quic_data(version_); |
Nick Harper | 23290b8 | 2019-05-02 00:02:56 | [diff] [blame] | 520 | if (version_.transport_version == quic::QUIC_VERSION_99) { |
Renjie Tang | aadb84b | 2019-08-31 01:00:23 | [diff] [blame] | 521 | quic_data.AddWrite(SYNCHRONOUS, client_maker_.MakeInitialSettingsPacket(1)); |
Frank Kastenholz | 878763bf | 2018-11-28 19:14:48 | [diff] [blame] | 522 | // The open stream limit is set to 50 by |
| 523 | // MockCryptoClientStream::SetConfigNegotiated() so when the 51st stream is |
David Schinazi | cc1bc59 | 2019-04-24 19:40:31 | [diff] [blame] | 524 | // requested, a STREAMS_BLOCKED will be sent, indicating that it's blocked |
Fan Yang | 2330d18 | 2019-08-05 14:50:50 | [diff] [blame] | 525 | // at the limit of 50. |
David Schinazi | cc1bc59 | 2019-04-24 19:40:31 | [diff] [blame] | 526 | quic_data.AddWrite(SYNCHRONOUS, client_maker_.MakeStreamsBlockedPacket( |
Fan Yang | 2330d18 | 2019-08-05 14:50:50 | [diff] [blame] | 527 | 2, true, 50, |
David Schinazi | cc1bc59 | 2019-04-24 19:40:31 | [diff] [blame] | 528 | /*unidirectional=*/false)); |
Fan Yang | 32c5a11 | 2018-12-10 20:06:33 | [diff] [blame] | 529 | quic_data.AddWrite( |
| 530 | SYNCHRONOUS, client_maker_.MakeRstPacket( |
| 531 | 3, true, GetNthClientInitiatedBidirectionalStreamId(0), |
Ryan Hamilton | b5d4c5a | 2019-06-21 22:08:41 | [diff] [blame] | 532 | quic::QUIC_STREAM_CANCELLED, |
Frank Kastenholz | 684ea41 | 2019-02-13 18:48:18 | [diff] [blame] | 533 | /*include_stop_sending_if_v99=*/false)); |
David Schinazi | cc1bc59 | 2019-04-24 19:40:31 | [diff] [blame] | 534 | // After the STREAMS_BLOCKED is sent, receive a MAX_STREAMS to increase |
Fan Yang | 2330d18 | 2019-08-05 14:50:50 | [diff] [blame] | 535 | // the limit to 52. |
Fan Yang | 32c5a11 | 2018-12-10 20:06:33 | [diff] [blame] | 536 | quic_data.AddRead( |
Fan Yang | 2330d18 | 2019-08-05 14:50:50 | [diff] [blame] | 537 | ASYNC, server_maker_.MakeMaxStreamsPacket(1, true, 52, |
David Schinazi | cc1bc59 | 2019-04-24 19:40:31 | [diff] [blame] | 538 | /*unidirectional=*/false)); |
Frank Kastenholz | c9b9bea | 2018-12-03 20:13:47 | [diff] [blame] | 539 | } else { |
| 540 | quic_data.AddWrite( |
Fan Yang | 32c5a11 | 2018-12-10 20:06:33 | [diff] [blame] | 541 | SYNCHRONOUS, client_maker_.MakeRstPacket( |
Renjie Tang | aadb84b | 2019-08-31 01:00:23 | [diff] [blame] | 542 | 1, true, GetNthClientInitiatedBidirectionalStreamId(0), |
Fan Yang | 32c5a11 | 2018-12-10 20:06:33 | [diff] [blame] | 543 | quic::QUIC_RST_ACKNOWLEDGEMENT)); |
Frank Kastenholz | 878763bf | 2018-11-28 19:14:48 | [diff] [blame] | 544 | } |
rch | 1baa747 | 2017-04-26 05:43:58 | [diff] [blame] | 545 | quic_data.AddRead(ASYNC, ERR_IO_PENDING); |
| 546 | quic_data.AddRead(ASYNC, OK); // EOF |
| 547 | quic_data.AddSocketDataToFactory(&socket_factory_); |
rch | 1baa747 | 2017-04-26 05:43:58 | [diff] [blame] | 548 | Initialize(); |
| 549 | CompleteCryptoHandshake(); |
| 550 | |
| 551 | // Open the maximum number of streams so that a subsequent request |
| 552 | // can not proceed immediately. |
Fan Yang | 32c5a11 | 2018-12-10 20:06:33 | [diff] [blame] | 553 | const size_t kMaxOpenStreams = GetMaxAllowedOutgoingBidirectionalStreams(); |
rch | 1baa747 | 2017-04-26 05:43:58 | [diff] [blame] | 554 | for (size_t i = 0; i < kMaxOpenStreams; i++) { |
Fan Yang | 5a3bddf | 2018-10-12 10:05:50 | [diff] [blame] | 555 | QuicChromiumClientSessionPeer::CreateOutgoingStream(session_.get()); |
rch | 1baa747 | 2017-04-26 05:43:58 | [diff] [blame] | 556 | } |
| 557 | EXPECT_EQ(kMaxOpenStreams, session_->GetNumOpenOutgoingStreams()); |
| 558 | |
| 559 | // Request a stream and verify that it's pending. |
rch | f0b18c8a | 2017-05-05 19:31:57 | [diff] [blame] | 560 | std::unique_ptr<QuicChromiumClientSession::Handle> handle = |
Ryan Hamilton | 6c2a2a8 | 2017-12-15 02:06:28 | [diff] [blame] | 561 | session_->CreateHandle(destination_); |
rch | 1baa747 | 2017-04-26 05:43:58 | [diff] [blame] | 562 | TestCompletionCallback callback; |
Ramin Halavati | 683bcaa9 | 2018-02-14 08:42:39 | [diff] [blame] | 563 | ASSERT_EQ( |
| 564 | ERR_IO_PENDING, |
| 565 | handle->RequestStream(/*requires_confirmation=*/false, |
| 566 | callback.callback(), TRAFFIC_ANNOTATION_FOR_TESTS)); |
rch | 1baa747 | 2017-04-26 05:43:58 | [diff] [blame] | 567 | |
| 568 | // Close a stream and ensure the stream request completes. |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 569 | quic::QuicRstStreamFrame rst(quic::kInvalidControlFrameId, |
Fan Yang | 32c5a11 | 2018-12-10 20:06:33 | [diff] [blame] | 570 | GetNthClientInitiatedBidirectionalStreamId(0), |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 571 | quic::QUIC_STREAM_CANCELLED, 0); |
rch | 1baa747 | 2017-04-26 05:43:58 | [diff] [blame] | 572 | session_->OnRstStream(rst); |
Nick Harper | 23290b8 | 2019-05-02 00:02:56 | [diff] [blame] | 573 | if (version_.transport_version == quic::QUIC_VERSION_99) { |
Frank Kastenholz | 684ea41 | 2019-02-13 18:48:18 | [diff] [blame] | 574 | // For version99, to close the stream completely, we also must receive a |
| 575 | // STOP_SENDING frame: |
| 576 | quic::QuicStopSendingFrame stop_sending( |
| 577 | quic::kInvalidControlFrameId, |
| 578 | GetNthClientInitiatedBidirectionalStreamId(0), |
| 579 | quic::QUIC_STREAM_CANCELLED); |
| 580 | session_->OnStopSendingFrame(stop_sending); |
| 581 | } |
Frank Kastenholz | 878763bf | 2018-11-28 19:14:48 | [diff] [blame] | 582 | // Pump the message loop to read the max stream id packet. |
| 583 | base::RunLoop().RunUntilIdle(); |
| 584 | |
rch | 1baa747 | 2017-04-26 05:43:58 | [diff] [blame] | 585 | ASSERT_TRUE(callback.have_result()); |
| 586 | EXPECT_THAT(callback.WaitForResult(), IsOk()); |
rch | 1bcfddf2 | 2017-06-03 00:26:29 | [diff] [blame] | 587 | EXPECT_TRUE(handle->ReleaseStream() != nullptr); |
rch | 1baa747 | 2017-04-26 05:43:58 | [diff] [blame] | 588 | |
| 589 | quic_data.Resume(); |
| 590 | EXPECT_TRUE(quic_data.AllReadDataConsumed()); |
| 591 | EXPECT_TRUE(quic_data.AllWriteDataConsumed()); |
| 592 | } |
| 593 | |
Ryan Hamilton | 9883ff6 | 2017-08-01 06:07:05 | [diff] [blame] | 594 | TEST_P(QuicChromiumClientSessionTest, ClosedWithAsyncStreamRequest) { |
Ryan Hamilton | abad59e | 2019-06-06 04:02:59 | [diff] [blame] | 595 | MockQuicData quic_data(version_); |
Nick Harper | 23290b8 | 2019-05-02 00:02:56 | [diff] [blame] | 596 | if (version_.transport_version == quic::QUIC_VERSION_99) { |
Renjie Tang | aadb84b | 2019-08-31 01:00:23 | [diff] [blame] | 597 | quic_data.AddWrite(SYNCHRONOUS, client_maker_.MakeInitialSettingsPacket(1)); |
Frank Kastenholz | c9b9bea | 2018-12-03 20:13:47 | [diff] [blame] | 598 | // The open stream limit is set to 50 by |
| 599 | // MockCryptoClientStream::SetConfigNegotiated() so when the 51st stream is |
David Schinazi | cc1bc59 | 2019-04-24 19:40:31 | [diff] [blame] | 600 | // requested, a STREAMS_BLOCKED will be sent, indicating that it's blocked |
Fan Yang | 2330d18 | 2019-08-05 14:50:50 | [diff] [blame] | 601 | // at the limit of 50. |
David Schinazi | cc1bc59 | 2019-04-24 19:40:31 | [diff] [blame] | 602 | quic_data.AddWrite(SYNCHRONOUS, client_maker_.MakeStreamsBlockedPacket( |
Fan Yang | 2330d18 | 2019-08-05 14:50:50 | [diff] [blame] | 603 | 2, true, 50, |
David Schinazi | cc1bc59 | 2019-04-24 19:40:31 | [diff] [blame] | 604 | /*unidirectional=*/false)); |
| 605 | quic_data.AddWrite(SYNCHRONOUS, client_maker_.MakeStreamsBlockedPacket( |
Fan Yang | 2330d18 | 2019-08-05 14:50:50 | [diff] [blame] | 606 | 3, true, 50, |
David Schinazi | cc1bc59 | 2019-04-24 19:40:31 | [diff] [blame] | 607 | /*unidirectional=*/false)); |
Frank Kastenholz | c9b9bea | 2018-12-03 20:13:47 | [diff] [blame] | 608 | } |
Ryan Hamilton | 9883ff6 | 2017-08-01 06:07:05 | [diff] [blame] | 609 | quic_data.AddRead(ASYNC, ERR_IO_PENDING); |
| 610 | quic_data.AddRead(ASYNC, OK); // EOF |
| 611 | quic_data.AddSocketDataToFactory(&socket_factory_); |
| 612 | |
| 613 | Initialize(); |
| 614 | CompleteCryptoHandshake(); |
| 615 | |
| 616 | // Open the maximum number of streams so that a subsequent request |
| 617 | // can not proceed immediately. |
Fan Yang | 32c5a11 | 2018-12-10 20:06:33 | [diff] [blame] | 618 | const size_t kMaxOpenStreams = GetMaxAllowedOutgoingBidirectionalStreams(); |
Ryan Hamilton | 9883ff6 | 2017-08-01 06:07:05 | [diff] [blame] | 619 | for (size_t i = 0; i < kMaxOpenStreams; i++) { |
Fan Yang | 5a3bddf | 2018-10-12 10:05:50 | [diff] [blame] | 620 | QuicChromiumClientSessionPeer::CreateOutgoingStream(session_.get()); |
Ryan Hamilton | 9883ff6 | 2017-08-01 06:07:05 | [diff] [blame] | 621 | } |
| 622 | EXPECT_EQ(kMaxOpenStreams, session_->GetNumOpenOutgoingStreams()); |
| 623 | |
| 624 | // Request two streams which will both be pending. |
Frank Kastenholz | c9b9bea | 2018-12-03 20:13:47 | [diff] [blame] | 625 | // In V99 each will generate a max stream id for each attempt. |
Ryan Hamilton | 9883ff6 | 2017-08-01 06:07:05 | [diff] [blame] | 626 | std::unique_ptr<QuicChromiumClientSession::Handle> handle = |
Ryan Hamilton | 6c2a2a8 | 2017-12-15 02:06:28 | [diff] [blame] | 627 | session_->CreateHandle(destination_); |
Ryan Hamilton | 9883ff6 | 2017-08-01 06:07:05 | [diff] [blame] | 628 | std::unique_ptr<QuicChromiumClientSession::Handle> handle2 = |
Ryan Hamilton | 6c2a2a8 | 2017-12-15 02:06:28 | [diff] [blame] | 629 | session_->CreateHandle(destination_); |
Ryan Hamilton | 9883ff6 | 2017-08-01 06:07:05 | [diff] [blame] | 630 | |
Yannic Bonenberger | 3c96beb | 2019-09-03 20:41:37 | [diff] [blame] | 631 | ASSERT_EQ( |
| 632 | ERR_IO_PENDING, |
| 633 | handle->RequestStream( |
| 634 | /*requires_confirmation=*/false, |
| 635 | base::BindOnce(&QuicChromiumClientSessionTest::ResetHandleOnError, |
| 636 | base::Unretained(this), &handle2), |
| 637 | TRAFFIC_ANNOTATION_FOR_TESTS)); |
Ryan Hamilton | 9883ff6 | 2017-08-01 06:07:05 | [diff] [blame] | 638 | |
| 639 | TestCompletionCallback callback2; |
| 640 | ASSERT_EQ(ERR_IO_PENDING, |
| 641 | handle2->RequestStream(/*requires_confirmation=*/false, |
Ramin Halavati | 683bcaa9 | 2018-02-14 08:42:39 | [diff] [blame] | 642 | callback2.callback(), |
| 643 | TRAFFIC_ANNOTATION_FOR_TESTS)); |
Ryan Hamilton | 9883ff6 | 2017-08-01 06:07:05 | [diff] [blame] | 644 | |
| 645 | session_->connection()->CloseConnection( |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 646 | quic::QUIC_NETWORK_IDLE_TIMEOUT, "Timed out", |
| 647 | quic::ConnectionCloseBehavior::SILENT_CLOSE); |
Ryan Hamilton | 9883ff6 | 2017-08-01 06:07:05 | [diff] [blame] | 648 | |
| 649 | // Pump the message loop to read the connection close packet. |
| 650 | base::RunLoop().RunUntilIdle(); |
| 651 | EXPECT_FALSE(handle2.get()); |
| 652 | quic_data.Resume(); |
| 653 | EXPECT_TRUE(quic_data.AllReadDataConsumed()); |
| 654 | EXPECT_TRUE(quic_data.AllWriteDataConsumed()); |
| 655 | } |
| 656 | |
rch | 1baa747 | 2017-04-26 05:43:58 | [diff] [blame] | 657 | TEST_P(QuicChromiumClientSessionTest, CancelPendingStreamRequest) { |
Ryan Hamilton | abad59e | 2019-06-06 04:02:59 | [diff] [blame] | 658 | MockQuicData quic_data(version_); |
Nick Harper | 23290b8 | 2019-05-02 00:02:56 | [diff] [blame] | 659 | if (version_.transport_version == quic::QUIC_VERSION_99) { |
Renjie Tang | aadb84b | 2019-08-31 01:00:23 | [diff] [blame] | 660 | quic_data.AddWrite(SYNCHRONOUS, client_maker_.MakeInitialSettingsPacket(1)); |
Frank Kastenholz | 878763bf | 2018-11-28 19:14:48 | [diff] [blame] | 661 | // The open stream limit is set to 50 by |
| 662 | // MockCryptoClientStream::SetConfigNegotiated() so when the 51st stream is |
David Schinazi | cc1bc59 | 2019-04-24 19:40:31 | [diff] [blame] | 663 | // requested, a STREAMS_BLOCKED will be sent. |
| 664 | quic_data.AddWrite(SYNCHRONOUS, client_maker_.MakeStreamsBlockedPacket( |
Fan Yang | 2330d18 | 2019-08-05 14:50:50 | [diff] [blame] | 665 | 2, true, 50, |
David Schinazi | cc1bc59 | 2019-04-24 19:40:31 | [diff] [blame] | 666 | /*unidirectional=*/false)); |
Frank Kastenholz | 684ea41 | 2019-02-13 18:48:18 | [diff] [blame] | 667 | // This node receives the RST_STREAM+STOP_SENDING, it responds |
| 668 | // with only a RST_STREAM. |
Fan Yang | 32c5a11 | 2018-12-10 20:06:33 | [diff] [blame] | 669 | quic_data.AddWrite( |
| 670 | SYNCHRONOUS, client_maker_.MakeRstPacket( |
| 671 | 3, true, GetNthClientInitiatedBidirectionalStreamId(0), |
Ryan Hamilton | b5d4c5a | 2019-06-21 22:08:41 | [diff] [blame] | 672 | quic::QUIC_STREAM_CANCELLED, |
Frank Kastenholz | 684ea41 | 2019-02-13 18:48:18 | [diff] [blame] | 673 | /*include_stop_sending_if_v99=*/false)); |
Frank Kastenholz | c9b9bea | 2018-12-03 20:13:47 | [diff] [blame] | 674 | } else { |
| 675 | quic_data.AddWrite( |
Fan Yang | 32c5a11 | 2018-12-10 20:06:33 | [diff] [blame] | 676 | SYNCHRONOUS, client_maker_.MakeRstPacket( |
Renjie Tang | aadb84b | 2019-08-31 01:00:23 | [diff] [blame] | 677 | 1, true, GetNthClientInitiatedBidirectionalStreamId(0), |
Fan Yang | 32c5a11 | 2018-12-10 20:06:33 | [diff] [blame] | 678 | quic::QUIC_RST_ACKNOWLEDGEMENT)); |
Frank Kastenholz | 878763bf | 2018-11-28 19:14:48 | [diff] [blame] | 679 | } |
rch | 1baa747 | 2017-04-26 05:43:58 | [diff] [blame] | 680 | quic_data.AddRead(ASYNC, ERR_IO_PENDING); |
| 681 | quic_data.AddRead(ASYNC, OK); // EOF |
| 682 | quic_data.AddSocketDataToFactory(&socket_factory_); |
| 683 | |
| 684 | Initialize(); |
| 685 | CompleteCryptoHandshake(); |
| 686 | |
| 687 | // Open the maximum number of streams so that a subsequent request |
| 688 | // can not proceed immediately. |
Fan Yang | 32c5a11 | 2018-12-10 20:06:33 | [diff] [blame] | 689 | const size_t kMaxOpenStreams = GetMaxAllowedOutgoingBidirectionalStreams(); |
rch | 1baa747 | 2017-04-26 05:43:58 | [diff] [blame] | 690 | for (size_t i = 0; i < kMaxOpenStreams; i++) { |
Fan Yang | 5a3bddf | 2018-10-12 10:05:50 | [diff] [blame] | 691 | QuicChromiumClientSessionPeer::CreateOutgoingStream(session_.get()); |
rch | 1baa747 | 2017-04-26 05:43:58 | [diff] [blame] | 692 | } |
| 693 | EXPECT_EQ(kMaxOpenStreams, session_->GetNumOpenOutgoingStreams()); |
| 694 | |
| 695 | // Request a stream and verify that it's pending. |
rch | f0b18c8a | 2017-05-05 19:31:57 | [diff] [blame] | 696 | std::unique_ptr<QuicChromiumClientSession::Handle> handle = |
Ryan Hamilton | 6c2a2a8 | 2017-12-15 02:06:28 | [diff] [blame] | 697 | session_->CreateHandle(destination_); |
rch | 1baa747 | 2017-04-26 05:43:58 | [diff] [blame] | 698 | TestCompletionCallback callback; |
Ramin Halavati | 683bcaa9 | 2018-02-14 08:42:39 | [diff] [blame] | 699 | ASSERT_EQ( |
| 700 | ERR_IO_PENDING, |
| 701 | handle->RequestStream(/*requires_confirmation=*/false, |
| 702 | callback.callback(), TRAFFIC_ANNOTATION_FOR_TESTS)); |
rch | 1baa747 | 2017-04-26 05:43:58 | [diff] [blame] | 703 | |
| 704 | // Cancel the pending stream request. |
rch | f0b18c8a | 2017-05-05 19:31:57 | [diff] [blame] | 705 | handle.reset(); |
rch | 1baa747 | 2017-04-26 05:43:58 | [diff] [blame] | 706 | |
| 707 | // Close a stream and ensure that no new stream is created. |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 708 | quic::QuicRstStreamFrame rst(quic::kInvalidControlFrameId, |
Fan Yang | 32c5a11 | 2018-12-10 20:06:33 | [diff] [blame] | 709 | GetNthClientInitiatedBidirectionalStreamId(0), |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 710 | quic::QUIC_STREAM_CANCELLED, 0); |
rch | 1baa747 | 2017-04-26 05:43:58 | [diff] [blame] | 711 | session_->OnRstStream(rst); |
Nick Harper | 23290b8 | 2019-05-02 00:02:56 | [diff] [blame] | 712 | if (version_.transport_version == quic::QUIC_VERSION_99) { |
Frank Kastenholz | 684ea41 | 2019-02-13 18:48:18 | [diff] [blame] | 713 | // For version99, we require a STOP_SENDING as well as a RESET_STREAM to |
| 714 | // fully close the stream. |
| 715 | quic::QuicStopSendingFrame stop_sending( |
| 716 | quic::kInvalidControlFrameId, |
| 717 | GetNthClientInitiatedBidirectionalStreamId(0), |
| 718 | quic::QUIC_STREAM_CANCELLED); |
| 719 | session_->OnStopSendingFrame(stop_sending); |
| 720 | } |
rch | 1baa747 | 2017-04-26 05:43:58 | [diff] [blame] | 721 | EXPECT_EQ(kMaxOpenStreams - 1, session_->GetNumOpenOutgoingStreams()); |
| 722 | |
| 723 | quic_data.Resume(); |
| 724 | EXPECT_TRUE(quic_data.AllReadDataConsumed()); |
| 725 | EXPECT_TRUE(quic_data.AllWriteDataConsumed()); |
| 726 | } |
| 727 | |
rch | 94c26d68 | 2017-04-29 02:49:27 | [diff] [blame] | 728 | TEST_P(QuicChromiumClientSessionTest, ConnectionCloseBeforeStreamRequest) { |
Ryan Hamilton | abad59e | 2019-06-06 04:02:59 | [diff] [blame] | 729 | MockQuicData quic_data(version_); |
Renjie Tang | aadb84b | 2019-08-31 01:00:23 | [diff] [blame] | 730 | int packet_num = 1; |
| 731 | if (VersionUsesQpack(version_.transport_version)) { |
| 732 | quic_data.AddWrite(SYNCHRONOUS, |
| 733 | client_maker_.MakeInitialSettingsPacket(packet_num++)); |
| 734 | } |
| 735 | quic_data.AddWrite(SYNCHRONOUS, |
| 736 | client_maker_.MakePingPacket(packet_num++, true)); |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 737 | quic_data.AddRead( |
| 738 | ASYNC, |
| 739 | server_maker_.MakeConnectionClosePacket( |
| 740 | 1, false, quic::QUIC_CRYPTO_VERSION_NOT_SUPPORTED, "Time to panic!")); |
Renjie Tang | aadb84b | 2019-08-31 01:00:23 | [diff] [blame] | 741 | |
rch | 94c26d68 | 2017-04-29 02:49:27 | [diff] [blame] | 742 | quic_data.AddSocketDataToFactory(&socket_factory_); |
| 743 | |
| 744 | Initialize(); |
| 745 | CompleteCryptoHandshake(); |
| 746 | |
Renjie Tang | aadb84b | 2019-08-31 01:00:23 | [diff] [blame] | 747 | // Send a ping so that client has outgoing traffic before receiving packets. |
| 748 | session_->SendPing(); |
| 749 | |
rch | 94c26d68 | 2017-04-29 02:49:27 | [diff] [blame] | 750 | // Pump the message loop to read the connection close packet. |
| 751 | base::RunLoop().RunUntilIdle(); |
| 752 | |
| 753 | // Request a stream and verify that it failed. |
rch | f0b18c8a | 2017-05-05 19:31:57 | [diff] [blame] | 754 | std::unique_ptr<QuicChromiumClientSession::Handle> handle = |
Ryan Hamilton | 6c2a2a8 | 2017-12-15 02:06:28 | [diff] [blame] | 755 | session_->CreateHandle(destination_); |
rch | 94c26d68 | 2017-04-29 02:49:27 | [diff] [blame] | 756 | TestCompletionCallback callback; |
Ramin Halavati | 683bcaa9 | 2018-02-14 08:42:39 | [diff] [blame] | 757 | ASSERT_EQ( |
| 758 | ERR_CONNECTION_CLOSED, |
| 759 | handle->RequestStream(/*requires_confirmation=*/false, |
| 760 | callback.callback(), TRAFFIC_ANNOTATION_FOR_TESTS)); |
rch | 94c26d68 | 2017-04-29 02:49:27 | [diff] [blame] | 761 | |
| 762 | EXPECT_TRUE(quic_data.AllReadDataConsumed()); |
| 763 | EXPECT_TRUE(quic_data.AllWriteDataConsumed()); |
| 764 | } |
| 765 | |
rch | d606b6c3 | 2017-05-02 17:32:57 | [diff] [blame] | 766 | TEST_P(QuicChromiumClientSessionTest, ConnectionCloseBeforeHandshakeConfirmed) { |
Ryan Hamilton | 8380c65 | 2019-06-04 02:25:06 | [diff] [blame] | 767 | if (version_.handshake_protocol == quic::PROTOCOL_TLS1_3) { |
| 768 | // TODO(nharper, b/112643533): Figure out why this test fails when TLS is |
| 769 | // enabled and fix it. |
| 770 | return; |
| 771 | } |
| 772 | |
David Schinazi | c828105 | 2019-01-24 06:14:17 | [diff] [blame] | 773 | // Force the connection close packet to use long headers with connection ID. |
Ryan Hamilton | 840358c | 2019-03-19 19:00:35 | [diff] [blame] | 774 | server_maker_.SetEncryptionLevel(quic::ENCRYPTION_INITIAL); |
David Schinazi | c828105 | 2019-01-24 06:14:17 | [diff] [blame] | 775 | |
Ryan Hamilton | abad59e | 2019-06-06 04:02:59 | [diff] [blame] | 776 | MockQuicData quic_data(version_); |
rch | d606b6c3 | 2017-05-02 17:32:57 | [diff] [blame] | 777 | quic_data.AddRead(ASYNC, ERR_IO_PENDING); |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 778 | quic_data.AddRead( |
| 779 | ASYNC, |
| 780 | server_maker_.MakeConnectionClosePacket( |
| 781 | 1, false, quic::QUIC_CRYPTO_VERSION_NOT_SUPPORTED, "Time to panic!")); |
rch | d606b6c3 | 2017-05-02 17:32:57 | [diff] [blame] | 782 | quic_data.AddSocketDataToFactory(&socket_factory_); |
| 783 | |
| 784 | Initialize(); |
| 785 | |
| 786 | // Request a stream and verify that it's pending. |
rch | f0b18c8a | 2017-05-05 19:31:57 | [diff] [blame] | 787 | std::unique_ptr<QuicChromiumClientSession::Handle> handle = |
Ryan Hamilton | 6c2a2a8 | 2017-12-15 02:06:28 | [diff] [blame] | 788 | session_->CreateHandle(destination_); |
rch | d606b6c3 | 2017-05-02 17:32:57 | [diff] [blame] | 789 | TestCompletionCallback callback; |
Ramin Halavati | 683bcaa9 | 2018-02-14 08:42:39 | [diff] [blame] | 790 | ASSERT_EQ( |
| 791 | ERR_IO_PENDING, |
| 792 | handle->RequestStream(/*requires_confirmation=*/true, callback.callback(), |
| 793 | TRAFFIC_ANNOTATION_FOR_TESTS)); |
rch | d606b6c3 | 2017-05-02 17:32:57 | [diff] [blame] | 794 | |
| 795 | // Close the connection and verify that the StreamRequest completes with |
| 796 | // an error. |
| 797 | quic_data.Resume(); |
| 798 | base::RunLoop().RunUntilIdle(); |
| 799 | |
| 800 | EXPECT_THAT(callback.WaitForResult(), IsError(ERR_CONNECTION_CLOSED)); |
| 801 | |
| 802 | EXPECT_TRUE(quic_data.AllReadDataConsumed()); |
| 803 | EXPECT_TRUE(quic_data.AllWriteDataConsumed()); |
| 804 | } |
| 805 | |
rch | 94c26d68 | 2017-04-29 02:49:27 | [diff] [blame] | 806 | TEST_P(QuicChromiumClientSessionTest, ConnectionCloseWithPendingStreamRequest) { |
Ryan Hamilton | abad59e | 2019-06-06 04:02:59 | [diff] [blame] | 807 | MockQuicData quic_data(version_); |
Renjie Tang | aadb84b | 2019-08-31 01:00:23 | [diff] [blame] | 808 | int packet_num = 1; |
| 809 | if (version_.transport_version == quic::QUIC_VERSION_99) { |
| 810 | quic_data.AddWrite(SYNCHRONOUS, |
| 811 | client_maker_.MakeInitialSettingsPacket(packet_num++)); |
| 812 | } |
| 813 | quic_data.AddWrite(SYNCHRONOUS, |
| 814 | client_maker_.MakePingPacket(packet_num++, true)); |
Nick Harper | 23290b8 | 2019-05-02 00:02:56 | [diff] [blame] | 815 | if (version_.transport_version == quic::QUIC_VERSION_99) { |
David Schinazi | cc1bc59 | 2019-04-24 19:40:31 | [diff] [blame] | 816 | quic_data.AddWrite(SYNCHRONOUS, client_maker_.MakeStreamsBlockedPacket( |
Renjie Tang | aadb84b | 2019-08-31 01:00:23 | [diff] [blame] | 817 | packet_num++, true, 50, |
David Schinazi | cc1bc59 | 2019-04-24 19:40:31 | [diff] [blame] | 818 | /*unidirectional=*/false)); |
Frank Kastenholz | c9b9bea | 2018-12-03 20:13:47 | [diff] [blame] | 819 | } |
rch | 94c26d68 | 2017-04-29 02:49:27 | [diff] [blame] | 820 | quic_data.AddRead(ASYNC, ERR_IO_PENDING); |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 821 | quic_data.AddRead( |
| 822 | ASYNC, |
| 823 | server_maker_.MakeConnectionClosePacket( |
| 824 | 1, false, quic::QUIC_CRYPTO_VERSION_NOT_SUPPORTED, "Time to panic!")); |
rch | 94c26d68 | 2017-04-29 02:49:27 | [diff] [blame] | 825 | quic_data.AddSocketDataToFactory(&socket_factory_); |
| 826 | |
| 827 | Initialize(); |
| 828 | CompleteCryptoHandshake(); |
| 829 | |
Renjie Tang | aadb84b | 2019-08-31 01:00:23 | [diff] [blame] | 830 | // Send a ping so that client has outgoing traffic before receiving packets. |
| 831 | session_->SendPing(); |
| 832 | |
rch | 94c26d68 | 2017-04-29 02:49:27 | [diff] [blame] | 833 | // Open the maximum number of streams so that a subsequent request |
| 834 | // can not proceed immediately. |
Fan Yang | 32c5a11 | 2018-12-10 20:06:33 | [diff] [blame] | 835 | const size_t kMaxOpenStreams = GetMaxAllowedOutgoingBidirectionalStreams(); |
rch | 94c26d68 | 2017-04-29 02:49:27 | [diff] [blame] | 836 | for (size_t i = 0; i < kMaxOpenStreams; i++) { |
Fan Yang | 5a3bddf | 2018-10-12 10:05:50 | [diff] [blame] | 837 | QuicChromiumClientSessionPeer::CreateOutgoingStream(session_.get()); |
rch | 94c26d68 | 2017-04-29 02:49:27 | [diff] [blame] | 838 | } |
| 839 | EXPECT_EQ(kMaxOpenStreams, session_->GetNumOpenOutgoingStreams()); |
| 840 | |
| 841 | // Request a stream and verify that it's pending. |
rch | f0b18c8a | 2017-05-05 19:31:57 | [diff] [blame] | 842 | std::unique_ptr<QuicChromiumClientSession::Handle> handle = |
Ryan Hamilton | 6c2a2a8 | 2017-12-15 02:06:28 | [diff] [blame] | 843 | session_->CreateHandle(destination_); |
rch | 94c26d68 | 2017-04-29 02:49:27 | [diff] [blame] | 844 | TestCompletionCallback callback; |
Ramin Halavati | 683bcaa9 | 2018-02-14 08:42:39 | [diff] [blame] | 845 | ASSERT_EQ( |
| 846 | ERR_IO_PENDING, |
| 847 | handle->RequestStream(/*requires_confirmation=*/false, |
| 848 | callback.callback(), TRAFFIC_ANNOTATION_FOR_TESTS)); |
rch | 94c26d68 | 2017-04-29 02:49:27 | [diff] [blame] | 849 | |
| 850 | // Close the connection and verify that the StreamRequest completes with |
| 851 | // an error. |
| 852 | quic_data.Resume(); |
| 853 | base::RunLoop().RunUntilIdle(); |
| 854 | |
| 855 | EXPECT_THAT(callback.WaitForResult(), IsError(ERR_CONNECTION_CLOSED)); |
| 856 | |
| 857 | EXPECT_TRUE(quic_data.AllReadDataConsumed()); |
| 858 | EXPECT_TRUE(quic_data.AllWriteDataConsumed()); |
| 859 | } |
| 860 | |
ckrasic | 4f9d88d | 2015-07-22 22:23:16 | [diff] [blame] | 861 | TEST_P(QuicChromiumClientSessionTest, MaxNumStreams) { |
Ryan Hamilton | abad59e | 2019-06-06 04:02:59 | [diff] [blame] | 862 | MockQuicData quic_data(version_); |
Nick Harper | 23290b8 | 2019-05-02 00:02:56 | [diff] [blame] | 863 | if (version_.transport_version == quic::QUIC_VERSION_99) { |
Renjie Tang | aadb84b | 2019-08-31 01:00:23 | [diff] [blame] | 864 | quic_data.AddWrite(SYNCHRONOUS, client_maker_.MakeInitialSettingsPacket(1)); |
David Schinazi | cc1bc59 | 2019-04-24 19:40:31 | [diff] [blame] | 865 | // Initial configuration is 50 dynamic streams. Taking into account |
Ryan Hamilton | 4aeec56 | 2019-05-17 21:22:52 | [diff] [blame] | 866 | // the static stream (headers), expect to block on when hitting the limit |
Fan Yang | 2330d18 | 2019-08-05 14:50:50 | [diff] [blame] | 867 | // of 50 streams |
David Schinazi | cc1bc59 | 2019-04-24 19:40:31 | [diff] [blame] | 868 | quic_data.AddWrite(SYNCHRONOUS, client_maker_.MakeStreamsBlockedPacket( |
Fan Yang | 2330d18 | 2019-08-05 14:50:50 | [diff] [blame] | 869 | 2, true, 50, |
David Schinazi | cc1bc59 | 2019-04-24 19:40:31 | [diff] [blame] | 870 | /*unidirectional=*/false)); |
Fan Yang | 32c5a11 | 2018-12-10 20:06:33 | [diff] [blame] | 871 | quic_data.AddWrite( |
| 872 | SYNCHRONOUS, client_maker_.MakeRstPacket( |
| 873 | 3, true, GetNthClientInitiatedBidirectionalStreamId(0), |
| 874 | quic::QUIC_RST_ACKNOWLEDGEMENT)); |
David Schinazi | cc1bc59 | 2019-04-24 19:40:31 | [diff] [blame] | 875 | // For the second CreateOutgoingStream that fails because of hitting the |
| 876 | // stream count limit. |
| 877 | quic_data.AddWrite(SYNCHRONOUS, client_maker_.MakeStreamsBlockedPacket( |
Fan Yang | 2330d18 | 2019-08-05 14:50:50 | [diff] [blame] | 878 | 4, true, 50, |
David Schinazi | cc1bc59 | 2019-04-24 19:40:31 | [diff] [blame] | 879 | /*unidirectional=*/false)); |
Fan Yang | 32c5a11 | 2018-12-10 20:06:33 | [diff] [blame] | 880 | quic_data.AddRead( |
Ryan Hamilton | 4aeec56 | 2019-05-17 21:22:52 | [diff] [blame] | 881 | ASYNC, server_maker_.MakeMaxStreamsPacket(1, true, 50 + 2, |
David Schinazi | cc1bc59 | 2019-04-24 19:40:31 | [diff] [blame] | 882 | /*unidirectional=*/false)); |
Frank Kastenholz | c9b9bea | 2018-12-03 20:13:47 | [diff] [blame] | 883 | } else { |
| 884 | quic_data.AddWrite( |
Fan Yang | 32c5a11 | 2018-12-10 20:06:33 | [diff] [blame] | 885 | SYNCHRONOUS, client_maker_.MakeRstPacket( |
Renjie Tang | aadb84b | 2019-08-31 01:00:23 | [diff] [blame] | 886 | 1, true, GetNthClientInitiatedBidirectionalStreamId(0), |
Fan Yang | 32c5a11 | 2018-12-10 20:06:33 | [diff] [blame] | 887 | quic::QUIC_RST_ACKNOWLEDGEMENT)); |
Frank Kastenholz | 878763bf | 2018-11-28 19:14:48 | [diff] [blame] | 888 | } |
| 889 | quic_data.AddRead(ASYNC, ERR_IO_PENDING); |
| 890 | quic_data.AddRead(ASYNC, OK); // EOF |
| 891 | quic_data.AddSocketDataToFactory(&socket_factory_); |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 892 | |
| 893 | Initialize(); |
[email protected] | ed3fc15d | 2013-03-08 18:37:44 | [diff] [blame] | 894 | CompleteCryptoHandshake(); |
Fan Yang | 32c5a11 | 2018-12-10 20:06:33 | [diff] [blame] | 895 | const size_t kMaxOpenStreams = GetMaxAllowedOutgoingBidirectionalStreams(); |
[email protected] | dd3fd0e | 2012-11-04 05:14:40 | [diff] [blame] | 896 | |
rch | 12fef55 | 2016-01-15 16:26:31 | [diff] [blame] | 897 | std::vector<QuicChromiumClientStream*> streams; |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 898 | for (size_t i = 0; i < kMaxOpenStreams; i++) { |
Ramin Halavati | f7788ea | 2018-02-26 07:02:57 | [diff] [blame] | 899 | QuicChromiumClientStream* stream = |
Fan Yang | 5a3bddf | 2018-10-12 10:05:50 | [diff] [blame] | 900 | QuicChromiumClientSessionPeer::CreateOutgoingStream(session_.get()); |
[email protected] | dd3fd0e | 2012-11-04 05:14:40 | [diff] [blame] | 901 | EXPECT_TRUE(stream); |
[email protected] | f702d57 | 2012-12-04 15:56:20 | [diff] [blame] | 902 | streams.push_back(stream); |
[email protected] | dd3fd0e | 2012-11-04 05:14:40 | [diff] [blame] | 903 | } |
David Schinazi | cc1bc59 | 2019-04-24 19:40:31 | [diff] [blame] | 904 | // This stream, the 51st dynamic stream, can not be opened. |
Fan Yang | 5a3bddf | 2018-10-12 10:05:50 | [diff] [blame] | 905 | EXPECT_FALSE( |
| 906 | QuicChromiumClientSessionPeer::CreateOutgoingStream(session_.get())); |
[email protected] | dd3fd0e | 2012-11-04 05:14:40 | [diff] [blame] | 907 | |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 908 | EXPECT_EQ(kMaxOpenStreams, session_->GetNumOpenOutgoingStreams()); |
rtenneti | ac06c2f | 2015-11-05 18:12:35 | [diff] [blame] | 909 | |
[email protected] | dd3fd0e | 2012-11-04 05:14:40 | [diff] [blame] | 910 | // Close a stream and ensure I can now open a new one. |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 911 | quic::QuicStreamId stream_id = streams[0]->id(); |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 912 | session_->CloseStream(stream_id); |
rtenneti | ac06c2f | 2015-11-05 18:12:35 | [diff] [blame] | 913 | |
Frank Kastenholz | c9b9bea | 2018-12-03 20:13:47 | [diff] [blame] | 914 | // Pump data, bringing in the max-stream-id |
| 915 | base::RunLoop().RunUntilIdle(); |
| 916 | |
Fan Yang | 5a3bddf | 2018-10-12 10:05:50 | [diff] [blame] | 917 | EXPECT_FALSE( |
| 918 | QuicChromiumClientSessionPeer::CreateOutgoingStream(session_.get())); |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 919 | quic::QuicRstStreamFrame rst1(quic::kInvalidControlFrameId, stream_id, |
| 920 | quic::QUIC_STREAM_NO_ERROR, 0); |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 921 | session_->OnRstStream(rst1); |
| 922 | EXPECT_EQ(kMaxOpenStreams - 1, session_->GetNumOpenOutgoingStreams()); |
Frank Kastenholz | 878763bf | 2018-11-28 19:14:48 | [diff] [blame] | 923 | base::RunLoop().RunUntilIdle(); |
Fan Yang | 5a3bddf | 2018-10-12 10:05:50 | [diff] [blame] | 924 | EXPECT_TRUE( |
| 925 | QuicChromiumClientSessionPeer::CreateOutgoingStream(session_.get())); |
[email protected] | dd3fd0e | 2012-11-04 05:14:40 | [diff] [blame] | 926 | } |
| 927 | |
zhongyi | e34c03566 | 2016-10-19 22:26:02 | [diff] [blame] | 928 | TEST_P(QuicChromiumClientSessionTest, PushStreamTimedOutNoResponse) { |
| 929 | base::HistogramTester histogram_tester; |
Renjie Tang | aadb84b | 2019-08-31 01:00:23 | [diff] [blame] | 930 | MockQuicData quic_data(version_); |
| 931 | int packet_num = 1; |
| 932 | if (VersionUsesQpack(version_.transport_version)) { |
| 933 | quic_data.AddWrite(ASYNC, |
| 934 | client_maker_.MakeInitialSettingsPacket(packet_num++)); |
| 935 | } |
| 936 | quic_data.AddWrite(ASYNC, client_maker_.MakeRstPacket( |
| 937 | packet_num++, true, |
| 938 | GetNthServerInitiatedUnidirectionalStreamId(0), |
| 939 | quic::QUIC_PUSH_STREAM_TIMED_OUT)); |
| 940 | quic_data.AddRead(ASYNC, ERR_IO_PENDING); |
| 941 | quic_data.AddRead(ASYNC, OK); // EOF |
| 942 | quic_data.AddSocketDataToFactory(&socket_factory_); |
zhongyi | e34c03566 | 2016-10-19 22:26:02 | [diff] [blame] | 943 | Initialize(); |
| 944 | |
| 945 | ProofVerifyDetailsChromium details; |
| 946 | details.cert_verify_result.verified_cert = |
| 947 | ImportCertFromFile(GetTestCertsDirectory(), "spdy_pooling.pem"); |
| 948 | ASSERT_TRUE(details.cert_verify_result.verified_cert.get()); |
| 949 | |
| 950 | CompleteCryptoHandshake(); |
| 951 | session_->OnProofVerifyDetailsAvailable(details); |
| 952 | |
Ramin Halavati | f7788ea | 2018-02-26 07:02:57 | [diff] [blame] | 953 | QuicChromiumClientStream* stream = |
Fan Yang | 5a3bddf | 2018-10-12 10:05:50 | [diff] [blame] | 954 | QuicChromiumClientSessionPeer::CreateOutgoingStream(session_.get()); |
zhongyi | e34c03566 | 2016-10-19 22:26:02 | [diff] [blame] | 955 | EXPECT_TRUE(stream); |
| 956 | |
Ryan Hamilton | 0239aac | 2018-05-19 00:03:13 | [diff] [blame] | 957 | spdy::SpdyHeaderBlock promise_headers; |
zhongyi | e34c03566 | 2016-10-19 22:26:02 | [diff] [blame] | 958 | promise_headers[":method"] = "GET"; |
| 959 | promise_headers[":authority"] = "www.example.org"; |
| 960 | promise_headers[":scheme"] = "https"; |
| 961 | promise_headers[":path"] = "/pushed.jpg"; |
| 962 | |
| 963 | // Receive a PUSH PROMISE from the server. |
ckrasic | bf2f59c | 2017-05-04 23:54:36 | [diff] [blame] | 964 | EXPECT_TRUE(session_->HandlePromised( |
Fan Yang | 32c5a11 | 2018-12-10 20:06:33 | [diff] [blame] | 965 | stream->id(), GetNthServerInitiatedUnidirectionalStreamId(0), |
| 966 | promise_headers)); |
zhongyi | e34c03566 | 2016-10-19 22:26:02 | [diff] [blame] | 967 | |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 968 | quic::QuicClientPromisedInfo* promised = |
Fan Yang | 32c5a11 | 2018-12-10 20:06:33 | [diff] [blame] | 969 | session_->GetPromisedById(GetNthServerInitiatedUnidirectionalStreamId(0)); |
zhongyi | e34c03566 | 2016-10-19 22:26:02 | [diff] [blame] | 970 | EXPECT_TRUE(promised); |
| 971 | // Fire alarm to time out the push stream. |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 972 | alarm_factory_.FireAlarm( |
| 973 | quic::test::QuicClientPromisedInfoPeer::GetAlarm(promised)); |
zhongyi | e34c03566 | 2016-10-19 22:26:02 | [diff] [blame] | 974 | EXPECT_FALSE( |
| 975 | session_->GetPromisedByUrl("https://www.example.org/pushed.jpg")); |
| 976 | EXPECT_EQ(0u, |
| 977 | QuicChromiumClientSessionPeer::GetPushedBytesCount(session_.get())); |
| 978 | EXPECT_EQ(0u, QuicChromiumClientSessionPeer::GetPushedAndUnclaimedBytesCount( |
| 979 | session_.get())); |
| 980 | } |
| 981 | |
| 982 | TEST_P(QuicChromiumClientSessionTest, PushStreamTimedOutWithResponse) { |
| 983 | base::HistogramTester histogram_tester; |
Renjie Tang | aadb84b | 2019-08-31 01:00:23 | [diff] [blame] | 984 | MockQuicData quic_data(version_); |
| 985 | int packet_num = 1; |
| 986 | if (VersionUsesQpack(version_.transport_version)) { |
| 987 | quic_data.AddWrite(ASYNC, |
| 988 | client_maker_.MakeInitialSettingsPacket(packet_num++)); |
| 989 | } |
| 990 | quic_data.AddWrite(ASYNC, client_maker_.MakeRstPacket( |
| 991 | packet_num++, true, |
| 992 | GetNthServerInitiatedUnidirectionalStreamId(0), |
| 993 | quic::QUIC_PUSH_STREAM_TIMED_OUT)); |
| 994 | quic_data.AddRead(ASYNC, ERR_IO_PENDING); |
| 995 | quic_data.AddRead(ASYNC, OK); // EOF |
| 996 | quic_data.AddSocketDataToFactory(&socket_factory_); |
zhongyi | e34c03566 | 2016-10-19 22:26:02 | [diff] [blame] | 997 | Initialize(); |
| 998 | |
| 999 | ProofVerifyDetailsChromium details; |
| 1000 | details.cert_verify_result.verified_cert = |
| 1001 | ImportCertFromFile(GetTestCertsDirectory(), "spdy_pooling.pem"); |
| 1002 | ASSERT_TRUE(details.cert_verify_result.verified_cert.get()); |
| 1003 | |
| 1004 | CompleteCryptoHandshake(); |
| 1005 | session_->OnProofVerifyDetailsAvailable(details); |
| 1006 | |
Ramin Halavati | f7788ea | 2018-02-26 07:02:57 | [diff] [blame] | 1007 | QuicChromiumClientStream* stream = |
Fan Yang | 5a3bddf | 2018-10-12 10:05:50 | [diff] [blame] | 1008 | QuicChromiumClientSessionPeer::CreateOutgoingStream(session_.get()); |
zhongyi | e34c03566 | 2016-10-19 22:26:02 | [diff] [blame] | 1009 | EXPECT_TRUE(stream); |
| 1010 | |
Ryan Hamilton | 0239aac | 2018-05-19 00:03:13 | [diff] [blame] | 1011 | spdy::SpdyHeaderBlock promise_headers; |
zhongyi | e34c03566 | 2016-10-19 22:26:02 | [diff] [blame] | 1012 | promise_headers[":method"] = "GET"; |
| 1013 | promise_headers[":authority"] = "www.example.org"; |
| 1014 | promise_headers[":scheme"] = "https"; |
| 1015 | promise_headers[":path"] = "/pushed.jpg"; |
| 1016 | |
Fan Yang | 32c5a11 | 2018-12-10 20:06:33 | [diff] [blame] | 1017 | session_->GetOrCreateStream(GetNthServerInitiatedUnidirectionalStreamId(0)); |
zhongyi | e34c03566 | 2016-10-19 22:26:02 | [diff] [blame] | 1018 | // Receive a PUSH PROMISE from the server. |
ckrasic | bf2f59c | 2017-05-04 23:54:36 | [diff] [blame] | 1019 | EXPECT_TRUE(session_->HandlePromised( |
Fan Yang | 32c5a11 | 2018-12-10 20:06:33 | [diff] [blame] | 1020 | stream->id(), GetNthServerInitiatedUnidirectionalStreamId(0), |
| 1021 | promise_headers)); |
| 1022 | session_->OnInitialHeadersComplete( |
| 1023 | GetNthServerInitiatedUnidirectionalStreamId(0), spdy::SpdyHeaderBlock()); |
zhongyi | e34c03566 | 2016-10-19 22:26:02 | [diff] [blame] | 1024 | // Read data on the pushed stream. |
Fan Yang | 32c5a11 | 2018-12-10 20:06:33 | [diff] [blame] | 1025 | quic::QuicStreamFrame data(GetNthServerInitiatedUnidirectionalStreamId(0), |
| 1026 | false, 0, quic::QuicStringPiece("SP")); |
zhongyi | e34c03566 | 2016-10-19 22:26:02 | [diff] [blame] | 1027 | session_->OnStreamFrame(data); |
| 1028 | |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 1029 | quic::QuicClientPromisedInfo* promised = |
Fan Yang | 32c5a11 | 2018-12-10 20:06:33 | [diff] [blame] | 1030 | session_->GetPromisedById(GetNthServerInitiatedUnidirectionalStreamId(0)); |
zhongyi | e34c03566 | 2016-10-19 22:26:02 | [diff] [blame] | 1031 | EXPECT_TRUE(promised); |
| 1032 | // Fire alarm to time out the push stream. |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 1033 | alarm_factory_.FireAlarm( |
| 1034 | quic::test::QuicClientPromisedInfoPeer::GetAlarm(promised)); |
zhongyi | e34c03566 | 2016-10-19 22:26:02 | [diff] [blame] | 1035 | EXPECT_EQ(2u, |
| 1036 | QuicChromiumClientSessionPeer::GetPushedBytesCount(session_.get())); |
| 1037 | EXPECT_EQ(2u, QuicChromiumClientSessionPeer::GetPushedAndUnclaimedBytesCount( |
| 1038 | session_.get())); |
| 1039 | } |
| 1040 | |
Renjie | 7f5894b | 2019-05-31 03:41:31 | [diff] [blame] | 1041 | // Regression test for crbug.com/968621. |
| 1042 | TEST_P(QuicChromiumClientSessionTest, PendingStreamOnRst) { |
| 1043 | if (!quic::VersionHasStreamType(version_.transport_version)) |
| 1044 | return; |
| 1045 | |
Renjie Tang | aadb84b | 2019-08-31 01:00:23 | [diff] [blame] | 1046 | MockQuicData quic_data(version_); |
| 1047 | int packet_num = 1; |
| 1048 | if (VersionUsesQpack(version_.transport_version)) { |
| 1049 | quic_data.AddWrite(ASYNC, |
| 1050 | client_maker_.MakeInitialSettingsPacket(packet_num++)); |
| 1051 | } |
| 1052 | quic_data.AddWrite(ASYNC, client_maker_.MakeRstPacket( |
| 1053 | packet_num++, true, |
| 1054 | GetNthServerInitiatedUnidirectionalStreamId(0), |
| 1055 | quic::QUIC_RST_ACKNOWLEDGEMENT)); |
| 1056 | quic_data.AddRead(ASYNC, ERR_IO_PENDING); |
| 1057 | quic_data.AddRead(ASYNC, OK); // EOF |
| 1058 | quic_data.AddSocketDataToFactory(&socket_factory_); |
Renjie | 7f5894b | 2019-05-31 03:41:31 | [diff] [blame] | 1059 | |
| 1060 | Initialize(); |
| 1061 | CompleteCryptoHandshake(); |
| 1062 | |
| 1063 | quic::QuicStreamFrame data(GetNthServerInitiatedUnidirectionalStreamId(0), |
| 1064 | false, 1, quic::QuicStringPiece("SP")); |
| 1065 | session_->OnStreamFrame(data); |
| 1066 | EXPECT_EQ(0u, session_->GetNumOpenIncomingStreams()); |
| 1067 | quic::QuicRstStreamFrame rst(quic::kInvalidControlFrameId, |
| 1068 | GetNthServerInitiatedUnidirectionalStreamId(0), |
| 1069 | quic::QUIC_STREAM_CANCELLED, 0); |
| 1070 | session_->OnRstStream(rst); |
| 1071 | } |
| 1072 | |
Renjie | 82ddaf4 | 2019-06-07 23:40:31 | [diff] [blame] | 1073 | // Regression test for crbug.com/971361. |
| 1074 | TEST_P(QuicChromiumClientSessionTest, ClosePendingStream) { |
| 1075 | if (!quic::VersionHasStreamType(version_.transport_version)) |
| 1076 | return; |
| 1077 | |
Renjie Tang | aadb84b | 2019-08-31 01:00:23 | [diff] [blame] | 1078 | MockQuicData quic_data(version_); |
| 1079 | int packet_num = 1; |
| 1080 | if (VersionUsesQpack(version_.transport_version)) { |
| 1081 | quic_data.AddWrite(ASYNC, |
| 1082 | client_maker_.MakeInitialSettingsPacket(packet_num++)); |
| 1083 | } |
| 1084 | quic_data.AddWrite(ASYNC, client_maker_.MakeRstPacket( |
| 1085 | packet_num++, true, |
| 1086 | GetNthServerInitiatedUnidirectionalStreamId(0), |
| 1087 | quic::QUIC_RST_ACKNOWLEDGEMENT)); |
| 1088 | quic_data.AddRead(ASYNC, ERR_IO_PENDING); |
| 1089 | quic_data.AddRead(ASYNC, OK); // EOF |
| 1090 | quic_data.AddSocketDataToFactory(&socket_factory_); |
Renjie | 82ddaf4 | 2019-06-07 23:40:31 | [diff] [blame] | 1091 | |
| 1092 | Initialize(); |
| 1093 | CompleteCryptoHandshake(); |
| 1094 | |
| 1095 | quic::QuicStreamId id = GetNthServerInitiatedUnidirectionalStreamId(0); |
| 1096 | quic::QuicStreamFrame data(id, false, 1, quic::QuicStringPiece("SP")); |
| 1097 | session_->OnStreamFrame(data); |
| 1098 | EXPECT_EQ(0u, session_->GetNumOpenIncomingStreams()); |
| 1099 | session_->CloseStream(id); |
| 1100 | } |
| 1101 | |
zhongyi | 7465c1b | 2016-11-11 01:38:05 | [diff] [blame] | 1102 | TEST_P(QuicChromiumClientSessionTest, CancelPushWhenPendingValidation) { |
Renjie Tang | aadb84b | 2019-08-31 01:00:23 | [diff] [blame] | 1103 | MockQuicData quic_data(version_); |
| 1104 | int packet_num = 1; |
| 1105 | if (VersionUsesQpack(version_.transport_version)) { |
| 1106 | quic_data.AddWrite(ASYNC, |
| 1107 | client_maker_.MakeInitialSettingsPacket(packet_num++)); |
| 1108 | } |
| 1109 | quic_data.AddWrite(ASYNC, client_maker_.MakeRstPacket( |
| 1110 | packet_num++, true, |
| 1111 | GetNthClientInitiatedBidirectionalStreamId(0), |
| 1112 | quic::QUIC_RST_ACKNOWLEDGEMENT)); |
| 1113 | quic_data.AddRead(ASYNC, ERR_IO_PENDING); |
| 1114 | quic_data.AddRead(ASYNC, OK); // EOF |
| 1115 | quic_data.AddSocketDataToFactory(&socket_factory_); |
zhongyi | 7465c1b | 2016-11-11 01:38:05 | [diff] [blame] | 1116 | Initialize(); |
| 1117 | |
| 1118 | ProofVerifyDetailsChromium details; |
| 1119 | details.cert_verify_result.verified_cert = |
| 1120 | ImportCertFromFile(GetTestCertsDirectory(), "spdy_pooling.pem"); |
| 1121 | ASSERT_TRUE(details.cert_verify_result.verified_cert.get()); |
| 1122 | |
| 1123 | CompleteCryptoHandshake(); |
| 1124 | session_->OnProofVerifyDetailsAvailable(details); |
| 1125 | |
Ramin Halavati | f7788ea | 2018-02-26 07:02:57 | [diff] [blame] | 1126 | QuicChromiumClientStream* stream = |
Fan Yang | 5a3bddf | 2018-10-12 10:05:50 | [diff] [blame] | 1127 | QuicChromiumClientSessionPeer::CreateOutgoingStream(session_.get()); |
zhongyi | 7465c1b | 2016-11-11 01:38:05 | [diff] [blame] | 1128 | EXPECT_TRUE(stream); |
| 1129 | |
Ryan Hamilton | 0239aac | 2018-05-19 00:03:13 | [diff] [blame] | 1130 | spdy::SpdyHeaderBlock promise_headers; |
zhongyi | 7465c1b | 2016-11-11 01:38:05 | [diff] [blame] | 1131 | promise_headers[":method"] = "GET"; |
| 1132 | promise_headers[":authority"] = "www.example.org"; |
| 1133 | promise_headers[":scheme"] = "https"; |
| 1134 | promise_headers[":path"] = "/pushed.jpg"; |
| 1135 | |
| 1136 | // Receive a PUSH PROMISE from the server. |
ckrasic | bf2f59c | 2017-05-04 23:54:36 | [diff] [blame] | 1137 | EXPECT_TRUE(session_->HandlePromised( |
Fan Yang | 32c5a11 | 2018-12-10 20:06:33 | [diff] [blame] | 1138 | stream->id(), GetNthServerInitiatedUnidirectionalStreamId(0), |
| 1139 | promise_headers)); |
zhongyi | 7465c1b | 2016-11-11 01:38:05 | [diff] [blame] | 1140 | |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 1141 | quic::QuicClientPromisedInfo* promised = |
Fan Yang | 32c5a11 | 2018-12-10 20:06:33 | [diff] [blame] | 1142 | session_->GetPromisedById(GetNthServerInitiatedUnidirectionalStreamId(0)); |
zhongyi | 7465c1b | 2016-11-11 01:38:05 | [diff] [blame] | 1143 | EXPECT_TRUE(promised); |
| 1144 | |
| 1145 | // Initiate rendezvous. |
Ryan Hamilton | 0239aac | 2018-05-19 00:03:13 | [diff] [blame] | 1146 | spdy::SpdyHeaderBlock client_request = promise_headers.Clone(); |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 1147 | quic::test::TestPushPromiseDelegate delegate(/*match=*/true); |
zhongyi | 7465c1b | 2016-11-11 01:38:05 | [diff] [blame] | 1148 | promised->HandleClientRequest(client_request, &delegate); |
| 1149 | |
| 1150 | // Cancel the push before receiving the response to the pushed request. |
| 1151 | GURL pushed_url("https://www.example.org/pushed.jpg"); |
zhongyi | 0009f3e | 2016-11-11 19:47:50 | [diff] [blame] | 1152 | test_push_delegate_.CancelPush(pushed_url); |
zhongyi | 7465c1b | 2016-11-11 01:38:05 | [diff] [blame] | 1153 | EXPECT_TRUE(session_->GetPromisedByUrl(pushed_url.spec())); |
| 1154 | |
| 1155 | // Reset the stream now before tear down. |
Fan Yang | 32c5a11 | 2018-12-10 20:06:33 | [diff] [blame] | 1156 | session_->CloseStream(GetNthClientInitiatedBidirectionalStreamId(0)); |
zhongyi | 7465c1b | 2016-11-11 01:38:05 | [diff] [blame] | 1157 | } |
| 1158 | |
zhongyi | 18bb2d9 | 2016-10-27 19:38:50 | [diff] [blame] | 1159 | TEST_P(QuicChromiumClientSessionTest, CancelPushBeforeReceivingResponse) { |
| 1160 | base::HistogramTester histogram_tester; |
Renjie Tang | aadb84b | 2019-08-31 01:00:23 | [diff] [blame] | 1161 | MockQuicData quic_data(version_); |
| 1162 | int packet_num = 1; |
| 1163 | if (VersionUsesQpack(version_.transport_version)) { |
| 1164 | quic_data.AddWrite(ASYNC, |
| 1165 | client_maker_.MakeInitialSettingsPacket(packet_num++)); |
| 1166 | } |
| 1167 | quic_data.AddWrite(ASYNC, client_maker_.MakeRstPacket( |
| 1168 | packet_num++, true, |
| 1169 | GetNthServerInitiatedUnidirectionalStreamId(0), |
| 1170 | quic::QUIC_STREAM_CANCELLED)); |
| 1171 | quic_data.AddRead(ASYNC, ERR_IO_PENDING); |
| 1172 | quic_data.AddRead(ASYNC, OK); // EOF |
| 1173 | quic_data.AddSocketDataToFactory(&socket_factory_); |
zhongyi | 18bb2d9 | 2016-10-27 19:38:50 | [diff] [blame] | 1174 | Initialize(); |
| 1175 | |
| 1176 | ProofVerifyDetailsChromium details; |
| 1177 | details.cert_verify_result.verified_cert = |
| 1178 | ImportCertFromFile(GetTestCertsDirectory(), "spdy_pooling.pem"); |
| 1179 | ASSERT_TRUE(details.cert_verify_result.verified_cert.get()); |
| 1180 | |
| 1181 | CompleteCryptoHandshake(); |
| 1182 | session_->OnProofVerifyDetailsAvailable(details); |
| 1183 | |
Ramin Halavati | f7788ea | 2018-02-26 07:02:57 | [diff] [blame] | 1184 | QuicChromiumClientStream* stream = |
Fan Yang | 5a3bddf | 2018-10-12 10:05:50 | [diff] [blame] | 1185 | QuicChromiumClientSessionPeer::CreateOutgoingStream(session_.get()); |
zhongyi | 18bb2d9 | 2016-10-27 19:38:50 | [diff] [blame] | 1186 | EXPECT_TRUE(stream); |
| 1187 | |
Ryan Hamilton | 0239aac | 2018-05-19 00:03:13 | [diff] [blame] | 1188 | spdy::SpdyHeaderBlock promise_headers; |
zhongyi | 18bb2d9 | 2016-10-27 19:38:50 | [diff] [blame] | 1189 | promise_headers[":method"] = "GET"; |
| 1190 | promise_headers[":authority"] = "www.example.org"; |
| 1191 | promise_headers[":scheme"] = "https"; |
| 1192 | promise_headers[":path"] = "/pushed.jpg"; |
| 1193 | |
| 1194 | // Receive a PUSH PROMISE from the server. |
ckrasic | bf2f59c | 2017-05-04 23:54:36 | [diff] [blame] | 1195 | EXPECT_TRUE(session_->HandlePromised( |
Fan Yang | 32c5a11 | 2018-12-10 20:06:33 | [diff] [blame] | 1196 | stream->id(), GetNthServerInitiatedUnidirectionalStreamId(0), |
| 1197 | promise_headers)); |
zhongyi | 18bb2d9 | 2016-10-27 19:38:50 | [diff] [blame] | 1198 | |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 1199 | quic::QuicClientPromisedInfo* promised = |
Fan Yang | 32c5a11 | 2018-12-10 20:06:33 | [diff] [blame] | 1200 | session_->GetPromisedById(GetNthServerInitiatedUnidirectionalStreamId(0)); |
zhongyi | 18bb2d9 | 2016-10-27 19:38:50 | [diff] [blame] | 1201 | EXPECT_TRUE(promised); |
| 1202 | // Cancel the push before receiving the response to the pushed request. |
| 1203 | GURL pushed_url("https://www.example.org/pushed.jpg"); |
zhongyi | 0009f3e | 2016-11-11 19:47:50 | [diff] [blame] | 1204 | test_push_delegate_.CancelPush(pushed_url); |
zhongyi | 18bb2d9 | 2016-10-27 19:38:50 | [diff] [blame] | 1205 | |
| 1206 | EXPECT_FALSE(session_->GetPromisedByUrl(pushed_url.spec())); |
| 1207 | EXPECT_EQ(0u, |
| 1208 | QuicChromiumClientSessionPeer::GetPushedBytesCount(session_.get())); |
| 1209 | EXPECT_EQ(0u, QuicChromiumClientSessionPeer::GetPushedAndUnclaimedBytesCount( |
| 1210 | session_.get())); |
| 1211 | } |
| 1212 | |
| 1213 | TEST_P(QuicChromiumClientSessionTest, CancelPushAfterReceivingResponse) { |
| 1214 | base::HistogramTester histogram_tester; |
Renjie Tang | aadb84b | 2019-08-31 01:00:23 | [diff] [blame] | 1215 | MockQuicData quic_data(version_); |
| 1216 | int packet_num = 1; |
| 1217 | if (VersionUsesQpack(version_.transport_version)) { |
| 1218 | quic_data.AddWrite(ASYNC, |
| 1219 | client_maker_.MakeInitialSettingsPacket(packet_num++)); |
| 1220 | } |
| 1221 | quic_data.AddWrite(ASYNC, client_maker_.MakeRstPacket( |
| 1222 | packet_num++, true, |
| 1223 | GetNthServerInitiatedUnidirectionalStreamId(0), |
| 1224 | quic::QUIC_STREAM_CANCELLED)); |
| 1225 | quic_data.AddRead(ASYNC, ERR_IO_PENDING); |
| 1226 | quic_data.AddRead(ASYNC, OK); // EOF |
| 1227 | quic_data.AddSocketDataToFactory(&socket_factory_); |
| 1228 | |
zhongyi | 18bb2d9 | 2016-10-27 19:38:50 | [diff] [blame] | 1229 | Initialize(); |
| 1230 | |
| 1231 | ProofVerifyDetailsChromium details; |
| 1232 | details.cert_verify_result.verified_cert = |
| 1233 | ImportCertFromFile(GetTestCertsDirectory(), "spdy_pooling.pem"); |
| 1234 | ASSERT_TRUE(details.cert_verify_result.verified_cert.get()); |
| 1235 | |
| 1236 | CompleteCryptoHandshake(); |
| 1237 | session_->OnProofVerifyDetailsAvailable(details); |
| 1238 | |
Ramin Halavati | f7788ea | 2018-02-26 07:02:57 | [diff] [blame] | 1239 | QuicChromiumClientStream* stream = |
Fan Yang | 5a3bddf | 2018-10-12 10:05:50 | [diff] [blame] | 1240 | QuicChromiumClientSessionPeer::CreateOutgoingStream(session_.get()); |
zhongyi | 18bb2d9 | 2016-10-27 19:38:50 | [diff] [blame] | 1241 | EXPECT_TRUE(stream); |
| 1242 | |
Ryan Hamilton | 0239aac | 2018-05-19 00:03:13 | [diff] [blame] | 1243 | spdy::SpdyHeaderBlock promise_headers; |
zhongyi | 18bb2d9 | 2016-10-27 19:38:50 | [diff] [blame] | 1244 | promise_headers[":method"] = "GET"; |
| 1245 | promise_headers[":authority"] = "www.example.org"; |
| 1246 | promise_headers[":scheme"] = "https"; |
| 1247 | promise_headers[":path"] = "/pushed.jpg"; |
| 1248 | |
Fan Yang | 32c5a11 | 2018-12-10 20:06:33 | [diff] [blame] | 1249 | session_->GetOrCreateStream(GetNthServerInitiatedUnidirectionalStreamId(0)); |
zhongyi | 18bb2d9 | 2016-10-27 19:38:50 | [diff] [blame] | 1250 | // Receive a PUSH PROMISE from the server. |
ckrasic | bf2f59c | 2017-05-04 23:54:36 | [diff] [blame] | 1251 | EXPECT_TRUE(session_->HandlePromised( |
Fan Yang | 32c5a11 | 2018-12-10 20:06:33 | [diff] [blame] | 1252 | stream->id(), GetNthServerInitiatedUnidirectionalStreamId(0), |
| 1253 | promise_headers)); |
| 1254 | session_->OnInitialHeadersComplete( |
| 1255 | GetNthServerInitiatedUnidirectionalStreamId(0), spdy::SpdyHeaderBlock()); |
zhongyi | 18bb2d9 | 2016-10-27 19:38:50 | [diff] [blame] | 1256 | // Read data on the pushed stream. |
Fan Yang | 32c5a11 | 2018-12-10 20:06:33 | [diff] [blame] | 1257 | quic::QuicStreamFrame data(GetNthServerInitiatedUnidirectionalStreamId(0), |
| 1258 | false, 0, quic::QuicStringPiece("SP")); |
zhongyi | 18bb2d9 | 2016-10-27 19:38:50 | [diff] [blame] | 1259 | session_->OnStreamFrame(data); |
| 1260 | |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 1261 | quic::QuicClientPromisedInfo* promised = |
Fan Yang | 32c5a11 | 2018-12-10 20:06:33 | [diff] [blame] | 1262 | session_->GetPromisedById(GetNthServerInitiatedUnidirectionalStreamId(0)); |
zhongyi | 18bb2d9 | 2016-10-27 19:38:50 | [diff] [blame] | 1263 | EXPECT_TRUE(promised); |
| 1264 | // Cancel the push after receiving data on the push stream. |
| 1265 | GURL pushed_url("https://www.example.org/pushed.jpg"); |
zhongyi | 0009f3e | 2016-11-11 19:47:50 | [diff] [blame] | 1266 | test_push_delegate_.CancelPush(pushed_url); |
zhongyi | 18bb2d9 | 2016-10-27 19:38:50 | [diff] [blame] | 1267 | |
| 1268 | EXPECT_FALSE(session_->GetPromisedByUrl(pushed_url.spec())); |
| 1269 | EXPECT_EQ(2u, |
| 1270 | QuicChromiumClientSessionPeer::GetPushedBytesCount(session_.get())); |
| 1271 | EXPECT_EQ(2u, QuicChromiumClientSessionPeer::GetPushedAndUnclaimedBytesCount( |
| 1272 | session_.get())); |
| 1273 | } |
| 1274 | |
ckrasic | 4f9d88d | 2015-07-22 22:23:16 | [diff] [blame] | 1275 | TEST_P(QuicChromiumClientSessionTest, MaxNumStreamsViaRequest) { |
Ryan Hamilton | abad59e | 2019-06-06 04:02:59 | [diff] [blame] | 1276 | MockQuicData quic_data(version_); |
Nick Harper | 23290b8 | 2019-05-02 00:02:56 | [diff] [blame] | 1277 | if (version_.transport_version == quic::QUIC_VERSION_99) { |
Renjie Tang | aadb84b | 2019-08-31 01:00:23 | [diff] [blame] | 1278 | quic_data.AddWrite(SYNCHRONOUS, client_maker_.MakeInitialSettingsPacket(1)); |
David Schinazi | cc1bc59 | 2019-04-24 19:40:31 | [diff] [blame] | 1279 | quic_data.AddWrite(SYNCHRONOUS, client_maker_.MakeStreamsBlockedPacket( |
Fan Yang | 2330d18 | 2019-08-05 14:50:50 | [diff] [blame] | 1280 | 2, true, 50, |
David Schinazi | cc1bc59 | 2019-04-24 19:40:31 | [diff] [blame] | 1281 | /*unidirectional=*/false)); |
Fan Yang | 32c5a11 | 2018-12-10 20:06:33 | [diff] [blame] | 1282 | quic_data.AddWrite( |
| 1283 | SYNCHRONOUS, client_maker_.MakeRstPacket( |
| 1284 | 3, true, GetNthClientInitiatedBidirectionalStreamId(0), |
| 1285 | quic::QUIC_RST_ACKNOWLEDGEMENT)); |
| 1286 | quic_data.AddRead( |
Fan Yang | 2330d18 | 2019-08-05 14:50:50 | [diff] [blame] | 1287 | ASYNC, server_maker_.MakeMaxStreamsPacket(1, true, 52, |
David Schinazi | cc1bc59 | 2019-04-24 19:40:31 | [diff] [blame] | 1288 | /*unidirectional=*/false)); |
Frank Kastenholz | c9b9bea | 2018-12-03 20:13:47 | [diff] [blame] | 1289 | } else { |
| 1290 | quic_data.AddWrite( |
Fan Yang | 32c5a11 | 2018-12-10 20:06:33 | [diff] [blame] | 1291 | SYNCHRONOUS, client_maker_.MakeRstPacket( |
Renjie Tang | aadb84b | 2019-08-31 01:00:23 | [diff] [blame] | 1292 | 1, true, GetNthClientInitiatedBidirectionalStreamId(0), |
Fan Yang | 32c5a11 | 2018-12-10 20:06:33 | [diff] [blame] | 1293 | quic::QUIC_RST_ACKNOWLEDGEMENT)); |
Frank Kastenholz | 878763bf | 2018-11-28 19:14:48 | [diff] [blame] | 1294 | } |
| 1295 | quic_data.AddRead(ASYNC, ERR_IO_PENDING); |
| 1296 | quic_data.AddRead(ASYNC, OK); // EOF |
| 1297 | quic_data.AddSocketDataToFactory(&socket_factory_); |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 1298 | |
| 1299 | Initialize(); |
[email protected] | 0b2294d3 | 2013-08-02 00:46:36 | [diff] [blame] | 1300 | CompleteCryptoHandshake(); |
Fan Yang | 32c5a11 | 2018-12-10 20:06:33 | [diff] [blame] | 1301 | const size_t kMaxOpenStreams = GetMaxAllowedOutgoingBidirectionalStreams(); |
rch | 12fef55 | 2016-01-15 16:26:31 | [diff] [blame] | 1302 | std::vector<QuicChromiumClientStream*> streams; |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 1303 | for (size_t i = 0; i < kMaxOpenStreams; i++) { |
Ramin Halavati | f7788ea | 2018-02-26 07:02:57 | [diff] [blame] | 1304 | QuicChromiumClientStream* stream = |
Fan Yang | 5a3bddf | 2018-10-12 10:05:50 | [diff] [blame] | 1305 | QuicChromiumClientSessionPeer::CreateOutgoingStream(session_.get()); |
[email protected] | 0b2294d3 | 2013-08-02 00:46:36 | [diff] [blame] | 1306 | EXPECT_TRUE(stream); |
| 1307 | streams.push_back(stream); |
| 1308 | } |
| 1309 | |
rch | f0b18c8a | 2017-05-05 19:31:57 | [diff] [blame] | 1310 | std::unique_ptr<QuicChromiumClientSession::Handle> handle = |
Ryan Hamilton | 6c2a2a8 | 2017-12-15 02:06:28 | [diff] [blame] | 1311 | session_->CreateHandle(destination_); |
[email protected] | 0b2294d3 | 2013-08-02 00:46:36 | [diff] [blame] | 1312 | TestCompletionCallback callback; |
Ramin Halavati | 683bcaa9 | 2018-02-14 08:42:39 | [diff] [blame] | 1313 | ASSERT_EQ( |
| 1314 | ERR_IO_PENDING, |
| 1315 | handle->RequestStream(/*requires_confirmation=*/false, |
| 1316 | callback.callback(), TRAFFIC_ANNOTATION_FOR_TESTS)); |
[email protected] | 0b2294d3 | 2013-08-02 00:46:36 | [diff] [blame] | 1317 | |
| 1318 | // Close a stream and ensure I can now open a new one. |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 1319 | quic::QuicStreamId stream_id = streams[0]->id(); |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 1320 | session_->CloseStream(stream_id); |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 1321 | quic::QuicRstStreamFrame rst1(quic::kInvalidControlFrameId, stream_id, |
| 1322 | quic::QUIC_STREAM_NO_ERROR, 0); |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 1323 | session_->OnRstStream(rst1); |
Frank Kastenholz | 878763bf | 2018-11-28 19:14:48 | [diff] [blame] | 1324 | // Pump data, bringing in the max-stream-id |
| 1325 | base::RunLoop().RunUntilIdle(); |
[email protected] | 0b2294d3 | 2013-08-02 00:46:36 | [diff] [blame] | 1326 | ASSERT_TRUE(callback.have_result()); |
robpercival | 214763f | 2016-07-01 23:27:01 | [diff] [blame] | 1327 | EXPECT_THAT(callback.WaitForResult(), IsOk()); |
rch | 1bcfddf2 | 2017-06-03 00:26:29 | [diff] [blame] | 1328 | EXPECT_TRUE(handle->ReleaseStream() != nullptr); |
[email protected] | 0b2294d3 | 2013-08-02 00:46:36 | [diff] [blame] | 1329 | } |
| 1330 | |
ckrasic | 4f9d88d | 2015-07-22 22:23:16 | [diff] [blame] | 1331 | TEST_P(QuicChromiumClientSessionTest, GoAwayReceived) { |
Renjie Tang | aadb84b | 2019-08-31 01:00:23 | [diff] [blame] | 1332 | MockQuicData quic_data(version_); |
| 1333 | if (VersionUsesQpack(version_.transport_version)) |
| 1334 | quic_data.AddWrite(SYNCHRONOUS, client_maker_.MakeInitialSettingsPacket(1)); |
| 1335 | quic_data.AddRead(ASYNC, ERR_IO_PENDING); |
| 1336 | quic_data.AddRead(ASYNC, OK); // EOF |
| 1337 | quic_data.AddSocketDataToFactory(&socket_factory_); |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 1338 | Initialize(); |
[email protected] | 8ba8121 | 2013-05-03 13:11:48 | [diff] [blame] | 1339 | CompleteCryptoHandshake(); |
[email protected] | 9db44391 | 2013-02-25 05:27:03 | [diff] [blame] | 1340 | |
| 1341 | // After receiving a GoAway, I should no longer be able to create outgoing |
| 1342 | // streams. |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 1343 | session_->connection()->OnGoAwayFrame( |
| 1344 | quic::QuicGoAwayFrame(quic::kInvalidControlFrameId, |
| 1345 | quic::QUIC_PEER_GOING_AWAY, 1u, "Going away.")); |
Fan Yang | 5a3bddf | 2018-10-12 10:05:50 | [diff] [blame] | 1346 | EXPECT_EQ(nullptr, QuicChromiumClientSessionPeer::CreateOutgoingStream( |
Ramin Halavati | f7788ea | 2018-02-26 07:02:57 | [diff] [blame] | 1347 | session_.get())); |
[email protected] | 9db44391 | 2013-02-25 05:27:03 | [diff] [blame] | 1348 | } |
| 1349 | |
ckrasic | 4f9d88d | 2015-07-22 22:23:16 | [diff] [blame] | 1350 | TEST_P(QuicChromiumClientSessionTest, CanPool) { |
Renjie Tang | aadb84b | 2019-08-31 01:00:23 | [diff] [blame] | 1351 | MockQuicData quic_data(version_); |
| 1352 | if (VersionUsesQpack(version_.transport_version)) |
| 1353 | quic_data.AddWrite(SYNCHRONOUS, client_maker_.MakeInitialSettingsPacket(1)); |
| 1354 | quic_data.AddRead(ASYNC, ERR_IO_PENDING); |
| 1355 | quic_data.AddRead(ASYNC, OK); // EOF |
| 1356 | quic_data.AddSocketDataToFactory(&socket_factory_); |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 1357 | Initialize(); |
[email protected] | f21ec37 | 2014-07-02 07:15:12 | [diff] [blame] | 1358 | // Load a cert that is valid for: |
| 1359 | // www.example.org |
| 1360 | // mail.example.org |
| 1361 | // www.example.com |
[email protected] | f21ec37 | 2014-07-02 07:15:12 | [diff] [blame] | 1362 | |
[email protected] | f21ec37 | 2014-07-02 07:15:12 | [diff] [blame] | 1363 | ProofVerifyDetailsChromium details; |
| 1364 | details.cert_verify_result.verified_cert = |
[email protected] | 5db45220 | 2014-08-19 05:22:15 | [diff] [blame] | 1365 | ImportCertFromFile(GetTestCertsDirectory(), "spdy_pooling.pem"); |
dcheng | 4227c6d | 2014-08-25 23:58:18 | [diff] [blame] | 1366 | ASSERT_TRUE(details.cert_verify_result.verified_cert.get()); |
[email protected] | f21ec37 | 2014-07-02 07:15:12 | [diff] [blame] | 1367 | |
[email protected] | f21ec37 | 2014-07-02 07:15:12 | [diff] [blame] | 1368 | CompleteCryptoHandshake(); |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 1369 | session_->OnProofVerifyDetailsAvailable(details); |
[email protected] | f21ec37 | 2014-07-02 07:15:12 | [diff] [blame] | 1370 | |
Matt Menke | 26e4154 | 2019-06-05 01:09:51 | [diff] [blame] | 1371 | EXPECT_TRUE(session_->CanPool("www.example.org", PRIVACY_MODE_DISABLED, |
| 1372 | SocketTag(), NetworkIsolationKey())); |
| 1373 | EXPECT_FALSE(session_->CanPool("www.example.org", PRIVACY_MODE_ENABLED, |
| 1374 | SocketTag(), NetworkIsolationKey())); |
Paul Jensen | 8e3c5d3 | 2018-02-19 17:06:33 | [diff] [blame] | 1375 | #if defined(OS_ANDROID) |
| 1376 | SocketTag tag1(SocketTag::UNSET_UID, 0x12345678); |
| 1377 | SocketTag tag2(getuid(), 0x87654321); |
Matt Menke | 26e4154 | 2019-06-05 01:09:51 | [diff] [blame] | 1378 | EXPECT_FALSE(session_->CanPool("www.example.org", PRIVACY_MODE_DISABLED, tag1, |
| 1379 | NetworkIsolationKey())); |
| 1380 | EXPECT_FALSE(session_->CanPool("www.example.org", PRIVACY_MODE_DISABLED, tag2, |
| 1381 | NetworkIsolationKey())); |
Paul Jensen | 8e3c5d3 | 2018-02-19 17:06:33 | [diff] [blame] | 1382 | #endif |
| 1383 | EXPECT_TRUE(session_->CanPool("mail.example.org", PRIVACY_MODE_DISABLED, |
Matt Menke | 26e4154 | 2019-06-05 01:09:51 | [diff] [blame] | 1384 | SocketTag(), NetworkIsolationKey())); |
Paul Jensen | 8e3c5d3 | 2018-02-19 17:06:33 | [diff] [blame] | 1385 | EXPECT_TRUE(session_->CanPool("mail.example.com", PRIVACY_MODE_DISABLED, |
Matt Menke | 26e4154 | 2019-06-05 01:09:51 | [diff] [blame] | 1386 | SocketTag(), NetworkIsolationKey())); |
| 1387 | EXPECT_FALSE(session_->CanPool("mail.google.com", PRIVACY_MODE_DISABLED, |
| 1388 | SocketTag(), NetworkIsolationKey())); |
| 1389 | |
Shivani Sharma | 8ae506c | 2019-07-21 21:08:27 | [diff] [blame] | 1390 | const auto kOriginFoo = url::Origin::Create(GURL("http://foo.test/")); |
| 1391 | |
Matt Menke | 26e4154 | 2019-06-05 01:09:51 | [diff] [blame] | 1392 | // Check that NetworkIsolationKey is respected when feature is enabled. |
| 1393 | { |
| 1394 | base::test::ScopedFeatureList feature_list; |
| 1395 | feature_list.InitAndDisableFeature( |
| 1396 | features::kPartitionConnectionsByNetworkIsolationKey); |
Shivani Sharma | 8ae506c | 2019-07-21 21:08:27 | [diff] [blame] | 1397 | EXPECT_TRUE(session_->CanPool("mail.example.com", PRIVACY_MODE_DISABLED, |
| 1398 | SocketTag(), |
| 1399 | NetworkIsolationKey(kOriginFoo, kOriginFoo))); |
Matt Menke | 26e4154 | 2019-06-05 01:09:51 | [diff] [blame] | 1400 | } |
| 1401 | { |
| 1402 | base::test::ScopedFeatureList feature_list; |
| 1403 | feature_list.InitAndEnableFeature( |
| 1404 | features::kPartitionConnectionsByNetworkIsolationKey); |
| 1405 | EXPECT_FALSE(session_->CanPool( |
| 1406 | "mail.example.com", PRIVACY_MODE_DISABLED, SocketTag(), |
Shivani Sharma | 8ae506c | 2019-07-21 21:08:27 | [diff] [blame] | 1407 | NetworkIsolationKey(kOriginFoo, kOriginFoo))); |
Matt Menke | 26e4154 | 2019-06-05 01:09:51 | [diff] [blame] | 1408 | } |
| 1409 | } |
| 1410 | |
| 1411 | // Much as above, but uses a non-empty NetworkIsolationKey. |
| 1412 | TEST_P(QuicChromiumClientSessionTest, CanPoolWithNetworkIsolationKey) { |
| 1413 | base::test::ScopedFeatureList feature_list; |
| 1414 | feature_list.InitAndEnableFeature( |
| 1415 | features::kPartitionConnectionsByNetworkIsolationKey); |
| 1416 | |
Shivani Sharma | 8ae506c | 2019-07-21 21:08:27 | [diff] [blame] | 1417 | const auto kOriginFoo = url::Origin::Create(GURL("http://foo.test/")); |
| 1418 | const auto kOriginBar = url::Origin::Create(GURL("http://bar.test/")); |
| 1419 | const NetworkIsolationKey kNetworkIsolationKey1(kOriginFoo, kOriginFoo); |
| 1420 | const NetworkIsolationKey kNetworkIsolationKey2(kOriginBar, kOriginBar); |
Matt Menke | 26e4154 | 2019-06-05 01:09:51 | [diff] [blame] | 1421 | |
| 1422 | session_key_ = |
| 1423 | QuicSessionKey(kServerHostname, kServerPort, PRIVACY_MODE_DISABLED, |
| 1424 | SocketTag(), kNetworkIsolationKey1); |
| 1425 | |
Renjie Tang | aadb84b | 2019-08-31 01:00:23 | [diff] [blame] | 1426 | MockQuicData quic_data(version_); |
| 1427 | if (VersionUsesQpack(version_.transport_version)) |
| 1428 | quic_data.AddWrite(SYNCHRONOUS, client_maker_.MakeInitialSettingsPacket(1)); |
| 1429 | quic_data.AddRead(ASYNC, ERR_IO_PENDING); |
| 1430 | quic_data.AddRead(ASYNC, OK); // EOF |
| 1431 | quic_data.AddSocketDataToFactory(&socket_factory_); |
Matt Menke | 26e4154 | 2019-06-05 01:09:51 | [diff] [blame] | 1432 | Initialize(); |
| 1433 | // Load a cert that is valid for: |
| 1434 | // www.example.org |
| 1435 | // mail.example.org |
| 1436 | // www.example.com |
| 1437 | |
| 1438 | ProofVerifyDetailsChromium details; |
| 1439 | details.cert_verify_result.verified_cert = |
| 1440 | ImportCertFromFile(GetTestCertsDirectory(), "spdy_pooling.pem"); |
| 1441 | ASSERT_TRUE(details.cert_verify_result.verified_cert.get()); |
| 1442 | |
| 1443 | CompleteCryptoHandshake(); |
| 1444 | session_->OnProofVerifyDetailsAvailable(details); |
| 1445 | |
| 1446 | EXPECT_TRUE(session_->CanPool("www.example.org", PRIVACY_MODE_DISABLED, |
| 1447 | SocketTag(), kNetworkIsolationKey1)); |
| 1448 | EXPECT_FALSE(session_->CanPool("www.example.org", PRIVACY_MODE_ENABLED, |
| 1449 | SocketTag(), kNetworkIsolationKey1)); |
| 1450 | #if defined(OS_ANDROID) |
| 1451 | SocketTag tag1(SocketTag::UNSET_UID, 0x12345678); |
| 1452 | SocketTag tag2(getuid(), 0x87654321); |
| 1453 | EXPECT_FALSE(session_->CanPool("www.example.org", PRIVACY_MODE_DISABLED, tag1, |
| 1454 | kNetworkIsolationKey1)); |
| 1455 | EXPECT_FALSE(session_->CanPool("www.example.org", PRIVACY_MODE_DISABLED, tag2, |
| 1456 | kNetworkIsolationKey1)); |
| 1457 | #endif |
| 1458 | EXPECT_TRUE(session_->CanPool("mail.example.org", PRIVACY_MODE_DISABLED, |
| 1459 | SocketTag(), kNetworkIsolationKey1)); |
| 1460 | EXPECT_TRUE(session_->CanPool("mail.example.com", PRIVACY_MODE_DISABLED, |
| 1461 | SocketTag(), kNetworkIsolationKey1)); |
| 1462 | EXPECT_FALSE(session_->CanPool("mail.google.com", PRIVACY_MODE_DISABLED, |
| 1463 | SocketTag(), kNetworkIsolationKey1)); |
| 1464 | |
| 1465 | EXPECT_FALSE(session_->CanPool("mail.example.com", PRIVACY_MODE_DISABLED, |
| 1466 | SocketTag(), kNetworkIsolationKey2)); |
| 1467 | EXPECT_FALSE(session_->CanPool("mail.example.com", PRIVACY_MODE_DISABLED, |
| 1468 | SocketTag(), NetworkIsolationKey())); |
[email protected] | f21ec37 | 2014-07-02 07:15:12 | [diff] [blame] | 1469 | } |
| 1470 | |
ckrasic | 4f9d88d | 2015-07-22 22:23:16 | [diff] [blame] | 1471 | TEST_P(QuicChromiumClientSessionTest, ConnectionNotPooledWithDifferentPin) { |
Matt Mueller | 230996f1 | 2018-10-22 19:39:44 | [diff] [blame] | 1472 | // Configure the TransportSecurityStateSource so that kPreloadedPKPHost will |
| 1473 | // have static PKP pins set. |
| 1474 | ScopedTransportSecurityStateSource scoped_security_state_source; |
| 1475 | |
| 1476 | // |net::test_default::kHSTSSource| defines pins for kPreloadedPKPHost. |
| 1477 | // (This hostname must be in the spdy_pooling.pem SAN.) |
| 1478 | const char kPreloadedPKPHost[] = "www.example.org"; |
| 1479 | // A hostname without any static state. (This hostname isn't in |
| 1480 | // spdy_pooling.pem SAN, but that's okay because the |
| 1481 | // ProofVerifyDetailsChromium are faked.) |
| 1482 | const char kNoPinsHost[] = "no-pkp.example.org"; |
| 1483 | |
Renjie Tang | aadb84b | 2019-08-31 01:00:23 | [diff] [blame] | 1484 | MockQuicData quic_data(version_); |
| 1485 | if (VersionUsesQpack(version_.transport_version)) |
| 1486 | quic_data.AddWrite(SYNCHRONOUS, client_maker_.MakeInitialSettingsPacket(1)); |
| 1487 | quic_data.AddRead(ASYNC, ERR_IO_PENDING); |
| 1488 | quic_data.AddRead(ASYNC, OK); // EOF |
| 1489 | quic_data.AddSocketDataToFactory(&socket_factory_); |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 1490 | Initialize(); |
| 1491 | |
Matt Mueller | 230996f1 | 2018-10-22 19:39:44 | [diff] [blame] | 1492 | transport_security_state_.EnableStaticPinsForTesting(); |
[email protected] | 5db45220 | 2014-08-19 05:22:15 | [diff] [blame] | 1493 | |
| 1494 | ProofVerifyDetailsChromium details; |
| 1495 | details.cert_verify_result.verified_cert = |
| 1496 | ImportCertFromFile(GetTestCertsDirectory(), "spdy_pooling.pem"); |
| 1497 | details.cert_verify_result.is_issued_by_known_root = true; |
Matt Mueller | 230996f1 | 2018-10-22 19:39:44 | [diff] [blame] | 1498 | uint8_t bad_pin = 3; |
[email protected] | 5db45220 | 2014-08-19 05:22:15 | [diff] [blame] | 1499 | details.cert_verify_result.public_key_hashes.push_back( |
| 1500 | GetTestHashValue(bad_pin)); |
| 1501 | |
dcheng | 4227c6d | 2014-08-25 23:58:18 | [diff] [blame] | 1502 | ASSERT_TRUE(details.cert_verify_result.verified_cert.get()); |
[email protected] | 5db45220 | 2014-08-19 05:22:15 | [diff] [blame] | 1503 | |
[email protected] | 5db45220 | 2014-08-19 05:22:15 | [diff] [blame] | 1504 | CompleteCryptoHandshake(); |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 1505 | session_->OnProofVerifyDetailsAvailable(details); |
Matt Mueller | 230996f1 | 2018-10-22 19:39:44 | [diff] [blame] | 1506 | QuicChromiumClientSessionPeer::SetHostname(session_.get(), kNoPinsHost); |
[email protected] | 5db45220 | 2014-08-19 05:22:15 | [diff] [blame] | 1507 | |
Matt Menke | 26e4154 | 2019-06-05 01:09:51 | [diff] [blame] | 1508 | EXPECT_FALSE(session_->CanPool(kPreloadedPKPHost, PRIVACY_MODE_DISABLED, |
| 1509 | SocketTag(), NetworkIsolationKey())); |
[email protected] | 5db45220 | 2014-08-19 05:22:15 | [diff] [blame] | 1510 | } |
| 1511 | |
ckrasic | 4f9d88d | 2015-07-22 22:23:16 | [diff] [blame] | 1512 | TEST_P(QuicChromiumClientSessionTest, ConnectionPooledWithMatchingPin) { |
Matt Mueller | 230996f1 | 2018-10-22 19:39:44 | [diff] [blame] | 1513 | ScopedTransportSecurityStateSource scoped_security_state_source; |
| 1514 | |
Renjie Tang | aadb84b | 2019-08-31 01:00:23 | [diff] [blame] | 1515 | MockQuicData quic_data(version_); |
| 1516 | if (VersionUsesQpack(version_.transport_version)) |
| 1517 | quic_data.AddWrite(SYNCHRONOUS, client_maker_.MakeInitialSettingsPacket(1)); |
| 1518 | quic_data.AddRead(ASYNC, ERR_IO_PENDING); |
| 1519 | quic_data.AddRead(ASYNC, OK); // EOF |
| 1520 | quic_data.AddSocketDataToFactory(&socket_factory_); |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 1521 | Initialize(); |
| 1522 | |
Matt Mueller | 230996f1 | 2018-10-22 19:39:44 | [diff] [blame] | 1523 | transport_security_state_.EnableStaticPinsForTesting(); |
[email protected] | 5db45220 | 2014-08-19 05:22:15 | [diff] [blame] | 1524 | |
| 1525 | ProofVerifyDetailsChromium details; |
| 1526 | details.cert_verify_result.verified_cert = |
| 1527 | ImportCertFromFile(GetTestCertsDirectory(), "spdy_pooling.pem"); |
| 1528 | details.cert_verify_result.is_issued_by_known_root = true; |
Matt Mueller | 230996f1 | 2018-10-22 19:39:44 | [diff] [blame] | 1529 | HashValue primary_pin(HASH_VALUE_SHA256); |
| 1530 | EXPECT_TRUE(primary_pin.FromString( |
| 1531 | "sha256/Nn8jk5By4Vkq6BeOVZ7R7AC6XUUBZsWmUbJR1f1Y5FY=")); |
| 1532 | details.cert_verify_result.public_key_hashes.push_back(primary_pin); |
[email protected] | 5db45220 | 2014-08-19 05:22:15 | [diff] [blame] | 1533 | |
dcheng | 4227c6d | 2014-08-25 23:58:18 | [diff] [blame] | 1534 | ASSERT_TRUE(details.cert_verify_result.verified_cert.get()); |
[email protected] | 5db45220 | 2014-08-19 05:22:15 | [diff] [blame] | 1535 | |
[email protected] | 5db45220 | 2014-08-19 05:22:15 | [diff] [blame] | 1536 | CompleteCryptoHandshake(); |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 1537 | session_->OnProofVerifyDetailsAvailable(details); |
| 1538 | QuicChromiumClientSessionPeer::SetHostname(session_.get(), "www.example.org"); |
[email protected] | 5db45220 | 2014-08-19 05:22:15 | [diff] [blame] | 1539 | |
Paul Jensen | 8e3c5d3 | 2018-02-19 17:06:33 | [diff] [blame] | 1540 | EXPECT_TRUE(session_->CanPool("mail.example.org", PRIVACY_MODE_DISABLED, |
Matt Menke | 26e4154 | 2019-06-05 01:09:51 | [diff] [blame] | 1541 | SocketTag(), NetworkIsolationKey())); |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 1542 | } |
| 1543 | |
| 1544 | TEST_P(QuicChromiumClientSessionTest, MigrateToSocket) { |
Renjie Tang | aadb84b | 2019-08-31 01:00:23 | [diff] [blame] | 1545 | MockQuicData quic_data(version_); |
| 1546 | int packet_num = 1; |
| 1547 | socket_data_.reset(); |
| 1548 | if (VersionUsesQpack(version_.transport_version)) { |
| 1549 | quic_data.AddWrite(SYNCHRONOUS, |
| 1550 | client_maker_.MakeInitialSettingsPacket(packet_num++)); |
| 1551 | } |
| 1552 | quic_data.AddRead(ASYNC, ERR_IO_PENDING); |
| 1553 | quic_data.AddRead(ASYNC, OK); // EOF |
| 1554 | quic_data.AddSocketDataToFactory(&socket_factory_); |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 1555 | Initialize(); |
| 1556 | CompleteCryptoHandshake(); |
| 1557 | |
| 1558 | char data[] = "ABCD"; |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 1559 | std::unique_ptr<quic::QuicEncryptedPacket> client_ping; |
| 1560 | std::unique_ptr<quic::QuicEncryptedPacket> ack_and_data_out; |
Renjie Tang | aadb84b | 2019-08-31 01:00:23 | [diff] [blame] | 1561 | if (VersionUsesQpack(version_.transport_version)) { |
| 1562 | client_ping = |
| 1563 | client_maker_.MakeAckAndPingPacket(packet_num++, false, 1, 1, 1); |
| 1564 | } else { |
| 1565 | client_ping = client_maker_.MakePingPacket(packet_num++, true); |
| 1566 | } |
Fan Yang | 32c5a11 | 2018-12-10 20:06:33 | [diff] [blame] | 1567 | ack_and_data_out = client_maker_.MakeDataPacket( |
Renjie Tang | aadb84b | 2019-08-31 01:00:23 | [diff] [blame] | 1568 | packet_num++, GetNthClientInitiatedBidirectionalStreamId(0), true, false, |
Fan Yang | 32c5a11 | 2018-12-10 20:06:33 | [diff] [blame] | 1569 | quic::QuicStringPiece(data)); |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 1570 | std::unique_ptr<quic::QuicEncryptedPacket> server_ping( |
alyssar | 2adf3ac | 2016-05-03 17:12:58 | [diff] [blame] | 1571 | server_maker_.MakePingPacket(1, /*include_version=*/false)); |
alyssar | 2adf3ac | 2016-05-03 17:12:58 | [diff] [blame] | 1572 | MockRead reads[] = { |
| 1573 | MockRead(SYNCHRONOUS, server_ping->data(), server_ping->length(), 0), |
| 1574 | MockRead(SYNCHRONOUS, ERR_IO_PENDING, 1)}; |
| 1575 | MockWrite writes[] = { |
| 1576 | MockWrite(SYNCHRONOUS, client_ping->data(), client_ping->length(), 2), |
| 1577 | MockWrite(SYNCHRONOUS, ack_and_data_out->data(), |
| 1578 | ack_and_data_out->length(), 3)}; |
Ryan Sleevi | b8d7ea0 | 2018-05-07 20:01:01 | [diff] [blame] | 1579 | StaticSocketDataProvider socket_data(reads, writes); |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 1580 | socket_factory_.AddSocketDataProvider(&socket_data); |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 1581 | // Create connected socket. |
danakj | ad1777e | 2016-04-16 00:56:42 | [diff] [blame] | 1582 | std::unique_ptr<DatagramClientSocket> new_socket = |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 1583 | socket_factory_.CreateDatagramClientSocket(DatagramSocket::DEFAULT_BIND, |
mikecirone | f22f981 | 2016-10-04 03:40:19 | [diff] [blame] | 1584 | &net_log_, NetLogSource()); |
robpercival | 214763f | 2016-07-01 23:27:01 | [diff] [blame] | 1585 | EXPECT_THAT(new_socket->Connect(kIpEndPoint), IsOk()); |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 1586 | |
| 1587 | // Create reader and writer. |
danakj | ad1777e | 2016-04-16 00:56:42 | [diff] [blame] | 1588 | std::unique_ptr<QuicChromiumPacketReader> new_reader( |
| 1589 | new QuicChromiumPacketReader(new_socket.get(), &clock_, session_.get(), |
| 1590 | kQuicYieldAfterPacketsRead, |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 1591 | quic::QuicTime::Delta::FromMilliseconds( |
danakj | ad1777e | 2016-04-16 00:56:42 | [diff] [blame] | 1592 | kQuicYieldAfterDurationMilliseconds), |
tfarina | 4283411 | 2016-09-22 13:38:20 | [diff] [blame] | 1593 | bound_test_net_log_.bound())); |
Zhongyi Shi | d3d5f50 | 2018-08-10 00:22:22 | [diff] [blame] | 1594 | new_reader->StartReading(); |
jri | ed79618b | 2016-07-02 03:18:52 | [diff] [blame] | 1595 | std::unique_ptr<QuicChromiumPacketWriter> new_writer( |
| 1596 | CreateQuicChromiumPacketWriter(new_socket.get(), session_.get())); |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 1597 | |
| 1598 | // Migrate session. |
jri | 9f30371 | 2016-09-13 01:10:22 | [diff] [blame] | 1599 | EXPECT_TRUE(session_->MigrateToSocket( |
| 1600 | std::move(new_socket), std::move(new_reader), std::move(new_writer))); |
| 1601 | // Spin message loop to complete migration. |
| 1602 | base::RunLoop().RunUntilIdle(); |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 1603 | |
| 1604 | // Write data to session. |
Ramin Halavati | f7788ea | 2018-02-26 07:02:57 | [diff] [blame] | 1605 | QuicChromiumClientStream* stream = |
Fan Yang | 5a3bddf | 2018-10-12 10:05:50 | [diff] [blame] | 1606 | QuicChromiumClientSessionPeer::CreateOutgoingStream(session_.get()); |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 1607 | struct iovec iov[1]; |
| 1608 | iov[0].iov_base = data; |
| 1609 | iov[0].iov_len = 4; |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 1610 | quic::test::QuicStreamPeer::SendBuffer(stream).SaveStreamData(iov, 1, 0, 4); |
| 1611 | quic::test::QuicStreamPeer::SetStreamBytesWritten(4, stream); |
| 1612 | session_->WritevData(stream, stream->id(), 4, 0, quic::NO_FIN); |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 1613 | |
| 1614 | EXPECT_TRUE(socket_data.AllReadDataConsumed()); |
| 1615 | EXPECT_TRUE(socket_data.AllWriteDataConsumed()); |
| 1616 | } |
| 1617 | |
| 1618 | TEST_P(QuicChromiumClientSessionTest, MigrateToSocketMaxReaders) { |
Renjie Tang | aadb84b | 2019-08-31 01:00:23 | [diff] [blame] | 1619 | MockQuicData quic_data(version_); |
| 1620 | socket_data_.reset(); |
| 1621 | int packet_num = 1; |
| 1622 | if (VersionUsesQpack(version_.transport_version)) { |
| 1623 | quic_data.AddWrite(SYNCHRONOUS, |
| 1624 | client_maker_.MakeInitialSettingsPacket(packet_num++)); |
| 1625 | } |
| 1626 | quic_data.AddRead(ASYNC, ERR_IO_PENDING); |
| 1627 | quic_data.AddRead(ASYNC, OK); // EOF |
| 1628 | quic_data.AddSocketDataToFactory(&socket_factory_); |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 1629 | Initialize(); |
| 1630 | CompleteCryptoHandshake(); |
| 1631 | |
| 1632 | for (size_t i = 0; i < kMaxReadersPerQuicSession; ++i) { |
| 1633 | MockRead reads[] = {MockRead(SYNCHRONOUS, ERR_IO_PENDING, 1)}; |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 1634 | std::unique_ptr<quic::QuicEncryptedPacket> ping_out( |
Renjie Tang | aadb84b | 2019-08-31 01:00:23 | [diff] [blame] | 1635 | client_maker_.MakePingPacket(i + packet_num, /*include_version=*/true)); |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 1636 | MockWrite writes[] = { |
| 1637 | MockWrite(SYNCHRONOUS, ping_out->data(), ping_out->length(), i + 2)}; |
Ryan Sleevi | b8d7ea0 | 2018-05-07 20:01:01 | [diff] [blame] | 1638 | StaticSocketDataProvider socket_data(reads, writes); |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 1639 | socket_factory_.AddSocketDataProvider(&socket_data); |
| 1640 | |
| 1641 | // Create connected socket. |
danakj | ad1777e | 2016-04-16 00:56:42 | [diff] [blame] | 1642 | std::unique_ptr<DatagramClientSocket> new_socket = |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 1643 | socket_factory_.CreateDatagramClientSocket(DatagramSocket::DEFAULT_BIND, |
mikecirone | f22f981 | 2016-10-04 03:40:19 | [diff] [blame] | 1644 | &net_log_, NetLogSource()); |
robpercival | 214763f | 2016-07-01 23:27:01 | [diff] [blame] | 1645 | EXPECT_THAT(new_socket->Connect(kIpEndPoint), IsOk()); |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 1646 | |
| 1647 | // Create reader and writer. |
danakj | ad1777e | 2016-04-16 00:56:42 | [diff] [blame] | 1648 | std::unique_ptr<QuicChromiumPacketReader> new_reader( |
rch | a02807b4 | 2016-01-29 21:56:15 | [diff] [blame] | 1649 | new QuicChromiumPacketReader(new_socket.get(), &clock_, session_.get(), |
| 1650 | kQuicYieldAfterPacketsRead, |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 1651 | quic::QuicTime::Delta::FromMilliseconds( |
rch | a02807b4 | 2016-01-29 21:56:15 | [diff] [blame] | 1652 | kQuicYieldAfterDurationMilliseconds), |
tfarina | 4283411 | 2016-09-22 13:38:20 | [diff] [blame] | 1653 | bound_test_net_log_.bound())); |
Zhongyi Shi | d3d5f50 | 2018-08-10 00:22:22 | [diff] [blame] | 1654 | new_reader->StartReading(); |
jri | ed79618b | 2016-07-02 03:18:52 | [diff] [blame] | 1655 | std::unique_ptr<QuicChromiumPacketWriter> new_writer( |
| 1656 | CreateQuicChromiumPacketWriter(new_socket.get(), session_.get())); |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 1657 | |
| 1658 | // Migrate session. |
| 1659 | if (i < kMaxReadersPerQuicSession - 1) { |
jri | 9f30371 | 2016-09-13 01:10:22 | [diff] [blame] | 1660 | EXPECT_TRUE(session_->MigrateToSocket( |
| 1661 | std::move(new_socket), std::move(new_reader), std::move(new_writer))); |
| 1662 | // Spin message loop to complete migration. |
| 1663 | base::RunLoop().RunUntilIdle(); |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 1664 | EXPECT_TRUE(socket_data.AllReadDataConsumed()); |
| 1665 | EXPECT_TRUE(socket_data.AllWriteDataConsumed()); |
| 1666 | } else { |
| 1667 | // Max readers exceeded. |
jri | 9f30371 | 2016-09-13 01:10:22 | [diff] [blame] | 1668 | EXPECT_FALSE(session_->MigrateToSocket( |
| 1669 | std::move(new_socket), std::move(new_reader), std::move(new_writer))); |
Zhongyi Shi | d3d5f50 | 2018-08-10 00:22:22 | [diff] [blame] | 1670 | EXPECT_TRUE(socket_data.AllReadDataConsumed()); |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 1671 | EXPECT_FALSE(socket_data.AllWriteDataConsumed()); |
| 1672 | } |
| 1673 | } |
| 1674 | } |
| 1675 | |
Renjie Tang | 786df91 | 2019-09-03 23:17:13 | [diff] [blame] | 1676 | TEST_P(QuicChromiumClientSessionTest, MigrateToSocketReadError) { |
Renjie Tang | aadb84b | 2019-08-31 01:00:23 | [diff] [blame] | 1677 | std::unique_ptr<quic::QuicEncryptedPacket> settings_packet; |
| 1678 | std::unique_ptr<quic::QuicEncryptedPacket> client_ping = |
| 1679 | client_maker_.MakeAckAndPingPacket(2, false, 1, 1, 1); |
| 1680 | std::unique_ptr<quic::QuicEncryptedPacket> initial_ping; |
Renjie Tang | 786df91 | 2019-09-03 23:17:13 | [diff] [blame] | 1681 | std::vector<MockWrite> old_writes; |
| 1682 | std::vector<MockRead> old_reads; |
Renjie Tang | aadb84b | 2019-08-31 01:00:23 | [diff] [blame] | 1683 | if (VersionUsesQpack(version_.transport_version)) { |
| 1684 | settings_packet = client_maker_.MakeInitialSettingsPacket(1); |
Renjie Tang | 786df91 | 2019-09-03 23:17:13 | [diff] [blame] | 1685 | old_writes.push_back(MockWrite(ASYNC, settings_packet->data(), |
| 1686 | settings_packet->length(), 0)); |
Renjie Tang | aadb84b | 2019-08-31 01:00:23 | [diff] [blame] | 1687 | } else { |
| 1688 | initial_ping = client_maker_.MakePingPacket(1, true); |
Renjie Tang | 786df91 | 2019-09-03 23:17:13 | [diff] [blame] | 1689 | old_writes.push_back( |
| 1690 | MockWrite(ASYNC, initial_ping->data(), initial_ping->length(), 0)); |
Renjie Tang | aadb84b | 2019-08-31 01:00:23 | [diff] [blame] | 1691 | } |
Renjie Tang | 786df91 | 2019-09-03 23:17:13 | [diff] [blame] | 1692 | old_reads.push_back(MockRead(ASYNC, ERR_IO_PENDING, 1)); |
| 1693 | old_reads.push_back(MockRead(ASYNC, ERR_NETWORK_CHANGED, 2)); |
| 1694 | |
| 1695 | socket_data_.reset(new SequencedSocketData( |
| 1696 | base::span<const MockRead>(old_reads.data(), old_reads.size()), |
| 1697 | base::span<const MockWrite>(old_writes.data(), old_writes.size()))); |
Renjie Tang | aadb84b | 2019-08-31 01:00:23 | [diff] [blame] | 1698 | |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 1699 | std::unique_ptr<quic::QuicEncryptedPacket> server_ping( |
alyssar | 2adf3ac | 2016-05-03 17:12:58 | [diff] [blame] | 1700 | server_maker_.MakePingPacket(1, /*include_version=*/false)); |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 1701 | Initialize(); |
| 1702 | CompleteCryptoHandshake(); |
Renjie Tang | aadb84b | 2019-08-31 01:00:23 | [diff] [blame] | 1703 | |
| 1704 | if (!VersionUsesQpack(version_.transport_version)) |
| 1705 | session_->SendPing(); |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 1706 | MockWrite writes[] = { |
alyssar | 2adf3ac | 2016-05-03 17:12:58 | [diff] [blame] | 1707 | MockWrite(SYNCHRONOUS, client_ping->data(), client_ping->length(), 1)}; |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 1708 | MockRead new_reads[] = { |
alyssar | 2adf3ac | 2016-05-03 17:12:58 | [diff] [blame] | 1709 | MockRead(SYNCHRONOUS, server_ping->data(), server_ping->length(), 0), |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 1710 | MockRead(ASYNC, ERR_IO_PENDING, 2), // pause reading. |
alyssar | 2adf3ac | 2016-05-03 17:12:58 | [diff] [blame] | 1711 | MockRead(ASYNC, server_ping->data(), server_ping->length(), 3), |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 1712 | MockRead(ASYNC, ERR_IO_PENDING, 4), // pause reading |
| 1713 | MockRead(ASYNC, ERR_NETWORK_CHANGED, 5)}; |
Ryan Sleevi | b8d7ea0 | 2018-05-07 20:01:01 | [diff] [blame] | 1714 | SequencedSocketData new_socket_data(new_reads, writes); |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 1715 | socket_factory_.AddSocketDataProvider(&new_socket_data); |
| 1716 | |
| 1717 | // Create connected socket. |
danakj | ad1777e | 2016-04-16 00:56:42 | [diff] [blame] | 1718 | std::unique_ptr<DatagramClientSocket> new_socket = |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 1719 | socket_factory_.CreateDatagramClientSocket(DatagramSocket::DEFAULT_BIND, |
mikecirone | f22f981 | 2016-10-04 03:40:19 | [diff] [blame] | 1720 | &net_log_, NetLogSource()); |
robpercival | 214763f | 2016-07-01 23:27:01 | [diff] [blame] | 1721 | EXPECT_THAT(new_socket->Connect(kIpEndPoint), IsOk()); |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 1722 | |
| 1723 | // Create reader and writer. |
danakj | ad1777e | 2016-04-16 00:56:42 | [diff] [blame] | 1724 | std::unique_ptr<QuicChromiumPacketReader> new_reader( |
| 1725 | new QuicChromiumPacketReader(new_socket.get(), &clock_, session_.get(), |
| 1726 | kQuicYieldAfterPacketsRead, |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 1727 | quic::QuicTime::Delta::FromMilliseconds( |
danakj | ad1777e | 2016-04-16 00:56:42 | [diff] [blame] | 1728 | kQuicYieldAfterDurationMilliseconds), |
tfarina | 4283411 | 2016-09-22 13:38:20 | [diff] [blame] | 1729 | bound_test_net_log_.bound())); |
Zhongyi Shi | d3d5f50 | 2018-08-10 00:22:22 | [diff] [blame] | 1730 | new_reader->StartReading(); |
jri | ed79618b | 2016-07-02 03:18:52 | [diff] [blame] | 1731 | std::unique_ptr<QuicChromiumPacketWriter> new_writer( |
| 1732 | CreateQuicChromiumPacketWriter(new_socket.get(), session_.get())); |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 1733 | |
| 1734 | // Store old socket and migrate session. |
jri | 9f30371 | 2016-09-13 01:10:22 | [diff] [blame] | 1735 | EXPECT_TRUE(session_->MigrateToSocket( |
| 1736 | std::move(new_socket), std::move(new_reader), std::move(new_writer))); |
| 1737 | // Spin message loop to complete migration. |
| 1738 | base::RunLoop().RunUntilIdle(); |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 1739 | |
| 1740 | // Read error on old socket does not impact session. |
| 1741 | EXPECT_TRUE(socket_data_->IsPaused()); |
| 1742 | socket_data_->Resume(); |
| 1743 | EXPECT_TRUE(session_->connection()->connected()); |
| 1744 | EXPECT_TRUE(new_socket_data.IsPaused()); |
| 1745 | new_socket_data.Resume(); |
| 1746 | |
| 1747 | // Read error on new socket causes session close. |
| 1748 | EXPECT_TRUE(new_socket_data.IsPaused()); |
| 1749 | EXPECT_TRUE(session_->connection()->connected()); |
| 1750 | new_socket_data.Resume(); |
| 1751 | EXPECT_FALSE(session_->connection()->connected()); |
| 1752 | |
| 1753 | EXPECT_TRUE(socket_data_->AllReadDataConsumed()); |
| 1754 | EXPECT_TRUE(socket_data_->AllWriteDataConsumed()); |
| 1755 | EXPECT_TRUE(new_socket_data.AllReadDataConsumed()); |
| 1756 | EXPECT_TRUE(new_socket_data.AllWriteDataConsumed()); |
| 1757 | } |
| 1758 | |
Zhongyi Shi | 5068bb0 | 2018-08-03 02:44:09 | [diff] [blame] | 1759 | TEST_P(QuicChromiumClientSessionTest, DetectPathDegradingDuringHandshake) { |
Ryan Hamilton | 8380c65 | 2019-06-04 02:25:06 | [diff] [blame] | 1760 | if (version_.handshake_protocol == quic::PROTOCOL_TLS1_3) { |
| 1761 | // TODO(nharper, b/112643533): Figure out why this test fails when TLS is |
| 1762 | // enabled and fix it. |
| 1763 | return; |
| 1764 | } |
Zhongyi Shi | 5068bb0 | 2018-08-03 02:44:09 | [diff] [blame] | 1765 | migrate_session_early_v2_ = true; |
| 1766 | |
Ryan Hamilton | abad59e | 2019-06-06 04:02:59 | [diff] [blame] | 1767 | MockQuicData quic_data(version_); |
Zhongyi Shi | 5068bb0 | 2018-08-03 02:44:09 | [diff] [blame] | 1768 | quic_data.AddRead(SYNCHRONOUS, ERR_IO_PENDING); // Hanging read |
| 1769 | quic_data.AddWrite(SYNCHRONOUS, client_maker_.MakeDummyCHLOPacket(1)); |
| 1770 | quic_data.AddWrite(SYNCHRONOUS, client_maker_.MakeDummyCHLOPacket(2)); |
| 1771 | quic_data.AddSocketDataToFactory(&socket_factory_); |
| 1772 | |
| 1773 | // Set the crypto handshake mode to cold start and send CHLO packets. |
| 1774 | crypto_client_stream_factory_.set_handshake_mode( |
| 1775 | MockCryptoClientStream::COLD_START_WITH_CHLO_SENT); |
| 1776 | Initialize(); |
| 1777 | |
| 1778 | session_->CryptoConnect(callback_.callback()); |
| 1779 | |
| 1780 | // Check retransmission alarm is set after sending the initial CHLO packet. |
| 1781 | quic::QuicAlarm* retransmission_alarm = |
| 1782 | quic::test::QuicConnectionPeer::GetRetransmissionAlarm( |
| 1783 | session_->connection()); |
| 1784 | EXPECT_TRUE(retransmission_alarm->IsSet()); |
| 1785 | quic::QuicTime retransmission_time = retransmission_alarm->deadline(); |
| 1786 | |
| 1787 | // Check path degrading alarm is set after sending the initial CHLO packet. |
| 1788 | quic::QuicAlarm* path_degrading_alarm = |
| 1789 | quic::test::QuicConnectionPeer::GetPathDegradingAlarm( |
| 1790 | session_->connection()); |
| 1791 | EXPECT_TRUE(path_degrading_alarm->IsSet()); |
| 1792 | quic::QuicTime path_degrading_time = path_degrading_alarm->deadline(); |
| 1793 | EXPECT_LE(retransmission_time, path_degrading_time); |
| 1794 | |
| 1795 | // Do not create outgoing stream since encryption is not established. |
| 1796 | std::unique_ptr<QuicChromiumClientSession::Handle> handle = |
| 1797 | session_->CreateHandle(destination_); |
| 1798 | TestCompletionCallback callback; |
| 1799 | EXPECT_TRUE(handle->IsConnected()); |
| 1800 | EXPECT_FALSE(handle->IsCryptoHandshakeConfirmed()); |
| 1801 | EXPECT_EQ( |
| 1802 | ERR_IO_PENDING, |
| 1803 | handle->RequestStream(/*require_handshake_confirmation=*/true, |
| 1804 | callback.callback(), TRAFFIC_ANNOTATION_FOR_TESTS)); |
| 1805 | |
| 1806 | // Fire the retransmission alarm to retransmit the crypto packet. |
| 1807 | quic::QuicTime::Delta delay = retransmission_time - clock_.ApproximateNow(); |
| 1808 | clock_.AdvanceTime(delay); |
| 1809 | alarm_factory_.FireAlarm(retransmission_alarm); |
| 1810 | |
| 1811 | // Fire the path degrading alarm to notify session that path is degrading |
| 1812 | // during crypto handshake. |
| 1813 | delay = path_degrading_time - clock_.ApproximateNow(); |
| 1814 | clock_.AdvanceTime(delay); |
| 1815 | EXPECT_CALL(*session_.get(), OnPathDegrading()); |
| 1816 | alarm_factory_.FireAlarm(path_degrading_alarm); |
| 1817 | |
| 1818 | EXPECT_TRUE(session_->connection()->IsPathDegrading()); |
| 1819 | EXPECT_TRUE(quic_data.AllReadDataConsumed()); |
| 1820 | EXPECT_TRUE(quic_data.AllWriteDataConsumed()); |
| 1821 | } |
| 1822 | |
Yixin Wang | f65c4b6d | 2018-03-08 22:37:40 | [diff] [blame] | 1823 | TEST_P(QuicChromiumClientSessionTest, RetransmittableOnWireTimeout) { |
| 1824 | migrate_session_early_v2_ = true; |
| 1825 | |
Ryan Hamilton | abad59e | 2019-06-06 04:02:59 | [diff] [blame] | 1826 | MockQuicData quic_data(version_); |
Renjie Tang | aadb84b | 2019-08-31 01:00:23 | [diff] [blame] | 1827 | int packet_num = 1; |
| 1828 | if (VersionUsesQpack(version_.transport_version)) { |
| 1829 | quic_data.AddWrite(SYNCHRONOUS, |
| 1830 | client_maker_.MakeInitialSettingsPacket(packet_num++)); |
| 1831 | } |
| 1832 | quic_data.AddWrite(SYNCHRONOUS, |
| 1833 | client_maker_.MakePingPacket(packet_num++, true)); |
Yixin Wang | f65c4b6d | 2018-03-08 22:37:40 | [diff] [blame] | 1834 | |
Renjie Tang | aadb84b | 2019-08-31 01:00:23 | [diff] [blame] | 1835 | quic_data.AddRead( |
| 1836 | ASYNC, server_maker_.MakeAckPacket(1, packet_num - 1, 1, 1, false)); |
| 1837 | |
| 1838 | quic_data.AddWrite(SYNCHRONOUS, |
| 1839 | client_maker_.MakePingPacket(packet_num++, false)); |
Yixin Wang | f65c4b6d | 2018-03-08 22:37:40 | [diff] [blame] | 1840 | quic_data.AddRead(ASYNC, ERR_IO_PENDING); |
| 1841 | quic_data.AddRead(ASYNC, OK); // EOF |
| 1842 | quic_data.AddSocketDataToFactory(&socket_factory_); |
| 1843 | |
| 1844 | Initialize(); |
| 1845 | CompleteCryptoHandshake(); |
| 1846 | |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 1847 | EXPECT_EQ(quic::QuicTime::Delta::FromMilliseconds(100), |
Yixin Wang | f65c4b6d | 2018-03-08 22:37:40 | [diff] [blame] | 1848 | session_->connection()->retransmittable_on_wire_timeout()); |
| 1849 | |
| 1850 | // Open a stream since the connection only sends PINGs to keep a |
| 1851 | // retransmittable packet on the wire if there's an open stream. |
Fan Yang | 5a3bddf | 2018-10-12 10:05:50 | [diff] [blame] | 1852 | EXPECT_TRUE( |
| 1853 | QuicChromiumClientSessionPeer::CreateOutgoingStream(session_.get())); |
Yixin Wang | f65c4b6d | 2018-03-08 22:37:40 | [diff] [blame] | 1854 | |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 1855 | quic::QuicAlarm* alarm = |
Zhongyi Shi | 4bd7bf4 | 2018-11-06 21:05:10 | [diff] [blame] | 1856 | quic::test::QuicConnectionPeer::GetPingAlarm(session_->connection()); |
Yixin Wang | f65c4b6d | 2018-03-08 22:37:40 | [diff] [blame] | 1857 | EXPECT_FALSE(alarm->IsSet()); |
| 1858 | |
| 1859 | // Send PING, which will be ACKed by the server. After the ACK, there will be |
| 1860 | // no retransmittable packets on the wire, so the alarm should be set. |
| 1861 | session_->SendPing(); |
| 1862 | base::RunLoop().RunUntilIdle(); |
| 1863 | EXPECT_TRUE(alarm->IsSet()); |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 1864 | EXPECT_EQ( |
| 1865 | clock_.ApproximateNow() + quic::QuicTime::Delta::FromMilliseconds(100), |
| 1866 | alarm->deadline()); |
Yixin Wang | f65c4b6d | 2018-03-08 22:37:40 | [diff] [blame] | 1867 | |
| 1868 | // Advance clock and simulate the alarm firing. This should cause a PING to be |
| 1869 | // sent. |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 1870 | clock_.AdvanceTime(quic::QuicTime::Delta::FromMilliseconds(100)); |
Yixin Wang | f65c4b6d | 2018-03-08 22:37:40 | [diff] [blame] | 1871 | alarm_factory_.FireAlarm(alarm); |
| 1872 | base::RunLoop().RunUntilIdle(); |
| 1873 | |
| 1874 | quic_data.Resume(); |
| 1875 | EXPECT_TRUE(quic_data.AllReadDataConsumed()); |
| 1876 | EXPECT_TRUE(quic_data.AllWriteDataConsumed()); |
| 1877 | } |
| 1878 | |
[email protected] | dd3fd0e | 2012-11-04 05:14:40 | [diff] [blame] | 1879 | } // namespace |
| 1880 | } // namespace test |
| 1881 | } // namespace net |