[Int-dir] Intdir telechat review of draft-ietf-sfc-nsh-tlv-09

Bob Halley via Datatracker <noreply@ietf.org> Tue, 09 November 2021 22:47 UTC

Return-Path: <noreply@ietf.org>
X-Original-To: int-dir@ietf.org
Delivered-To: int-dir@ietfa.amsl.com
Received: from ietfa.amsl.com (localhost [IPv6:::1]) by ietfa.amsl.com (Postfix) with ESMTP id 8AB9D3A046A; Tue, 9 Nov 2021 14:47:01 -0800 (PST)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
From: Bob Halley via Datatracker <noreply@ietf.org>
To: int-dir@ietf.org
Cc: draft-ietf-sfc-nsh-tlv.all@ietf.org, last-call@ietf.org, sfc@ietf.org
X-Test-IDTracker: no
X-IETF-IDTracker: 7.39.0
Auto-Submitted: auto-generated
Precedence: bulk
Message-ID: <163649802149.2765.2534157306702225276@ietfa.amsl.com>
Reply-To: Bob Halley <rthalley@gmail.com>
Date: Tue, 09 Nov 2021 14:47:01 -0800
Archived-At: <https://mailarchive.ietf.org/arch/msg/int-dir/b4Nrc3z9dCPofR7BY9x2oFsOSus>
Subject: [Int-dir] Intdir telechat review of draft-ietf-sfc-nsh-tlv-09
X-BeenThere: int-dir@ietf.org
X-Mailman-Version: 2.1.29
List-Id: "This list is for discussion between the members of the Internet Area directorate." <int-dir.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/int-dir>, <mailto:int-dir-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/int-dir/>
List-Post: <mailto:int-dir@ietf.org>
List-Help: <mailto:int-dir-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/int-dir>, <mailto:int-dir-request@ietf.org?subject=subscribe>
X-List-Received-Date: Tue, 09 Nov 2021 22:47:02 -0000

Reviewer: Bob Halley
Review result: Ready

I am an assigned INT directorate reviewer for draft-ietf-sfc-nsh-tlv-09. These
comments were written primarily for the benefit of the Internet Area Directors.
Document editors and shepherd(s) should treat these comments just like they
would treat comments from any other IETF contributors and resolve them along
with any other Last Call comments that have been received. For more details on
the INT Directorate, see https://datatracker.ietf.org/group/intdir/about/
<https://datatracker.ietf.org/group/intdir/about/>.

Based on my review, if I was on the IESG I would ballot this document as YES.

The following are other issues I found with this document that SHOULD be
corrected before publication:  none

The following are minor issues (typos, misspelling, minor text improvements)
with the document: none

I found the document to be well organized and the metadata contexts to be well
specified.  I agree with the SECDIR last call review by Charlie Kaufman that it
would have been nice to be specific about what to do when bad length values
occur, but am not worried about it as I'd expect an implementation to do
appropriate checking, including bounds checking, and then do something
reasonable (i.e. ignore or reject the packet) if there is a problem.