Downgrade QUIC_BUG for late packet processing
Formerly a QUIC_BUG() would be triggered when it was found a packet had taken two minutes between being read and processed. Since this can happen when a device or process is suspended, it was found this was actually triggered in Chrome builds in the wild.
Downgrade the QUIC_BUG() to a LOG(WARNING).
Not protected by a flag as it is a diagnostic change only.
PiperOrigin-RevId: 533113227
diff --git a/quiche/quic/core/quic_connection.cc b/quiche/quic/core/quic_connection.cc
index 4c2fe67..b68aad0 100644
--- a/quiche/quic/core/quic_connection.cc
+++ b/quiche/quic/core/quic_connection.cc
@@ -2760,10 +2760,10 @@
// Ensure the time coming from the packet reader is within 2 minutes of now.
if (std::abs((packet.receipt_time() - clock_->ApproximateNow()).ToSeconds()) >
2 * 60) {
- QUIC_BUG(quic_bug_10511_21)
- << "Packet receipt time:" << packet.receipt_time().ToDebuggingValue()
- << " too far from current time:"
- << clock_->ApproximateNow().ToDebuggingValue();
+ QUIC_LOG(WARNING) << "(Formerly quic_bug_10511_21): Packet receipt time: "
+ << packet.receipt_time().ToDebuggingValue()
+ << " too far from current time: "
+ << clock_->ApproximateNow().ToDebuggingValue();
}
QUIC_DVLOG(1) << ENDPOINT << "time of last received packet: "
<< packet.receipt_time().ToDebuggingValue() << " from peer "