Re: [quicwg/base-drafts] Update draft-ietf-quic-transport.md (#3693)

[ianswett <[e-mail]>]

@ianswett commented on this pull request.

Thanks for updating this.  I'm worried there are some unintended changes in this PR, I've found a few so far.

> @@ -3797,17 +3797,23 @@ later time in the connection.
 The QUIC packet size includes the QUIC header and protected payload, but not the
 UDP or IP header.
 
+QUIC defines the minimum QUIC packet size as at least 1280 bytes.

```suggestion
QUIC depends upon a minimum packet size as at least 1280 bytes.
```

>  A client MUST expand the payload of all UDP datagrams carrying Initial packets
-to at least 1200 bytes, by adding PADDING frames to the Initial packet or by
+to at least the minimum QUIC packet size, by adding PADDING frames to the Initial packet or by

Keep this as 1200 bytes.

```suggestion
to at least 1200 bytes, by adding PADDING frames to the Initial packet or by
```

>  In the absence of these mechanisms, QUIC endpoints SHOULD NOT send IP packets
-larger than 1280 bytes. Assuming the minimum IP header size, this results in a
-QUIC maximum packet size of 1232 bytes for IPv6 and 1252 bytes for IPv4. A QUIC
-implementation MAY be more conservative in computing the QUIC maximum packet
-size to allow for unknown tunnel overheads or IP header options/extensions.
+larger than the minimum QUIC packet size. 
+
+All QUIC
+packets (except for PMTUD/DPLPMTUD probe packets) SHOULD be sized to fit within the
+maximum packet size to avoid the packet being fragmented or dropped
+{{?RFC8085}}.
+
+If a QUIC endpoint determines that the PMTU between any pair of local and remote
+IP addresses has fallen below the minimum QUIC packet size, it MUST immediately 

Please keep the existing wording of "smallest allowed maximum packet size", since that's very different from the minimum QUIC packet size.

```suggestion
IP addresses has fallen below the smallest allowed maximum packet size, it MUST immediately 
```

-- 
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/3693#pullrequestreview-417836536