Subject: Re: [Gen-art] [Extra] Genart telechat review of draft-ietf-extra-imap-objectid-06

From: Bron Gondwana <[email-masked]>
Date: Thu, 02 Aug 2018 23:11:46 +1000

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
  [email-masked]