IETF Logo Mail Archive

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

Christer Holmberg <christer.holmberg@ericsson.com> Mon, 10 July 2017 08:08 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 B22521275AB; Mon, 10 Jul 2017 01:08:32 -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 VGPINilPIaYE; Mon, 10 Jul 2017 01:08:31 -0700 (PDT)
Received: from sessmg22.ericsson.net (sessmg22.ericsson.net [193.180.251.58]) (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 54FF5126C3D; Mon, 10 Jul 2017 01:08:30 -0700 (PDT)
X-AuditID: c1b4fb3a-bea2a9c000001b2f-be-596335fcc238
Received: from ESESSHC010.ericsson.se (Unknown_Domain [153.88.183.48]) by sessmg22.ericsson.net (Symantec Mail Security) with SMTP id 0D.D0.06959.CF533695; Mon, 10 Jul 2017 10:08:28 +0200 (CEST)
Received: from ESESSMB109.ericsson.se ([169.254.9.98]) by ESESSHC010.ericsson.se ([153.88.183.48]) with mapi id 14.03.0352.000; Mon, 10 Jul 2017 10:08:24 +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: AQHS910XUh6Pan5Z40qWLa0ZRshaF6JMyUmA
Date: Mon, 10 Jul 2017 08:08:24 +0000
Message-ID: <D5890AA1.1F0D6%christer.holmberg@ericsson.com>
References: <23fb3891-1d34-2e9e-42ac-99df18c0d5ed@alum.mit.edu>
In-Reply-To: <23fb3891-1d34-2e9e-42ac-99df18c0d5ed@alum.mit.edu>
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.146]
Content-Type: text/plain; charset="utf-8"
Content-ID: <B0836CD605AC7E4E894A2E161B8F9F06@ericsson.com>
Content-Transfer-Encoding: base64
MIME-Version: 1.0
X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFlrBIsWRmVeSWpSXmKPExsUyM2K7ge4f0+RIg+2dWhY77u5gs7j66jOL xdTlj1ksVmw4wOrA4vH3/QcmjyVLfjIFMEVx2aSk5mSWpRbp2yVwZfyd9Y+1oIu/4sq5VSwN jDf4uhg5OSQETCSmLetn6mLk4hASOMIo0XpjCzOEs5hRouF4I1CGg4NNwEKi+582SFxEoJFR onH6fmaQbmaBYIm9+7cxgtjCAm4Ss/cdZgexRQTcJW5cOs0CYRtJTH+1hg3EZhFQlVj9eh9Y L6+AtcSdl1vBbCEBe4lfh/eCzeEUcJCYvPYmE4jNKCAm8f3UGiaIXeISt57MZ4K4WkBiyZ7z zBC2qMTLx/9YQe4UFdCTeLffEyKsJPFjwyUWkDCzgKbE+l36EFOsJV683cgOYStKTOl+yA5x jaDEyZlPWCYwis9CsmwWQvcsJN2zkHTPQtK9gJF1FaNocWpxcW66kZFealFmcnFxfp5eXmrJ JkZg/B3c8ttqB+PB546HGAU4GJV4eHnlkyOFWBPLiitzDzFKcDArifDONAIK8aYkVlalFuXH F5XmpBYfYpTmYFES53XYdyFCSCA9sSQ1OzW1ILUIJsvEwSnVwMgyf6LGgk9m/qsf/v22aqZP 2gznmmr/DbGaRt/6hC7N+yu9MrdZrNkihnXNG9Y1mvz5QqcX7e6bwWHqdcpxVeete2t0QkWE PhZ/kX42N1nI0PZc4zuXZJWPDycoGxXv4TE7MlG+IV+jdfo2F4YYxy8x3K9fL8o9eejCG5mr 134tLlp/+F5FI7cSS3FGoqEWc1FxIgAMrntkuwIAAA==
Archived-At: <https://mailarchive.ietf.org/arch/msg/mmusic/gpLeIJWeHvjAnSAyoRko7sP36aw>
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: Mon, 10 Jul 2017 08:08:32 -0000

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

v2.23.0 | Report a Bug | By Email