[dnsdir] Dnsdir last call review of draft-ietf-stir-passport-rcd-21

Florian Obser via Datatracker <noreply@ietf.org> Wed, 12 October 2022 07:05 UTC

Return-Path: <noreply@ietf.org>
X-Original-To: dnsdir@ietf.org
Delivered-To: dnsdir@ietfa.amsl.com
Received: from ietfa.amsl.com (localhost [IPv6:::1]) by ietfa.amsl.com (Postfix) with ESMTP id C3D86C14CF03; Wed, 12 Oct 2022 00:05:33 -0700 (PDT)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
From: Florian Obser via Datatracker <noreply@ietf.org>
To: dnsdir@ietf.org
Cc: draft-ietf-stir-passport-rcd.all@ietf.org, last-call@ietf.org, stir@ietf.org
X-Test-IDTracker: no
X-IETF-IDTracker: 8.17.0
Auto-Submitted: auto-generated
Precedence: bulk
Message-ID: <166555833379.15634.9073684363757241642@ietfa.amsl.com>
Reply-To: Florian Obser <fobser@ripe.net>
Date: Wed, 12 Oct 2022 00:05:33 -0700
Archived-At: <https://mailarchive.ietf.org/arch/msg/dnsdir/Fv-pKTtPyHLVXYDD2phBfvab9nc>
Subject: [dnsdir] Dnsdir last call review of draft-ietf-stir-passport-rcd-21
X-BeenThere: dnsdir@ietf.org
X-Mailman-Version: 2.1.39
List-Id: DNS Directorate <dnsdir.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/dnsdir>, <mailto:dnsdir-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/dnsdir/>
List-Post: <mailto:dnsdir@ietf.org>
List-Help: <mailto:dnsdir-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/dnsdir>, <mailto:dnsdir-request@ietf.org?subject=subscribe>
X-List-Received-Date: Wed, 12 Oct 2022 07:05:33 -0000

Reviewer: Florian Obser
Review result: Ready with Nits

I'm the assigned reviewer of the DNS Directorate for this draft.
This document does not reference the DNS, and has no considerations
that reflect on the DNS.

The reviewer is not familiar with PASSporT and is not in a position to
make qualified comment on the topic.
That being said, this specification to add signed and integrity
protected rich call data is clearly motivated and the proposed
extension is clearly expressed.

A list of nits follows.

Page 5:
3 times: s/passport/PASSporT/

Page 13:
s/contraint/constraint/

Page 15:

   This document defines a new JSON Web Token claim for "crn", Call
   Reason, the value of which is a single string that can contains
   information as defined in

... that can contain


Page 20
s/consisent/consistent/

Page 22
s/PASSport/PASSporT/

Page 25
s/PASSport/PASSporT/

Page 26
s/cooresponding/corresponding/

Page 29
s/availbility/availability/