Re: [MMUSIC] Gen-ART Last Call review of draft-ietf-mmusic-dtls-sdp-26

Paul Kyzivat <pkyzivat@alum.mit.edu> Wed, 12 July 2017 19:31 UTC

Return-Path: <pkyzivat@alum.mit.edu>
X-Original-To: mmusic@ietfa.amsl.com
Delivered-To: mmusic@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 7526F131766 for <mmusic@ietfa.amsl.com>; Wed, 12 Jul 2017 12:31:36 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.935
X-Spam-Level:
X-Spam-Status: No, score=-1.935 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, RCVD_IN_DNSWL_LOW=-0.7, SPF_SOFTFAIL=0.665] autolearn=no 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 hqG2-QkDgIIt for <mmusic@ietfa.amsl.com>; Wed, 12 Jul 2017 12:31:35 -0700 (PDT)
Received: from resqmta-ch2-11v.sys.comcast.net (resqmta-ch2-11v.sys.comcast.net [IPv6:2001:558:fe21:29:69:252:207:43]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 6119C13178C for <mmusic@ietf.org>; Wed, 12 Jul 2017 12:31:35 -0700 (PDT)
Received: from resomta-ch2-09v.sys.comcast.net ([69.252.207.105]) by resqmta-ch2-11v.sys.comcast.net with ESMTP id VNLkdSLXTyXIHVNMEdS3tO; Wed, 12 Jul 2017 19:31:34 +0000
Received: from [192.168.1.110] ([24.62.227.142]) by resomta-ch2-09v.sys.comcast.net with SMTP id VNMDdbr42h8eeVNMEdDLNU; Wed, 12 Jul 2017 19:31:34 +0000
To: Christer Holmberg <christer.holmberg@ericsson.com>, "draft-ietf-mmusic-dtls-sdp.all@ietf.org" <draft-ietf-mmusic-dtls-sdp.all@ietf.org>
Cc: General Area Review Team <gen-art@ietf.org>, IETF MMUSIC WG <mmusic@ietf.org>
References: <23fb3891-1d34-2e9e-42ac-99df18c0d5ed@alum.mit.edu> <D5890AA1.1F0D6%christer.holmberg@ericsson.com> <D5891B24.1F11B%christer.holmberg@ericsson.com> <D58BF7A0.1F340%christer.holmberg@ericsson.com>
From: Paul Kyzivat <pkyzivat@alum.mit.edu>
Message-ID: <edd68a99-1036-e3a1-2fc0-d05bd24bb4b5@alum.mit.edu>
Date: Wed, 12 Jul 2017 15:31:33 -0400
User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.10; rv:52.0) Gecko/20100101 Thunderbird/52.2.1
MIME-Version: 1.0
In-Reply-To: <D58BF7A0.1F340%christer.holmberg@ericsson.com>
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Language: en-US
Content-Transfer-Encoding: 8bit
X-CMAE-Envelope: MS4wfIHRKjoIsKPle0FgjpkzBAB4KbE/FoFeMmiNx1BShndpGBgk6OfJsedJ+sm5ukcRejYDeoCgnrc9yHpr+t6a1rZIdfOxh3+0ZBuv+OknRDUz/qKYoR8h ijVJI34ur78b5DOi9Tjl02aHbdUiIrDER91CRvgy0RkBCO1lmhNJxnRY99xrSl3K/ytYGCYxr24vGu/6y/xm0FBF3crt5n2C6uA03BZ02Z6kOzO4/+TDkQ15 U6+kSU1PLBx/BjXk5oD9FuGepWifgZ3G2wA9jCJPydFER9FR2x6MMfkOha54RPB0
Archived-At: <https://mailarchive.ietf.org/arch/msg/mmusic/rtaSpfCvzDtCjTTxDwSw8TTFY3c>
Subject: Re: [MMUSIC] Gen-ART Last Call review of draft-ietf-mmusic-dtls-sdp-26
X-BeenThere: mmusic@ietf.org
X-Mailman-Version: 2.1.22
Precedence: list
List-Id: Multiparty Multimedia Session Control Working Group <mmusic.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/mmusic>, <mailto:mmusic-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/mmusic/>
List-Post: <mailto:mmusic@ietf.org>
List-Help: <mailto:mmusic-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/mmusic>, <mailto:mmusic-request@ietf.org?subject=subscribe>
X-List-Received-Date: Wed, 12 Jul 2017 19:31:36 -0000

On 7/12/17 8:58 AM, Christer Holmberg wrote:
> Hi Paul,
> 
> Are you ok with my reply?

Yes.

	Thanks,
	Paul

> Regards,
> 
> Christer
> 
> 
> 
> On 10/07/17 12:02, "Christer Holmberg" <christer.holmberg@ericsson.com>
> wrote:
> 
>> Pull request: https://github.com/cdh4u/draft-dtls-sdp/pull/33
>>
>> (I kept the example in Section 8, but I fixed the reference)
>>
>> Regards,
>>
>> Christer
>>
>>
>> On 10/07/17 11:08, "Christer Holmberg" <christer.holmberg@ericsson.com>
>> wrote:
>>
>>> Hi Paul,
>>>
>>> Thanks for your review! Please see inline.
>>>
>>>
>>>> I am the assigned Gen-ART reviewer for this draft. The General Area
>>>> Review Team (Gen-ART) reviews all IETF documents being processed by the
>>>> IESG for the IETF Chair. Please treat these comments just like any other
>>>> last call comments. For more information, please see the FAQ at
>>>> <​http://wiki.tools.ietf.org/area/gen/trac/wiki/GenArtfaq>.
>>>>
>>>> Document: draft-ietf-mmusic-dtls-sdp-26
>>>> Reviewer: Paul Kyzivat
>>>> Review Date: 2017-07-07
>>>> IETF LC End Date: 2017-07-24
>>>> IESG Telechat date: TBD
>>>>
>>>> Summary:
>>>>
>>>> This draft is basically ready for publication, but has a few nits that
>>>> should be fixed before publication.
>>>>
>>>> Issues:
>>>>
>>>> Major: 0
>>>> Minor: 0
>>>> Nits:  4
>>>>
>>>> (1) NIT:
>>>>
>>>> Section 5.3: s/Eventhough/Even though/
>>>
>>> Will be fixed.
>>>
>>>> (2) NIT:
>>>>
>>>> Section 8: s/aTLS/a TLS/
>>>
>>> Will be fixed.
>>>
>>>> (3) NIT:
>>>>
>>>> Section 8: What is the point of including the example? I don't see how
>>>> it adds anything. Perhaps worked out O/A examples contrasting the
>>>> differences between the new and existing cases might be marginally
>>>> helpful. (But IMO not enough to bother with.)
>>>
>>> The idea was to take the existing example from RFC 4572 (I note the
>>> reference is wrong: s/3261/4572), and show how it looks with the tls-id
>>> attribute.
>>>
>>>
>>>> (4) NIT:
>>>>
>>>> Section 10.3.2: s/Througout/throughout/
>>>
>>> Will be fixed.
>>>
>>> Regards,
>>>
>>> Christer
>>>
>>
>