gr1ffon/quic-go
1
0
Fork 0
forked from quic-go/quic-go
Code Pull Requests Activity

remove special case for Retry packets from the session

Browse Source 
Retry packets are never passed to the session, so there's no need to
handle them there.
This commit is contained in:
Marten Seemann
2019-01-01 15:24:30 +07:00
parent 3fb58da7e6
commit 05645b546c
1 changed files with 3 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
session.go
View File

@@ -539,13 +539,9 @@ func (s *session) handleUnpackedPacket(packet *unpackedPacket, rcvTime time.Time
} }
} }
// If this is a Retry packet, there's no need to send an ACK. isRetransmittable := ackhandler.HasRetransmittableFrames(packet.frames)
// The session will be closed and recreated as soon as the crypto setup processed the HRR. if err := s.receivedPacketHandler.ReceivedPacket(packet.packetNumber, rcvTime, isRetransmittable); err != nil {
if packet.hdr.Type != protocol.PacketTypeRetry { return err
isRetransmittable := ackhandler.HasRetransmittableFrames(packet.frames)
if err := s.receivedPacketHandler.ReceivedPacket(packet.packetNumber, rcvTime, isRetransmittable); err != nil {
return err
}
} }
return s.handleFrames(packet.frames, packet.packetNumber, packet.encryptionLevel) return s.handleFrames(packet.frames, packet.packetNumber, packet.encryptionLevel)