Re: [quicwg/base-drafts] Fix to ECN section regarding validation (#2113)

[Benjamin Saunders <[e-mail]>]

Ralith commented on this pull request.

That addresses the issue, thanks!

>  
 * The total increase in ECT(0), ECT(1), and CE counters reported in the ACK
   frame MUST be at least the total number of QUIC packets newly acknowledged in
-  this ACK frame.
+  this ACK frame. This detects if the network changes ECT(0), ECT(1) or CE to
+  Not-ECT.
+
+If the sender does not have state to determine if a particular packet number is
+newly acknowledged or not, then the verifications using this acknowledgement
+cannot be performed. If an acknowledgement arrive with a packet number that
+isn't the highest to arrived so far, i.e. some reordering has occured, then the
+verification may fail erronously. To prevent this the sender MUST NOT perform
+ECN verification using older acknwoeldgements. By the end of the verification

```suggestion
ECN verification using older acknowledgements. By the end of the verification
```

-- 
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/2113#pullrequestreview-184305206