commit 965ce09b8d738f0d362551b098dc1e7d8782e354
author dschinazi <dschinazi@google.com> Thu May 23 06:29:01 2019 -0700
committer Copybara-Service <copybara-worker@google.com> Tue May 28 02:59:37 2019 -0700
tree 73c03ea0ce5785f2edd7fb1853dfe53d4c5f323a
parent 8cf087e619efcccf61831ff077ba37b0fa721c72

Add more debug logs that were useful during interop

gfe-relnote: n/a, debug-only logs
PiperOrigin-RevId: 249631003
Change-Id: Ib510bca1f4f1cb010e6449849d326d0a20df834d

quic/core/quic_connection.cc

diff --git a/quic/core/quic_connection.cc b/quic/core/quic_connection.cc
index 4e93fdf..db717b0 100644
--- a/quic/core/quic_connection.cc
+++ b/quic/core/quic_connection.cc
@@ -1809,10 +1809,15 @@
   QUIC_DLOG(INFO) << ENDPOINT << "Sending version negotiation packet: {"
                   << ParsedQuicVersionVectorToString(
                          framer_.supported_versions())
-                  << "}, ietf_quic: " << ietf_quic;
+                  << "}, " << (ietf_quic ? "" : "!") << "ietf_quic";
   std::unique_ptr<QuicEncryptedPacket> version_packet(
       packet_generator_.SerializeVersionNegotiationPacket(
           ietf_quic, framer_.supported_versions()));
+  QUIC_DVLOG(2) << ENDPOINT << "Sending version negotiation packet: {"
+                << ParsedQuicVersionVectorToString(framer_.supported_versions())
+                << "}, " << (ietf_quic ? "" : "!") << "ietf_quic:" << std::endl
+                << QuicTextUtils::HexDump(QuicStringPiece(
+                       version_packet->data(), version_packet->length()));
   WriteResult result = writer_->WritePacket(
       version_packet->data(), version_packet->length(), self_address().host(),
       peer_address(), per_packet_options_);
@@ -3652,6 +3657,12 @@
   DCHECK_EQ(IsRetransmittable(*probing_packet), NO_RETRANSMITTABLE_DATA);
 
   const QuicTime packet_send_time = clock_->Now();
+  QUIC_DVLOG(2) << ENDPOINT
+                << "Sending path probe packet for server connection ID "
+                << server_connection_id_ << std::endl
+                << QuicTextUtils::HexDump(
+                       QuicStringPiece(probing_packet->encrypted_buffer,
+                                       probing_packet->encrypted_length));
   WriteResult result = probing_writer->WritePacket(
       probing_packet->encrypted_buffer, probing_packet->encrypted_length,
       self_address().host(), peer_address, per_packet_options_);
@@ -4044,7 +4055,9 @@
   }
   if (sent_packet_manager_.handshake_confirmed()) {
     // A forward secure packet has been received.
-    QUIC_BUG_IF(encryption_level_ != ENCRYPTION_FORWARD_SECURE);
+    QUIC_BUG_IF(encryption_level_ != ENCRYPTION_FORWARD_SECURE)
+        << ENDPOINT << "Unexpected connection close encryption level "
+        << QuicUtils::EncryptionLevelToString(encryption_level_);
     return ENCRYPTION_FORWARD_SECURE;
   }
   if (framer_.HasEncrypterOfEncryptionLevel(ENCRYPTION_ZERO_RTT)) {