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 <dschinazi.i...@gmail.com> 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 <m...@lowentropy.net> 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 >> TLS@ietf.org >> https://www.ietf.org/mailman/listinfo/tls >> > _______________________________________________ > TLS mailing list > TLS@ietf.org > https://www.ietf.org/mailman/listinfo/tls >
_______________________________________________ TLS mailing list TLS@ietf.org https://www.ietf.org/mailman/listinfo/tls
