commit 9075615a2d5d0bafa8af6948e3cff6b5e1631393
author rch <rch@google.com> Fri Jan 10 15:09:04 2020 -0800
committer Copybara-Service <copybara-worker@google.com> Fri Jan 10 15:09:54 2020 -0800
tree 70551c5d9b74bc2ac195dd85b5644670649f9f24
parent 402103c13366ac8644a294d6701d7545c9d88979

gfe-relnote: When a QUIC client receives an unexpected crypto tag, include the tag in the connection close details. Not flag protected.

PiperOrigin-RevId: 289173708
Change-Id: I7ac33fb410670e7603f124058c1e5a6eb37c9f59

quic/core/quic_crypto_client_handshaker.cc

diff --git a/quic/core/quic_crypto_client_handshaker.cc b/quic/core/quic_crypto_client_handshaker.cc
index eed6993..ae48bdc 100644
--- a/quic/core/quic_crypto_client_handshaker.cc
+++ b/quic/core/quic_crypto_client_handshaker.cc
@@ -516,8 +516,10 @@
   }
 
   if (in->tag() != kSHLO) {
-    stream_->CloseConnectionWithDetails(QUIC_INVALID_CRYPTO_MESSAGE_TYPE,
-                                        "Expected SHLO or REJ");
+    stream_->CloseConnectionWithDetails(
+        QUIC_INVALID_CRYPTO_MESSAGE_TYPE,
+        quiche::QuicheStrCat("Expected SHLO or REJ. Received: ",
+                             QuicTagToString(in->tag())));
     return;
   }