[trafficserver] 01/02: Process 0-RTT packet on state_connection_established

[ma...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

masaori pushed a commit to branch quic-latest
in repository https://gitbox.apache.org/repos/asf/trafficserver.git

commit 1fdeea385abbe66f2e2ed051a46a54716e08e038
Author: Masaori Koshiba <ma...@apache.org>
AuthorDate: Wed Aug 8 11:53:36 2018 +0900

    Process 0-RTT packet on state_connection_established
    
    Some client send STREAM frame on 0-RTT packet after handshake is completed.
    Need to clarify, but it looks not prohibited in draft-13.
---
 iocore/net/QUICNetVConnection.cc | 7 +------
 1 file changed, 1 insertion(+), 6 deletions(-)

diff --git a/iocore/net/QUICNetVConnection.cc b/iocore/net/QUICNetVConnection.cc
index 6cc1c75..6b92aed 100644
--- a/iocore/net/QUICNetVConnection.cc
+++ b/iocore/net/QUICNetVConnection.cc
@@ -1014,15 +1014,10 @@ QUICNetVConnection::_state_common_receive_packet()
       break;
     case QUICPacketType::INITIAL:
     case QUICPacketType::HANDSHAKE:
+    case QUICPacketType::ZERO_RTT_PROTECTED:
       // Pass packet to _recv_and_ack to send ack to the packet. Stream data will be discarded by offset mismatch.
       error = this->_recv_and_ack(std::move(p));
       break;
-    case QUICPacketType::ZERO_RTT_PROTECTED:
-      // Probably retransmitted unintentionally. Do nothing.
-      QUICConDebug("Ignore %s(%" PRIu8 ")", QUICDebugNames::packet_type(p->type()), static_cast<uint8_t>(p->type()));
-
-      error = QUICErrorUPtr(new QUICNoError());
-      break;
     default:
       QUICConDebug("Unknown packet type: %s(%" PRIu8 ")", QUICDebugNames::packet_type(p->type()), static_cast<uint8_t>(p->type()));