commit 6aa0d5322bb1534814d288064b686b55eaf3f129
author fayang <fayang@google.com> Tue Jan 10 20:06:35 2023 -0800
committer Copybara-Service <copybara-worker@google.com> Tue Jan 10 20:08:34 2023 -0800
tree 6b0e1678347848030147b4b0812505a2a3b1aa49
parent c290b82e379656bf045d4d31a05e90919eaeea09

Change SPAD from client_connection_option to connection_option. Server only send connection ID and token for preferred address if SPAD has been received.

Note for chromium merge:
This would cause some QuicStreamFactoryTest tests failures. Please change
 quic_params_->client_connection_options.push_back(quic::kSPAD)
 to
 quic_params_->connection_options.push_back(quic::kSPAD)

PiperOrigin-RevId: 501162626

quiche/quic/core/quic_connection_test.cc

diff --git a/quiche/quic/core/quic_connection_test.cc b/quiche/quic/core/quic_connection_test.cc
index 2a5c9c1..b567587 100644
--- a/quiche/quic/core/quic_connection_test.cc
+++ b/quiche/quic/core/quic_connection_test.cc
@@ -1491,7 +1491,6 @@
     connection_.RemoveEncrypter(ENCRYPTION_INITIAL);
     connection_.SetDefaultEncryptionLevel(ENCRYPTION_FORWARD_SECURE);
 
-    config.SetConnectionOptionsToSend(QuicTagVector{kRVCM});
     QuicConfigPeer::SetReceivedStatelessResetToken(&config,
                                                    kTestStatelessResetToken);
     QuicConfigPeer::SetReceivedAlternateServerAddress(&config,
@@ -3002,8 +3001,7 @@
   connection_.SetDefaultEncryptionLevel(ENCRYPTION_FORWARD_SECURE);
 
   QuicConfig config;
-  config.SetClientConnectionOptions(QuicTagVector{kSPAD});
-  config.SetConnectionOptionsToSend(QuicTagVector{kRVCM});
+  config.SetConnectionOptionsToSend(QuicTagVector{kRVCM, kSPAD});
   QuicConfigPeer::SetReceivedStatelessResetToken(&config,
                                                  kTestStatelessResetToken);
   QuicConfigPeer::SetReceivedAlternateServerAddress(&config,
@@ -16065,7 +16063,7 @@
     return;
   }
   QuicConfig config;
-  config.SetClientConnectionOptions(QuicTagVector{kSPAD});
+  config.SetConnectionOptionsToSend(QuicTagVector{kRVCM, kSPAD});
   ServerPreferredAddressInit(config);
   const QuicSocketAddress kNewSelfAddress =
       QuicSocketAddress(QuicIpAddress::Loopback6(), /*port=*/23456);
@@ -16147,7 +16145,7 @@
     return;
   }
   QuicConfig config;
-  config.SetClientConnectionOptions(QuicTagVector{kSPAD});
+  config.SetConnectionOptionsToSend(QuicTagVector{kRVCM, kSPAD});
   ServerPreferredAddressInit(config);
   const QuicSocketAddress kNewSelfAddress =
       QuicSocketAddress(QuicIpAddress::Loopback6(), /*port=*/23456);
@@ -16215,7 +16213,7 @@
     return;
   }
   QuicConfig config;
-  config.SetClientConnectionOptions(QuicTagVector{kSPAD});
+  config.SetConnectionOptionsToSend(QuicTagVector{kRVCM, kSPAD});
   ServerPreferredAddressInit(config);
   const QuicSocketAddress kNewSelfAddress =
       QuicSocketAddress(QuicIpAddress::Loopback6(), /*port=*/23456);
@@ -16297,7 +16295,8 @@
                 &connection_));
       }));
   QuicConfig config;
-  config.SetClientConnectionOptions(QuicTagVector{kSPAD, kSPA2});
+  config.SetConnectionOptionsToSend(QuicTagVector{kRVCM, kSPAD});
+  config.SetClientConnectionOptions(QuicTagVector{kSPA2});
   ServerPreferredAddressInit(config);
   EXPECT_TRUE(connection_.HasPendingPathValidation());
   ASSERT_FALSE(new_writer.path_challenge_frames().empty());
@@ -16334,7 +16333,8 @@
                 &connection_));
       }));
   QuicConfig config;
-  config.SetClientConnectionOptions(QuicTagVector{kSPAD, kSPA2});
+  config.SetConnectionOptionsToSend(QuicTagVector{kRVCM, kSPAD});
+  config.SetClientConnectionOptions(QuicTagVector{kSPA2});
   ServerPreferredAddressInit(config);
   EXPECT_TRUE(connection_.HasPendingPathValidation());
   ASSERT_FALSE(new_writer.path_challenge_frames().empty());
@@ -16377,7 +16377,8 @@
                 &connection_));
       }));
   QuicConfig config;
-  config.SetClientConnectionOptions(QuicTagVector{kSPAD, kSPA2});
+  config.SetConnectionOptionsToSend(QuicTagVector{kRVCM, kSPAD});
+  config.SetClientConnectionOptions(QuicTagVector{kSPA2});
   ServerPreferredAddressInit(config);
   EXPECT_TRUE(connection_.HasPendingPathValidation());
   ASSERT_FALSE(new_writer.path_challenge_frames().empty());
@@ -16410,7 +16411,8 @@
     return;
   }
   QuicConfig config;
-  config.SetClientConnectionOptions(QuicTagVector{kMPQC, kSPAD});
+  config.SetConnectionOptionsToSend(QuicTagVector{kRVCM, kSPAD});
+  config.SetClientConnectionOptions(QuicTagVector{kMPQC});
   ServerPreferredAddressInit(config);
   if (!connection_.connection_migration_use_new_cid()) {
     return;