[secdir] Secdir last call review of draft-ietf-nfsv4-rfc5661sesqui-msns-03

Sean Turner via Datatracker <noreply@ietf.org> Mon, 09 December 2019 19:37 UTC

Return-Path: <noreply@ietf.org>
X-Original-To: secdir@ietf.org
Delivered-To: secdir@ietfa.amsl.com
Received: from ietfa.amsl.com (localhost [IPv6:::1]) by ietfa.amsl.com (Postfix) with ESMTP id B1DD712012A; Mon, 9 Dec 2019 11:37:16 -0800 (PST)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
From: Sean Turner via Datatracker <noreply@ietf.org>
To: <secdir@ietf.org>
Cc: last-call@ietf.org, nfsv4@ietf.org, draft-ietf-nfsv4-rfc5661sesqui-msns.all@ietf.org
X-Test-IDTracker: no
X-IETF-IDTracker: 6.113.0
Auto-Submitted: auto-generated
Precedence: bulk
Reply-To: Sean Turner <sean@sn3rd.com>
Message-ID: <157592023668.2110.4367333458987159995@ietfa.amsl.com>
Date: Mon, 09 Dec 2019 11:37:16 -0800
Archived-At: <https://mailarchive.ietf.org/arch/msg/secdir/vqPpJS7g8OT5LQcs3MIdRzuLmf0>
Subject: [secdir] Secdir last call review of draft-ietf-nfsv4-rfc5661sesqui-msns-03
X-BeenThere: secdir@ietf.org
X-Mailman-Version: 2.1.29
List-Id: Security Area Directorate <secdir.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/secdir>, <mailto:secdir-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/secdir/>
List-Post: <mailto:secdir@ietf.org>
List-Help: <mailto:secdir-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/secdir>, <mailto:secdir-request@ietf.org?subject=subscribe>
X-List-Received-Date: Mon, 09 Dec 2019 19:37:17 -0000

Reviewer: Sean Turner
Review result: Ready

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.



[1] https://mailarchive.ietf.org/arch/msg/secdir/zjgQKz0KnQbQHrJmQGwWioNwEpg