[trafficserver] branch quic-latest updated: Send ACK frames during handshake

[ma...@apache.org]

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

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


The following commit(s) were added to refs/heads/quic-latest by this push:
     new d1ef4a4  Send ACK frames during handshake
d1ef4a4 is described below

commit d1ef4a49c61a9628f2359cbae3ce385f76128bb0
Author: Masakazu Kitajo <ma...@apache.org>
AuthorDate: Tue Dec 19 16:28:40 2017 +0900

    Send ACK frames during handshake
    
    This fixes #2955
---
 iocore/net/QUICNetVConnection.cc | 13 +------------
 1 file changed, 1 insertion(+), 12 deletions(-)

diff --git a/iocore/net/QUICNetVConnection.cc b/iocore/net/QUICNetVConnection.cc
index e3621ff..b1be90f 100644
--- a/iocore/net/QUICNetVConnection.cc
+++ b/iocore/net/QUICNetVConnection.cc
@@ -625,18 +625,7 @@ QUICNetVConnection::_state_handshake_process_initial_client_packet(QUICPacketUPt
   // Start handshake
   QUICErrorUPtr error = this->_handshake_handler->start(packet.get(), &this->_packet_factory);
   if (this->_handshake_handler->is_version_negotiated()) {
-    bool should_send_ack;
-    error = this->_frame_dispatcher->receive_frames(packet->payload(), packet->payload_size(), should_send_ack);
-    if (error->cls != QUICErrorClass::NONE) {
-      return error;
-    }
-    int ret = this->_local_flow_controller->update(this->_stream_manager->total_offset_received());
-    Debug("quic_flow_ctrl", "Connection [%" PRIx64 "] [LOCAL] %" PRIu64 "/%" PRIu64,
-          static_cast<uint64_t>(this->_quic_connection_id), this->_local_flow_controller->current_offset(),
-          this->_local_flow_controller->current_limit());
-    if (ret != 0) {
-      return QUICErrorUPtr(new QUICConnectionError(QUICTransErrorCode::FLOW_CONTROL_ERROR));
-    }
+    error = this->_recv_and_ack(packet->payload(), packet->payload_size(), packet->packet_number());
   } else {
     // Perhaps response packets for initial client packet were lost, but no need to start handshake again because loss detector will
     // retransmit the packets.

-- 
To stop receiving notification emails like this one, please contact
['"commits@trafficserver.apache.org" <co...@trafficserver.apache.org>'].