On Fri, Oct 9, 2020 at 8:38 AM Benno Overeinder <be...@nlnetlabs.nl> wrote:
> This starts a Working Group Last Call for draft-ietf-dnsop-server-cookies. > > Current versions of the draft is available here: > https://datatracker.ietf.org/doc/draft-ietf-dnsop-server-cookies/ > > The Current Intended Status of this document is: Standards Track > > FYI, I will not shepherd this document, as it was written with one of my > coworkers. > Tim Wicinski will be Document Shepherd. > > Please review the draft and offer relevant comments. > If this does not seem appropriate please speak out. > If someone feels the document is *not* ready for publication, please speak > out with your reasons. > I have read the document, and support publication (modulo very minor nits that should be fixed). In addition to these nits, I do have one further suggestion for Section 8. I'm not sure if it is too late to make such a suggestion, but on reading (and thinking about) the spec, it could be useful guidance (particularly for clients which may not be aware of changes to their Client-IP address): "o In order to determine that a Server has detected a change to the Client-IP, a Client may consider a BADCOOKIE error sooner than would be expected from a Server Cookie refresh as a signal that the Client-IP may have changed, and thus that a new Client Cookie should be created for each Server." Nits: Introduction - I believe "provides" should be "provide", to agree with the singular "is" of the verb. (Sorry, grammar nit.) Section 1.1 - I believe all the "Section Section" instances should really just be "Section". Section 4 - "too frequent" -> "too frequently". Section 4.3 - "in the anycast." -> "in the anycast set." Section 4.4 - hash calculation, end of first line "Client-IP," -> "Client-IP |" Section 5 - "anycast group" -> "anycast set"; "us used" -> "is used" Section 8 - "like for example five minute." -> "for example five minutes." Brian
_______________________________________________ DNSOP mailing list DNSOP@ietf.org https://www.ietf.org/mailman/listinfo/dnsop
