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

[Bron Gondwana <brong@xxxxxxxxxxxxxxxx>]

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

  brong@xxxxxxxxxxxxxxxx