Re: [MMUSIC] Fwd: New Version Notification for draft-ietf-mmusic-rfc4566bis-19.txt

Paul Kyzivat <paul.kyzivat@comcast.net> Mon, 19 June 2017 15:02 UTC

Return-Path: <paul.kyzivat@comcast.net>
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 59710131526 for <mmusic@ietfa.amsl.com>; Mon, 19 Jun 2017 08:02:26 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.7
X-Spam-Level:
X-Spam-Status: No, score=-2.7 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, FREEMAIL_FROM=0.001, RCVD_IN_DNSWL_LOW=-0.7, RP_MATCHES_RCVD=-0.001, SPF_PASS=-0.001, URIBL_BLOCKED=0.001] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (2048-bit key) header.d=comcast.net
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 dhN-GXXuk00M for <mmusic@ietfa.amsl.com>; Mon, 19 Jun 2017 08:02:24 -0700 (PDT)
Received: from resqmta-ch2-09v.sys.comcast.net (resqmta-ch2-09v.sys.comcast.net [IPv6:2001:558:fe21:29:69:252:207:41]) (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 A4D6E13151B for <mmusic@ietf.org>; Mon, 19 Jun 2017 08:02:18 -0700 (PDT)
Received: from resomta-ch2-17v.sys.comcast.net ([69.252.207.113]) by resqmta-ch2-09v.sys.comcast.net with SMTP id MyBBdPafVQe9cMyC1dOtXC; Mon, 19 Jun 2017 15:02:17 +0000
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=comcast.net; s=q20161114; t=1497884537; bh=vYCW2IFRKOsDDbKDzExMsPke3MCbDFdK7Wc7E6rCOzo=; h=Received:Received:Subject:To:From:Message-ID:Date:MIME-Version: Content-Type; b=hvF5oq076xBAXn63Nsh9xR0R2GFIX8+j6KvYVI5lujAg5sG6Zp1ChjqhKfxCE2mrB pU5Fdqlwk5dS5g5cqXHzLvfrCyR/vBvRVjGBWrUFFM91b3A4yXxZUY8gw7NmSVSRRV V0r7mN8klvak3uflKwaEbf79ymwqV7Dr3NVsUMhmrIBmmhXD/qokygtwbV0mQR7lbe B872/88fTEADat2EfGAAvhRdM7eLVvcLb2kl3c9zo8PDlYSCxeeHDasRpMVx+V8YvI 9qF2aQTfQQRoiIhQomkseGtjvhOEvvJ+Snjvqy+xwK7hudR4w0UZp4tI4kKj6I5vmq TP1w4Df8ML7Ew==
Received: from [192.168.1.110] ([24.62.227.142]) by resomta-ch2-17v.sys.comcast.net with SMTP id MyC1dfPby8hkfMyC1dQxzl; Mon, 19 Jun 2017 15:02:17 +0000
To: mmusic@ietf.org
References: <149765179618.24205.7010922835529746440.idtracker@ietfa.amsl.com> <CAA4Mczvi-+sCYHO9NO48wE8o6YTvZ-a216d1L_yLvgwFS9O3zw@mail.gmail.com>
From: Paul Kyzivat <paul.kyzivat@comcast.net>
Message-ID: <aa23c37c-7e6a-fa18-29fb-0ae19be28b3c@comcast.net>
Date: Mon, 19 Jun 2017 11:02:16 -0400
User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.10; rv:52.0) Gecko/20100101 Thunderbird/52.2.0
MIME-Version: 1.0
In-Reply-To: <CAA4Mczvi-+sCYHO9NO48wE8o6YTvZ-a216d1L_yLvgwFS9O3zw@mail.gmail.com>
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Language: en-US
Content-Transfer-Encoding: 7bit
X-CMAE-Envelope: MS4wfKT4dURaYyqZj2r+OG4SlxzinFDR77gMnlI1Nbez9DnxRtffYtU1jnqSmgCoXIhFEHSfwMq5QjIn3VwACeMlGs7CR/l4ZZ2ucJJkBFIGNlPzoT+VS97M ZrnuXzrGs+l4R8AAnDt1SbAsYUdefpLYpxpcLkutOLxDFY0pONOp27Ib
Archived-At: <https://mailarchive.ietf.org/arch/msg/mmusic/TC5hA1_9d5yajKjA00hk8Yy1yIQ>
Subject: Re: [MMUSIC] Fwd: New Version Notification for draft-ietf-mmusic-rfc4566bis-19.txt
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, 19 Jun 2017 15:02:26 -0000

Hi Ali,

On 6/16/17 6:32 PM, Ali C. Begen wrote:
> Hi everyone
> 
> There have been some comments on and off the list regarding the 4566bis 
> draft and with this update, I am hoping that I made all the changed 
> asked for. There were a few errors that were taken care of, some ABNF 
> syntax has been improved, references have been updated, etc.
> 
> https://www.ietf.org/rfcdiff?url1=draft-ietf-mmusic-rfc4566bis-18&url2=draft-ietf-mmusic-rfc4566bis-19
> 
> Please check whether you have outstanding issues or not. I saw that the 
> chairs updated the milestone for this draft, and from my perspective, 
> the draft is ready to go to WGLC.

I like the change to the grammar that moves the notation for 
optionality/repetition of individual fields from the definition of those 
fields to the overall definition of the session-description:

    session-description = proto-version
                          origin-field
                          session-name-field
                          [information-field]
                          [uri-field]
                          *email-fields
                          *phone-fields
                          [connection-field]
                          *bandwidth-fields
                          1*time-fields
                          [key-field]
                          *attribute-fields
                          *media-descriptions

However, this change induced a bug in the definition of media-descriptions:

    media-descriptions =  media-field
                          information-field
                          *connection-field
                          bandwidth-fields
                          key-field
                          attribute-fields

This needs the same changes made to session-description. So this needs 
to be changed to:

    media-descriptions =  media-field
                          [information-field]
                          *connection-field
                          *bandwidth-fields
                          [key-field]
                          *attribute-fields

Also this change does result in some very odd naming, where the names of 
some fields are plural but now ought to be singular. These ought to be 
fixed. Specifically:

email-fields => email-field
phone-fields => phone-field
bandwidth-fields => bandwidth-field
time-fields => time-field
attribute-fields => attribute-field
media-descriptions => media-description

Otherwise the new version seems good to me.

	Thanks,
	Paul