[Taps] Secdir telechat review of draft-ietf-taps-interface-22

Sean Turner via Datatracker <noreply@ietf.org> Tue, 10 October 2023 14:39 UTC

Return-Path: <noreply@ietf.org>
X-Original-To: taps@ietf.org
Delivered-To: taps@ietfa.amsl.com
Received: from ietfa.amsl.com (localhost [IPv6:::1]) by ietfa.amsl.com (Postfix) with ESMTP id 595D1C151984; Tue, 10 Oct 2023 07:39:45 -0700 (PDT)
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: draft-ietf-taps-interface.all@ietf.org, last-call@ietf.org, taps@ietf.org
X-Test-IDTracker: no
X-IETF-IDTracker: 11.12.1
Auto-Submitted: auto-generated
Precedence: bulk
Message-ID: <169694878534.56161.9894888404515834877@ietfa.amsl.com>
Reply-To: Sean Turner <sean@sn3rd.com>
Date: Tue, 10 Oct 2023 07:39:45 -0700
Archived-At: <https://mailarchive.ietf.org/arch/msg/taps/bEwlAm4OKmzz6xcPQubWzeiiQdA>
Subject: [Taps] Secdir telechat review of draft-ietf-taps-interface-22
X-BeenThere: taps@ietf.org
X-Mailman-Version: 2.1.39
List-Id: "IETF Transport Services \(TAPS\) Working Group" <taps.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/taps>, <mailto:taps-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/taps/>
List-Post: <mailto:taps@ietf.org>
List-Help: <mailto:taps-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/taps>, <mailto:taps-request@ietf.org?subject=subscribe>
X-List-Received-Date: Tue, 10 Oct 2023 14:39:45 -0000

Reviewer: Sean Turner
Review result: Ready

Hi! I've reviewed this I-D a couple of times along the way. This time I just
looked at the diffs between -20 and -22.

tl;dr: It's ready.

The security related changes this time around were pretty minor:

1. Downgrading TLS to informative. Sounds inflammatory doesn't it, but it's not
;) Something has to be implemented under the API, but it doesn't have to be TLS.

2. Dropped zeroRttMsgMaxLen read-only property. Turns out it wasn't used and
didn't offer much utility anyway:
https://github.com/ietf-tapswg/api-drafts/pull/1173