Last Call Review of draft-hansen-scram-sha256-03
review-hansen-scram-sha256-03-genart-lc-sparks-2015-08-04-00
Request | Review of | draft-hansen-scram-sha256 |
---|---|---|
Requested revision | No specific revision (document currently at 04) | |
Type | Last Call Review | |
Team | General Area Review Team (Gen-ART) (genart) | |
Deadline | 2015-08-25 | |
Requested | 2015-07-30 | |
Authors | Tony Hansen | |
I-D last updated | 2015-08-04 | |
Completed reviews |
Genart Last Call review of -02
by Robert Sparks
(diff)
Genart Last Call review of -03 by Robert Sparks (diff) Genart Telechat review of -04 by Robert Sparks Secdir Last Call review of -02 by Vincent Roca (diff) Secdir Telechat review of -04 by Vincent Roca Opsdir Last Call review of -02 by Mehmet Ersue (diff) |
|
Assignment | Reviewer | Robert Sparks |
State | Completed | |
Request | Last Call review on draft-hansen-scram-sha256 by General Area Review Team (Gen-ART) Assigned | |
Reviewed revision | 03 (document currently at 04) | |
Result | Ready | |
Completed | 2015-08-04 |
review-hansen-scram-sha256-03-genart-lc-sparks-2015-08-04-00
I am the assigned Gen-ART reviewer for this draft. For background on Gen-ART, please see the FAQ at <http://wiki.tools.ietf.org/area/gen/trac/wiki/GenArtfaq>. Please resolve these comments along with any other Last Call comments you may receive. Document: draft-hansen-scram-sha256 Reviewer: Robert Sparks Review Date: 04-Aug-2015 IETF LC End Date: 25-Aug-2015 IESG Telechat date: not yet on any telechat Summary: Ready for publication as a Proposed Standard The change to PS addresses the concern I expressed in the review below. miniscule-nit: I still think the extra URI section after the references is not needed. RjS On 4/2/15 1:41 PM, Robert Sparks wrote: > I am the assigned Gen-ART reviewer for this draft. For background on > Gen-ART, please see the FAQ at > > <http://wiki.tools.ietf.org/area/gen/trac/wiki/GenArtfaq>. > > Please resolve these comments along with any other Last Call comments > you may receive. > > Document: draft-hansen-scram-sha256 > Reviewer: Robert Sparks > Review Date: 2Apr2015 > IETF LC End Date: 24Apr2015 > IESG Telechat date: (if known) > > Summary: Ready for publication as Informational, with nits that should > be considered. > > Nits/editorial comments: > > Nit: > It raises flags for me when an Informational document uses "Updates" > on a standards track document. > I would argue that this does _not_ update 5802. IANA did the things > that 5802 requested, and this document > is requesting something else that happens to change those things. That > makes this more of a "see also" than > a "the protocol changed", and I think the Updates should be removed. > > I don't feel super strongly about the difference in _this particular > case_, hence its classification as a Nit. > But for consistency, and avoiding the issue of having an Informational > update a PS, I hope you choose to remove it. > > Editorial comment: > The URLs in the references section seem superfluous since you've > already expanded them in the introduction? > > _______________________________________________ > Gen-art mailing list > Gen-art@ietf.org > https://www.ietf.org/mailman/listinfo/gen-art