Re: [MMUSIC] Opsdir last call review of draft-ietf-mmusic-rfc4566bis-34
Paul Kyzivat <pkyzivat@alum.mit.edu> Wed, 10 April 2019 20:23 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 B622C12021D; Wed, 10 Apr 2019 13:23:37 -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, SPF_PASS=-0.001, URIBL_BLOCKED=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 EY46DygoMXsR; Wed, 10 Apr 2019 13:23:36 -0700 (PDT)
Received: from outgoing-alum.mit.edu (outgoing-alum.mit.edu [18.7.68.33]) (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 E8360120165; Wed, 10 Apr 2019 13:23:35 -0700 (PDT)
Received: from PaulKyzivatsMBP.localdomain (c-24-62-227-142.hsd1.ma.comcast.net [24.62.227.142]) (authenticated bits=0) (User authenticated as pkyzivat@ALUM.MIT.EDU) by outgoing-alum.mit.edu (8.14.7/8.12.4) with ESMTP id x3AKNRgx021968 (version=TLSv1/SSLv3 cipher=AES128-SHA bits=128 verify=NOT); Wed, 10 Apr 2019 16:23:27 -0400
To: wangzitao <wangzitao@huawei.com>, "ops-dir@ietf.org" <ops-dir@ietf.org>
Cc: "ietf@ietf.org" <ietf@ietf.org>, "draft-ietf-mmusic-rfc4566bis.all@ietf.org" <draft-ietf-mmusic-rfc4566bis.all@ietf.org>, "mmusic@ietf.org" <mmusic@ietf.org>
References: <E6BC9BBCBCACC246846FC685F9FF41EA2D9A1635@DGGEMM527-MBX.china.huawei.com>
From: Paul Kyzivat <pkyzivat@alum.mit.edu>
Message-ID: <7541a94f-e539-f78b-fc92-5b0dbed22da1@alum.mit.edu>
Date: Wed, 10 Apr 2019 16:24:53 -0400
User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.10; rv:60.0) Gecko/20100101 Thunderbird/60.6.1
MIME-Version: 1.0
In-Reply-To: <E6BC9BBCBCACC246846FC685F9FF41EA2D9A1635@DGGEMM527-MBX.china.huawei.com>
Content-Type: text/plain; charset="utf-8"; format="flowed"
Content-Language: en-US
Content-Transfer-Encoding: 8bit
Archived-At: <https://mailarchive.ietf.org/arch/msg/mmusic/REgoQYgHsLxGyV7gmPUaPuTeGp8>
Subject: Re: [MMUSIC] Opsdir last call review of draft-ietf-mmusic-rfc4566bis-34
X-BeenThere: mmusic@ietf.org
X-Mailman-Version: 2.1.29
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, 10 Apr 2019 20:23:38 -0000
Please see inline. On 4/9/19 11:03 PM, wangzitao wrote: > Hi Paul, > > Please find my comments at [MW]. > > B.R. > -Michael > > -----邮件原件----- > 发件人: Paul Kyzivat [mailto:pkyzivat@alum.mit.edu] > 发送时间: 2019年4月10日 1:11 > 收件人: wangzitao <wangzitao@huawei.com>; ops-dir@ietf.org > 抄送: ietf@ietf.org; draft-ietf-mmusic-rfc4566bis.all@ietf.org; mmusic@ietf.org > 主题: Re: [MMUSIC] Opsdir last call review of draft-ietf-mmusic-rfc4566bis-34 > > Thank you for the comments. I have some questions: > > On 4/8/19 2:37 AM, Zitao Wang via Datatracker wrote: >> Reviewer: Zitao Wang >> Review result: Has Issues >> >> Summary: This memo defines the Session Description Protocol (SDP). SDP is >> intended for describing multimedia sessions for the purposes of >> session announcement, session invitation, and other forms of >> multimedia session initiation. This document obsoletes RFC 4566. I >> think the document make sense and is written very clear, except some small nits: > > >> # In Section 5, there are >> several terms that miss references, such as "US-ASCII subset of >> UTF-8", "ASN.1 or XDR", etc. > > There is already a reference to the definition of UTF-8 [RFC3629] in section 4.5. Do you think the reference needs to be included with every use? The US-ASCII subset of UTF-8 is also defined in RFC3629, so I am inclined to use the same reference for that. There are also a couple of uses of US-ASCII without mention of UTF-8. I'm inclined to change those to "the US-ASCII subset of UTF-8". I changed the naked uses of US-ASCII to reference the US-ASCII subset of UTF-8. I also added another reference to RFC3629 to the *first* use of "US-ASCII subset of UTF-8". I haven't added references to other places with the same usage. > Regarding ASN.1 and XDR, I can add references if you think it important. > But their use is very peripheral, and it isn't necessary to know what they are to read the text. > [MW]: I am OK if it is not commonly used. > > # s/session- specific/session-specific/ > > Regarding "session- specific" vs. "session-specific": > > The context for this is: > > "Attribute scopes in addition to media- and session- specific may also..." > > The space was intentional so that there are equivalent constructions for "media" and "session". The intent is as a shorthand for: > > "Attribute scopes in addition to media-specific and session-specific may also..." > > To avoid confusion I think I'll just change to the latter. > [MW]: I agree. Done. > # Suggest to add tags on >> "overview optional items" to identified now-obsolete items, such as >> "a=cat", "a=keywds", "k=". > > I'm not clear what you want me to do. > > I guess you are suggesting adding something to the first figure in section 5. > [MW]: Yes, IMO, adding some tag/description to the figure is better. > > I don't see how that would be possible for a=cat and a=keywds, since the figure doesn't mention individual attributes. > [MW]: Agree. For a=cat and a=keywds, it is difficult to show on the figure. > > While it is possible to add something for k=, IMO it is better to leave that level of detail to the complete description in section 5.12. > [MW]: Maybe it can be updated as following: > > Before : > k=* (encryption key) > After: > k=* (encryption key, this line is obsoleted) I changed these to k=* (obsolete) I have these changes in my working copy for inclusion the next time I submit a revision. Thanks, Paul
- [MMUSIC] Opsdir last call review of draft-ietf-mm… Zitao Wang via Datatracker
- Re: [MMUSIC] Opsdir last call review of draft-iet… Paul Kyzivat
- Re: [MMUSIC] Opsdir last call review of draft-iet… Adam Roach
- Re: [MMUSIC] Opsdir last call review of draft-iet… wangzitao
- Re: [MMUSIC] Opsdir last call review of draft-iet… Paul Kyzivat
- Re: [MMUSIC] Opsdir last call review of draft-iet… wangzitao