Add DV2 logs for received packets and clarify a log statement We already had DV2 logs for sent packets, this adds them for received ones. These were critical when debugging interop issues. gfe-relnote: n/a, only changes log statements PiperOrigin-RevId: 241832618 Change-Id: I6ff675c680b66cd90c3ac920a0d3dce75c8b2e9e
diff --git a/quic/core/quic_connection.cc b/quic/core/quic_connection.cc index c27d1cb..68567b7 100644 --- a/quic/core/quic_connection.cc +++ b/quic/core/quic_connection.cc
@@ -1867,6 +1867,10 @@ if (!connected_) { return; } + QUIC_DVLOG(2) << ENDPOINT << "Received encrypted " << packet.length() + << " bytes:" << std::endl + << QuicTextUtils::HexDump( + QuicStringPiece(packet.data(), packet.length())); QUIC_BUG_IF(current_packet_data_ != nullptr) << "ProcessUdpPacket must not be called while processing a packet."; if (debug_visitor_ != nullptr) { @@ -2109,8 +2113,15 @@ last_decrypted_packet_level_, packet_number) : received_packet_manager_.IsAwaitingPacket(packet_number); if (!is_awaiting) { - QUIC_DLOG(INFO) << ENDPOINT << "Packet " << packet_number - << " no longer being waited for. Discarding."; + if (use_uber_received_packet_manager_) { + QUIC_DLOG(INFO) << ENDPOINT << "Packet " << packet_number + << " no longer being waited for at level " + << static_cast<int>(last_decrypted_packet_level_) + << ". Discarding."; + } else { + QUIC_DLOG(INFO) << ENDPOINT << "Packet " << packet_number + << " no longer being waited for. Discarding."; + } if (debug_visitor_ != nullptr) { debug_visitor_->OnDuplicatePacket(packet_number); }