Hi Rob, The SHOULD from your point (1) is there to address Daniel's concern about IoT. The SHOULD from (2) is indeed not required for interoperability, but important to ensure servers put this protection in place.
That said, I've taken your suggestion to add a comma since that is clearer. Thanks! David On Thu, Nov 21, 2019 at 2:18 PM Rob Sayre <[email protected]> wrote: > So, the current PR says: > > "Clients can use TicketRequestContents.count to indicate the number of > tickets they would prefer to receive. Servers SHOULD NOT send more tickets > than TicketRequestContents.count, as clients will most likely discard any > additional tickets. Servers SHOULD additionally place a limit on the number > of tickets they are willing to send to save resources. Therefore, the > number of NewSessionTicket messages sent will be the minimum of the > server's self-imposed limit and TicketRequestContents.count." > > I would suggest two edits to this text: > > 1: The second sentence can read: "If servers send more tickets than > TicketRequestContents.count, clients will likely discard excess tickets.". > This avoids a "SHOULD NOT" that should be a "MAY" or unmentioned. > > 2: The third sentence is implementation advice, not an interoperability > requirement: "Servers SHOULD additionally place a limit on the number of > tickets they are willing to send to save resources." I think this sentence > should be struck, or rewritten in non-2119 English. If the editors wish to > keep the text, I think there should be a comma after "send". > > These proposed edits make sense to me, because servers can't know if their > tickets will be used successfully (network problems, unreliable clients, > etc). > > thanks, > Rob > > > On Wed, Nov 20, 2019 at 9:45 PM David Schinazi <[email protected]> > wrote: > >> Thanks. I've updated the PR to take MT's suggestion s/SHOULD/will/. >> >> David >> >> On Thu, Nov 21, 2019 at 1:38 PM Martin Thomson <[email protected]> wrote: >> >>> On Thu, Nov 21, 2019, at 11:19, David Schinazi wrote: >>> > resources. Therefore, the number of NewSessionTicket messages sent >>> > SHOULD be the minimum of the server's self-imposed limit and >>> > TicketRequestContents.count. >>> >>> Thanks for doing this David. >>> >>> Friendly amendment: remove the SHOULD from this sentence (it's a >>> consequence of the two other "SHOULD"s): >>> >>> Therefore, the number of NewSessionTicket messages sent will be the >>> minimum [...] >>> >>> _______________________________________________ >>> TLS mailing list >>> [email protected] >>> https://www.ietf.org/mailman/listinfo/tls >>> >> _______________________________________________ >> TLS mailing list >> [email protected] >> https://www.ietf.org/mailman/listinfo/tls >> >
_______________________________________________ TLS mailing list [email protected] https://www.ietf.org/mailman/listinfo/tls
