Skip to main content

Last Call Review of draft-ietf-nfsv4-rfc5661sesqui-msns-03
review-ietf-nfsv4-rfc5661sesqui-msns-03-secdir-lc-turner-2019-12-09-00

Request Review of draft-ietf-nfsv4-rfc5661sesqui-msns
Requested revision No specific revision (document currently at 04)
Type Last Call Review
Team Security Area Directorate (secdir)
Deadline 2019-11-25
Requested 2019-11-04
Authors David Noveck , Chuck Lever
I-D last updated 2019-12-09
Completed reviews Secdir Last Call review of -03 by Sean Turner (diff)
Genart Last Call review of -03 by Vijay K. Gurbani (diff)
Assignment Reviewer Sean Turner
State Completed
Request Last Call review on draft-ietf-nfsv4-rfc5661sesqui-msns by Security Area Directorate Assigned
Posted at https://mailarchive.ietf.org/arch/msg/secdir/vqPpJS7g8OT5LQcs3MIdRzuLmf0
Reviewed revision 03 (document currently at 04)
Result Ready
Completed 2019-12-09
review-ietf-nfsv4-rfc5661sesqui-msns-03-secdir-lc-turner-2019-12-09-00
I'm no NFS expert, but I did review draft-ietf-nfsv4-mv1-msns-update [0], which
dealt with fixes in RFC5661 for multi-server namespace.  This draft replaces
that draft and obsoletes RFC5661 to incorporate changes documented in
draft-ietf-nfsv4-mv1-msns-update.  Almost all of the changes are to section 11
and 21.  I briefly examined the changes in this draft and those noted in
draft-ietf-nfsv4-mv1-msns-update, and it appears the changes are incorporated
(I trust the authors faithfully made the changes).

As far as the changes I suggested during my previous review [1], it appears as
if the authors have:

- major: 0-2 accepted and incorporated.  The spirit of 3 was incorporated;
makes it clear that AUTH_SYS provides "no provision to avoid modification of
data in flight".

- nits: incorporated 0-4, 6-8, and 10.  I see the point to not making the
changes suggested in 5 and 9.

Cheers,

spt

[0]
https://datatracker.ietf.org/doc/review-ietf-nfsv4-mv1-msns-update-04-secdir-lc-turner-2019-02-27/
[1] https://mailarchive.ietf.org/arch/msg/secdir/zjgQKz0KnQbQHrJmQGwWioNwEpg