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

Christer Holmberg <christer.holmberg@ericsson.com> Wed, 12 July 2017 19:39 UTC

Return-Path: <christer.holmberg@ericsson.com>
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 AC98712EC17; Wed, 12 Jul 2017 12:39:15 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -4.221
X-Spam-Level:
X-Spam-Status: No, score=-4.221 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, RCVD_IN_DNSWL_MED=-2.3, RCVD_IN_MSPIKE_H3=-0.01, RCVD_IN_MSPIKE_WL=-0.01, SPF_PASS=-0.001] autolearn=ham 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 eGTqV9ViUqxO; Wed, 12 Jul 2017 12:39:14 -0700 (PDT)
Received: from sessmg23.ericsson.net (sessmg23.ericsson.net [193.180.251.45]) (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 6CBAF12EA74; Wed, 12 Jul 2017 12:39:13 -0700 (PDT)
X-AuditID: c1b4fb2d-7ebff70000005faa-20-59667adf0bf8
Received: from ESESSHC019.ericsson.se (Unknown_Domain [153.88.183.75]) by sessmg23.ericsson.net (Symantec Mail Security) with SMTP id 7A.B0.24490.FDA76695; Wed, 12 Jul 2017 21:39:11 +0200 (CEST)
Received: from ESESSMB109.ericsson.se ([169.254.9.98]) by ESESSHC019.ericsson.se ([153.88.183.75]) with mapi id 14.03.0352.000; Wed, 12 Jul 2017 21:39:10 +0200
From: Christer Holmberg <christer.holmberg@ericsson.com>
To: Paul Kyzivat <pkyzivat@alum.mit.edu>, "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>
Thread-Topic: Gen-ART Last Call review of draft-ietf-mmusic-dtls-sdp-26
Thread-Index: AQHS910XUh6Pan5Z40qWLa0ZRshaF6JMyUmAgAAPmYCAA2ZpAIAAO2OAgAAjmYA=
Date: Wed, 12 Jul 2017 19:39:09 +0000
Message-ID: <7594FB04B1934943A5C02806D1A2204B4CC6012F@ESESSMB109.ericsson.se>
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> <edd68a99-1036-e3a1-2fc0-d05bd24bb4b5@alum.mit.edu>
In-Reply-To: <edd68a99-1036-e3a1-2fc0-d05bd24bb4b5@alum.mit.edu>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
x-originating-ip: [153.88.183.148]
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: base64
MIME-Version: 1.0
X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFmplkeLIzCtJLcpLzFFi42KZGbHdW/d+VVqkwdw7chY77u5gs7j66jOL xdTlj1ksVmw4wOrA4vH3/QcmjyVLfjIFMEVx2aSk5mSWpRbp2yVwZSw98Jix4Ix0xflF+g2M H6S6GDk5JARMJF5M2cXaxcjFISRwhFHi3r55zBDOYkaJ6ZO+sXcxcnCwCVhIdP/TBomLCDQy SjRO388M0s0sECyxd/82RhBbWMBNYs3N32wgtoiAu8SNS6dZIGw/iYd/FrKD2CwCqhJPDrwB i/MK+Er8vfGUHWJZN5PEgrdTwIo4BRwkbt1ZDTaIUUBM4vupNUwQy8Qlbj2ZzwRxtoDEkj3n mSFsUYmXj/+xQthKEo1LnrCCHM0soCmxfpc+RKuixJTuh+wQewUlTs58wjKBUXQWkqmzEDpm IemYhaRjASPLKkbR4tTi4tx0I2O91KLM5OLi/Dy9vNSSTYzAyDm45bfuDsbVrx0PMQpwMCrx 8BYUpkUKsSaWFVfmHmKU4GBWEuFVLwMK8aYkVlalFuXHF5XmpBYfYpTmYFES53XYdyFCSCA9 sSQ1OzW1ILUIJsvEwSnVwNj2QWH9hO4XvC7zfJQcBRqcNoRd/8CrcVo6fPWsjULP/x4MWfjw 2+87M27evNXDOjnubcz1hnWzhJbs7Tmnf/r9Z4HGqT9r7DJ8YzJUvu7zO67lvZZ5djD/q3vt epGRK9o/zprGu9P0+pfrfPOsnk7/M7NJM/m+1PKMtVISu39es+i7cGc/t8MjJZbijERDLeai 4kQAC7GDNJgCAAA=
Archived-At: <https://mailarchive.ietf.org/arch/msg/mmusic/LAmMGu9d--SckBzaM41OBdqltEU>
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:39:16 -0000

Thanks! :)

Regards,

Christer

-----Original Message-----
From: Paul Kyzivat [mailto:pkyzivat@alum.mit.edu] 
Sent: 12 July 2017 21:32
To: Christer Holmberg <christer.holmberg@ericsson.com>; draft-ietf-mmusic-dtls-sdp.all@ietf.org
Cc: General Area Review Team <gen-art@ietf.org>; IETF MMUSIC WG <mmusic@ietf.org>
Subject: Re: Gen-ART Last Call review of draft-ietf-mmusic-dtls-sdp-26

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
>>>
>>
>