On Sat, Jul 28, 2018, at 03:57, Pete Resnick wrote: > Reviewer: Pete Resnick > Review result: Ready with Nits > > I am the assigned Gen-ART reviewer for this draft. The General Area > Review Team (Gen-ART) reviews all IETF documents being processed > by the IESG for the IETF Chair. Please wait for direction from your > document shepherd or AD before posting a new version of the draft. > > For more information, please see the FAQ at > > <https://trac.ietf.org/trac/gen/wiki/GenArtfaq>. > > Document: draft-ietf-extra-imap-objectid-06 > Reviewer: Pete Resnick > Review Date: 2018-07-27 > IETF LC End Date: 2018-07-13 > IESG Telechat date: 2018-08-02 > > Summary: Ready with Nits > > Major issues: None > > Minor issues: None > > Nits/editorial comments: > > Thanks for the changes responding to my review. Good work. > > §5.2, ¶6: > > OLD > THREADID is optional, if the server doesn't support THREADID or is > NEW > THREADID is OPTIONAL; if the server doesn't support THREADID or is
Yep, fixed. > §5.2 ¶7: > > Not clear to me why the THREADID and EMAILID can't be the same. I > assume given> the MUST it's going to be some sort of interoperability problem, > but it does> seem odd. But don't change it on my account. This is basically a "developers will make assumptions about relatedness which may not always hold". We even XOR our THREADID against a constant after calculating it from the EMAILID in our server now, so that developers don't get lazy and make shortcuts. > §8.2 ¶2: > > s/backend object collide/backend object identifiers Fixed this too. Thanks again for great feedback, Bron. -- Bron Gondwana, CEO, FastMail Pty Ltd br...@fastmailteam.com
_______________________________________________ Gen-art mailing list Gen-art@ietf.org https://www.ietf.org/mailman/listinfo/gen-art