Skip to content

Commit

Permalink
reverted as per PR comments
Browse files Browse the repository at this point in the history
  • Loading branch information
SahibYar committed Jul 6, 2023
1 parent b7e3f29 commit b1a88ec
Showing 1 changed file with 7 additions and 5 deletions.
12 changes: 7 additions & 5 deletions packet_packer.go
Expand Up @@ -331,12 +331,14 @@ func (p *packetPacker) PackCoalescedPacket(onlyAck bool, maxPacketSize protocol.
initialHdr, initialPayload = p.maybeGetCryptoPacket(maxPacketSize-protocol.ByteCount(initialSealer.Overhead()), protocol.EncryptionInitial, onlyAck, true, v)
if initialPayload.length > 0 {
size += p.longHeaderPacketLength(initialHdr, initialPayload, v) + protocol.ByteCount(initialSealer.Overhead())
}

if size >= maxPacketSize-protocol.MinCoalescedPacketSize {
ping := ackhandler.Frame{Frame: &wire.PingFrame{}}
initialPayload.frames = append(initialPayload.frames, ping)
initialPayload.length += ping.Frame.Length(v)
}
// Add a PING frame to Initial packets, if they don't contain any other frames.
// This is not strictly necessary, but it helps speed up the handshake.
if !onlyAck && len(initialPayload.frames) == 0 && initialPayload.ack != nil {
ping := ackhandler.Frame{Frame: &wire.PingFrame{}}
initialPayload.frames = append(initialPayload.frames, ping)
initialPayload.length += ping.Frame.Length(v)
}
}
// Add a Handshake packet.
Expand Down

0 comments on commit b1a88ec

Please sign in to comment.