Re: [quicwg/base-drafts] renamed max_packet_size to max_udp_size (#3473)

[David Schinazi <[e-mail]>]

DavidSchinazi commented on this pull request.



>  
-: The maximum packet size parameter is an integer value that limits the size of
-  packets that the endpoint is willing to receive.  This indicates that packets
-  larger than this limit will be dropped.  The default for this parameter is the
-  maximum permitted UDP payload of 65527.  Values below 1200 are invalid.  This
-  limit only applies to protected packets ({{packet-protected}}).
+: The maximum udp size parameter is an integer value that limits the size of
+  UDP payloads that the endpoint is willing to receive.  This indicates that
+  packets larger than this limit are likely not to be processed by the receiver.

Are we really concerned about nodes performing this check? My understanding is that we added this transport parameter in the first place to allow folks to read UDP into a fixed-size stack buffer, and this allows an endpoint to communicate the size of that buffer to its peer. If I'm able to decrypt this packet it means I'm able to read and process it, so I shouldn't be sending a lower transport parameter. I think the sentence I propose above makes it clear that this is advisory and should not be enforced.

-- 
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/3473#discussion_r382332227