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

Christer Holmberg <christer.holmberg@ericsson.com> Wed, 12 July 2017 12:58 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 5269812EC0F; Wed, 12 Jul 2017 05:58:12 -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 z60tKTJS48fa; Wed, 12 Jul 2017 05:58:06 -0700 (PDT)
Received: from sesbmg23.ericsson.net (sesbmg23.ericsson.net [193.180.251.37]) (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 CCBE01318AA; Wed, 12 Jul 2017 05:58:05 -0700 (PDT)
X-AuditID: c1b4fb25-607ff70000001eeb-4e-59661cdb6ad2
Received: from ESESSHC008.ericsson.se (Unknown_Domain [153.88.183.42]) by sesbmg23.ericsson.net (Symantec Mail Security) with SMTP id 7B.49.07915.BDC16695; Wed, 12 Jul 2017 14:58:03 +0200 (CEST)
Received: from ESESSMB109.ericsson.se ([169.254.9.98]) by ESESSHC008.ericsson.se ([153.88.183.42]) with mapi id 14.03.0352.000; Wed, 12 Jul 2017 14:58:03 +0200
From: Christer Holmberg <christer.holmberg@ericsson.com>
To: Christer Holmberg <christer.holmberg@ericsson.com>, 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: AQHS910XUh6Pan5Z40qWLa0ZRshaF6JMyUmAgAAPmYCAA2ZpAA==
Date: Wed, 12 Jul 2017 12:58:03 +0000
Message-ID: <D58BF7A0.1F340%christer.holmberg@ericsson.com>
References: <23fb3891-1d34-2e9e-42ac-99df18c0d5ed@alum.mit.edu> <D5890AA1.1F0D6%christer.holmberg@ericsson.com> <D5891B24.1F11B%christer.holmberg@ericsson.com>
In-Reply-To: <D5891B24.1F11B%christer.holmberg@ericsson.com>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
user-agent: Microsoft-MacOutlook/14.7.4.170508
x-originating-ip: [153.88.183.19]
Content-Type: text/plain; charset="utf-8"
Content-ID: <549240967A866F4B8DEE19095C5320B2@ericsson.com>
Content-Transfer-Encoding: base64
MIME-Version: 1.0
X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFlrBIsWRmVeSWpSXmKPExsUyM2K7lu5tmbRIg5az1hY77u5gs7j66jOL xdTlj1ksVmw4wOrA4vH3/QcmjyVLfjIFMEVx2aSk5mSWpRbp2yVwZax91MBe8EikYvmN+8wN jHNEuhg5OSQETCROn5rP2MXIxSEkcIRR4tTlOywgCSGBxYwS907ndzFycLAJWEh0/9MGqRER 2MooseLpdjaQGmaBYIm9+7cxgtjCAm4Ss/cdZgexRQTcJW5cOs0CYTtJTDz+lBXEZhFQlVg+ /TpYnFfAWmLr9ddMEIuBdk05+QCsmVPARmJ263Qwm1FATOL7qTVMEMvEJW49mc8EcbWAxJI9 55khbFGJl4//sYIcKiqgJ/FuvydEWFGi/WkDI0iYWUBTYv0ufYgp1hIL2layQNiKElO6H7JD nCMocXLmE5YJjOKzkCybhdA9C0n3LCTds5B0L2BkXcUoWpxanJSbbmSsl1qUmVxcnJ+nl5da sokRGH8Ht/xW3cF4+Y3jIUYBDkYlHt7+L6mRQqyJZcWVuYcYJTiYlUR4fYXTIoV4UxIrq1KL 8uOLSnNSiw8xSnOwKInzOu67ECEkkJ5YkpqdmlqQWgSTZeLglGpgDIsW9Xi5XLiWZVveGzUB j4exLuZGE2d/lOyLSvXl3eAldjwufcdK3t8L9s773xDmfmlO1NxJzP8V8x3zNlSIzdx1ufPe DqtV1Q1PvCSWbeN7u/gX15+ZPzUncjEss010Pdk8afPUCWcmz1p3mC3vxJHLbhJZz6adtpu5 le3N7bmPTrF9vCX4TUiJpTgj0VCLuag4EQChOdG6uwIAAA==
Archived-At: <https://mailarchive.ietf.org/arch/msg/mmusic/BqLKbz3bVUZh2j6jZKsv53OUFg4>
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 12:58:12 -0000

Hi Paul,

Are you ok with my reply?

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