Re: [quicwg/base-drafts] Do not reset pto_count on Initial ACKs (#3551)

[Martin Thomson <[e-mail]>]

@martinthomson approved this pull request.



> @@ -1139,8 +1144,9 @@ OnAckReceived(ack, pn_space):
     OnPacketAcked(acked_packet.packet_number, pn_space)
 
   DetectLostPackets(pn_space)
-
-  pto_count = 0
+  // Reset pto_count, unless it's an Initial ACK.
+  if (pn_space != Initial)

```suggestion
  if (pn_space != Initial):
```

> @@ -501,7 +501,12 @@ and prevents a server from sending a 1-RTT packet on a PTO expiration before it
 has the keys to process an acknowledgement.
 
 When a PTO timer expires, the PTO period MUST be set to twice its current
-value. This exponential reduction in the sender's rate is important because
+value. The PTO is set back to the original value upon receiving an

```suggestion
value. The PTO period is set back to the original value upon receiving an
```

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/quicwg/base-drafts/pull/3551#pullrequestreview-383484088