Re: [Sipbrandy] Alissa Cooper's Yes on draft-ietf-sipbrandy-rtpsec-07: (with COMMENT)

Russ Housley <housley@vigilsec.com> Sun, 10 March 2019 11:29 UTC

Return-Path: <housley@vigilsec.com>
X-Original-To: sipbrandy@ietfa.amsl.com
Delivered-To: sipbrandy@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id BB098124D68 for <sipbrandy@ietfa.amsl.com>; Sun, 10 Mar 2019 04:29:57 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.9
X-Spam-Level:
X-Spam-Status: No, score=-1.9 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, RCVD_IN_DNSWL_NONE=-0.0001] autolearn=unavailable autolearn_force=no
Received: from mail.ietf.org ([4.31.198.44]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id EsnhyVTmVurd for <sipbrandy@ietfa.amsl.com>; Sun, 10 Mar 2019 04:29:55 -0700 (PDT)
Received: from mail.smeinc.net (mail.smeinc.net [209.135.209.11]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 8C9781200ED for <sipbrandy@ietf.org>; Sun, 10 Mar 2019 04:29:55 -0700 (PDT)
Received: from localhost (localhost [127.0.0.1]) by mail.smeinc.net (Postfix) with ESMTP id 7F6A1300AA5 for <sipbrandy@ietf.org>; Sun, 10 Mar 2019 07:11:37 -0400 (EDT)
X-Virus-Scanned: amavisd-new at mail.smeinc.net
Received: from mail.smeinc.net ([127.0.0.1]) by localhost (mail.smeinc.net [127.0.0.1]) (amavisd-new, port 10026) with ESMTP id 4pq_2R72XCtr for <sipbrandy@ietf.org>; Sun, 10 Mar 2019 07:11:36 -0400 (EDT)
Received: from [10.196.217.15] (3-197.icannmeeting.org [199.91.197.3]) by mail.smeinc.net (Postfix) with ESMTPSA id 3DBBB300064; Sun, 10 Mar 2019 07:11:33 -0400 (EDT)
Content-Type: text/plain; charset="us-ascii"
Mime-Version: 1.0 (Mac OS X Mail 12.2 \(3445.102.3\))
From: Russ Housley <housley@vigilsec.com>
In-Reply-To: <155190914064.14212.1784537858104594420.idtracker@ietfa.amsl.com>
Date: Sun, 10 Mar 2019 07:29:48 -0400
Cc: IESG <iesg@ietf.org>, draft-ietf-sipbrandy-rtpsec@ietf.org, sipbrandy@ietf.org, Gonzalo Camarillo <gonzalo.camarillo@ericsson.com>
Content-Transfer-Encoding: quoted-printable
Message-Id: <6C1A0B14-8494-4834-9AC1-04DF119A851C@vigilsec.com>
References: <155190914064.14212.1784537858104594420.idtracker@ietfa.amsl.com>
To: Alissa Cooper <alissa@cooperw.in>
X-Mailer: Apple Mail (2.3445.102.3)
Archived-At: <https://mailarchive.ietf.org/arch/msg/sipbrandy/wqDLbf1GrXQHC0mvlmT0ucZJpaQ>
Subject: Re: [Sipbrandy] Alissa Cooper's Yes on draft-ietf-sipbrandy-rtpsec-07: (with COMMENT)
X-BeenThere: sipbrandy@ietf.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: SIPBRANDY working group discussion list <sipbrandy.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/sipbrandy>, <mailto:sipbrandy-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/sipbrandy/>
List-Post: <mailto:sipbrandy@ietf.org>
List-Help: <mailto:sipbrandy-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/sipbrandy>, <mailto:sipbrandy-request@ietf.org?subject=subscribe>
X-List-Received-Date: Sun, 10 Mar 2019 11:29:58 -0000

Alissa:

> ----------------------------------------------------------------------
> COMMENT:
> ----------------------------------------------------------------------
> 
> Very glad to see this work coming to a close.
> 
> = Section 3.2 =
> 
> OLD
> Work is already underway on defining approaches to opportunistic media security
> for SIP in [I-D.johnston-dispatch-osrtp]
> 
> NEW
> At the time of this writing, work was underway to define approaches to
> opportunistic media security for SIP in [draft-ietf-sipbrandy-osrtp]
> 
> = Section 4 =
> 
> s/at the endpoint obtain/at the endpoint to obtain/
> 
> = Section 5 =
> 
> Please update the OSRTP citation to point to draft-ietf-sipbrandy-osrtp.
> 
> = Section 10 =
> 
> Presumably "PASSporT Type registry" should say "PASSporT Resource Priority
> Header (rph) Types registry."

I believe that the "msec" entry should be added to the Personal Assertion Token (PASSporT) Extensions list, not the rph list.

I made all of the other changes in my edit buffer.  Comments from other ADs need to be resolved before an update can be posted.

Russ