Re: [quicwg/base-drafts] Fix some of the nits in #3214 (#4016)

[Mike Bishop <[e-mail]>]

@MikeBishop commented on this pull request.



> +The definition of new transport parameters ({{new-transport-parameters}}) is
+REQUIRED to specify whether they MUST, MAY, or MUST NOT be stored for 0-RTT. A

```suggestion
The definition of a new transport parameter ({{new-transport-parameters}}) is
REQUIRED to specify whether it MUST, MAY, or MUST NOT be stored for 0-RTT. A
```
I think the singular reads better, especially with the change in keyword.

> @@ -1889,7 +1891,7 @@ confirms that the client received the Initial packet from the server.  Once the
 server has successfully processed a Handshake packet from the client, it can
 consider the client address to have been validated.
 
-Additionally, a server MAY consider the client address valididated if the
+Additionally, a server MAY consider the client address validated if the

😄

> -subsequent ACK frames; see {{generating-acks}} and {{frame-ack}}.  Note that
-this requires being able to read the ECN codepoints from the enclosing IP
-packet, which is not possible on all platforms.

I'm not sure about this one.  IIRC, we were pretty deliberate about adding this text to note that compliance with the ECN guidance was conditional on OS support for doing so.

-- 
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/4016#pullrequestreview-470768701