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

[Martin Thomson <[e-mail]>]

@martinthomson requested changes on this pull request.

What Mike said applies also.

> +port reuse. An endpoint MUST NOT use an IP address and port pair for multiple
+connections unless it is certain that those protocol features are not in use.

This misses the zero-length connection ID qualification, which is the key point.

> @@ -1232,7 +1232,9 @@ cannot be reverted.
 Valid packets sent to clients always include a Destination Connection ID that
 matches a value the client selects.  Clients that choose to receive
 zero-length connection IDs can use the local address and port to identify a
-connection.  Packets that do not match an existing connection are discarded.
+connection.  Packets that do not match an existing connection, based on
+Destination Connection ID or, if zero-length, local IP address and port,

```suggestion
Destination Connection ID or, if this value zero-length, local IP address and port,
```

> +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

Also, MUST > REQUIRED.

-- 
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-471048275