Internal change
PiperOrigin-RevId: 448995434
diff --git a/quiche/quic/core/quic_connection.cc b/quiche/quic/core/quic_connection.cc
index 49681b7..2b611f7 100644
--- a/quiche/quic/core/quic_connection.cc
+++ b/quiche/quic/core/quic_connection.cc
@@ -3014,15 +3014,8 @@
", encryption level: ",
EncryptionLevelToString(
last_received_packet_info_.decrypted_level));
- if (GetQuicReloadableFlag(
- quic_drop_packets_with_changed_server_address)) {
- QUIC_LOG_EVERY_N_SEC(INFO, 100) << error_details;
- QUIC_CODE_COUNT(quic_dropped_packets_with_changed_server_address);
- return false;
- }
- QUIC_PEER_BUG(Server self address change) << error_details;
- CloseConnection(QUIC_ERROR_MIGRATING_ADDRESS, error_details,
- ConnectionCloseBehavior::SEND_CONNECTION_CLOSE_PACKET);
+ QUIC_LOG_EVERY_N_SEC(INFO, 100) << error_details;
+ QUIC_CODE_COUNT(quic_dropped_packets_with_changed_server_address);
return false;
}
}
diff --git a/quiche/quic/core/quic_connection_test.cc b/quiche/quic/core/quic_connection_test.cc
index 692e3ef..b8cac26 100644
--- a/quiche/quic/core/quic_connection_test.cc
+++ b/quiche/quic/core/quic_connection_test.cc
@@ -1563,23 +1563,9 @@
QuicSocketAddress self_address(host, 123);
EXPECT_EQ(0u, connection_.GetStats().packets_dropped);
EXPECT_CALL(visitor_, AllowSelfAddressChange()).WillOnce(Return(false));
- if (GetQuicReloadableFlag(quic_drop_packets_with_changed_server_address)) {
- ProcessFramePacketWithAddresses(MakeCryptoFrame(), self_address,
- kPeerAddress, ENCRYPTION_INITIAL);
- EXPECT_TRUE(connection_.connected());
- EXPECT_EQ(1u, connection_.GetStats().packets_dropped);
- return;
- }
- if (version().handshake_protocol == PROTOCOL_TLS1_3) {
- EXPECT_CALL(visitor_, BeforeConnectionCloseSent());
- }
- EXPECT_CALL(visitor_, OnConnectionClosed(_, _));
- EXPECT_QUIC_PEER_BUG(
- ProcessFramePacketWithAddresses(MakeCryptoFrame(), self_address,
- kPeerAddress, ENCRYPTION_INITIAL),
- "Self address migration is not supported at the server");
- EXPECT_FALSE(connection_.connected());
- TestConnectionCloseQuicErrorCode(QUIC_ERROR_MIGRATING_ADDRESS);
+ ProcessFramePacketWithAddresses(MakeCryptoFrame(), self_address, kPeerAddress,
+ ENCRYPTION_INITIAL);
+ EXPECT_TRUE(connection_.connected());
EXPECT_EQ(1u, connection_.GetStats().packets_dropped);
}
diff --git a/quiche/quic/core/quic_flags_list.h b/quiche/quic/core/quic_flags_list.h
index ab90711..3278596 100644
--- a/quiche/quic/core/quic_flags_list.h
+++ b/quiche/quic/core/quic_flags_list.h
@@ -77,8 +77,6 @@
QUIC_FLAG(FLAGS_quic_reloadable_flag_quic_retire_cid_on_reverse_path_validation_failure, true)
// If true, server sends bandwidth eastimate when network is idle for a while.
QUIC_FLAG(FLAGS_quic_restart_flag_quic_enable_sending_bandwidth_estimate_when_network_idle, true)
-// If true, servers drop received packets with changed server address.
-QUIC_FLAG(FLAGS_quic_reloadable_flag_quic_drop_packets_with_changed_server_address, true)
// If true, set burst token to 2 in cwnd bootstrapping experiment.
QUIC_FLAG(FLAGS_quic_reloadable_flag_quic_conservative_bursts, false)
// If true, stop resetting ideal_next_packet_send_time_ in pacing sender.
