Telechat Review of draft-ietf-extra-imap-objectid-06
review-ietf-extra-imap-objectid-06-genart-telechat-resnick-2018-07-27-00
Request | Review of | draft-ietf-extra-imap-objectid |
---|---|---|
Requested revision | No specific revision (document currently at 08) | |
Type | Telechat Review | |
Team | General Area Review Team (Gen-ART) (genart) | |
Deadline | 2018-07-31 | |
Requested | 2018-07-18 | |
Authors | Bron Gondwana | |
I-D last updated | 2018-07-27 | |
Completed reviews |
Secdir Last Call review of -04
by Chris M. Lonvick
(diff)
Genart Last Call review of -03 by Pete Resnick (diff) Genart Telechat review of -06 by Pete Resnick (diff) |
|
Assignment | Reviewer | Pete Resnick |
State | Completed | |
Request | Telechat review on draft-ietf-extra-imap-objectid by General Area Review Team (Gen-ART) Assigned | |
Reviewed revision | 06 (document currently at 08) | |
Result | Ready w/nits | |
Completed | 2018-07-27 |
review-ietf-extra-imap-objectid-06-genart-telechat-resnick-2018-07-27-00
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 §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. §8.2 ¶2: s/backend object collide/backend object identifiers