commit d9467b578a7b699887b398f6d5501e3c63d85c38
author dschinazi <dschinazi@google.com> Wed Apr 03 16:47:08 2019 -0700
committer Copybara-Service <copybara-worker@google.com> Thu Apr 04 09:14:08 2019 -0700
tree b1ee0d07a237037ce303db23d5570bcae17c4fff
parent d1967a23b1bd5f7e591573a4ec2c4bb5af842546

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

quic/core/quic_connection.cc

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);
       }