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

"Ali C. Begen" <ali.begen@networked.media> Wed, 21 June 2017 23:39 UTC

Return-Path: <ali.begen@networked.media>
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 EED8F12426E for <mmusic@ietfa.amsl.com>; Wed, 21 Jun 2017 16:39:37 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.898
X-Spam-Level:
X-Spam-Status: No, score=-1.898 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, HTML_MESSAGE=0.001, RCVD_IN_DNSWL_NONE=-0.0001, URIBL_BLOCKED=0.001] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (2048-bit key) header.d=networked-media.20150623.gappssmtp.com
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 SNwcPVwq8pTB for <mmusic@ietfa.amsl.com>; Wed, 21 Jun 2017 16:39:35 -0700 (PDT)
Received: from mail-yb0-x235.google.com (mail-yb0-x235.google.com [IPv6:2607:f8b0:4002:c09::235]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id B53C81200CF for <mmusic@ietf.org>; Wed, 21 Jun 2017 16:39:35 -0700 (PDT)
Received: by mail-yb0-x235.google.com with SMTP id 84so150503ybe.0 for <mmusic@ietf.org>; Wed, 21 Jun 2017 16:39:35 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=networked-media.20150623.gappssmtp.com; s=20150623; h=mime-version:in-reply-to:references:from:date:message-id:subject:to :cc; bh=6uYqiEQHcEs7cnMCWMjy0O0L7eFZ4/Daq7sAS0aQ7fM=; b=062fMNhx7aqA7922rOmYP/4Df+eTtDtkQTrBiK+nsq2MTqc7oDBhFahRb8Fl2wday8 IK9W+vCaGTMuohPLiBmDoSl0DPDh8XWArYeu7vIkxBYSo8icVeeWM3riiWJyiAZDgIfj 1to85LsoKKbwOW7fu73tgydCAbsmxLu+hbGppF0qhwovURV5Fw9NPM7qOX6O90HYMgLx 8So5HKTZazHXSdhAOt5kc4OTuDOqIu2aHx2uMyzlJoQrTDxs0oMRlSE9/d0MQ7td5168 qNwauCC6ZqfRyK+4Ymefe1bJtyNO5TWiz5hjRxTaURUJCFIj6pcDhnpXm2bx9whsWrk/ AHmA==
X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:in-reply-to:references:from:date :message-id:subject:to:cc; bh=6uYqiEQHcEs7cnMCWMjy0O0L7eFZ4/Daq7sAS0aQ7fM=; b=e+DvaZ3jB0pUWQu9pE20Zh4SJszlQKg+lKQiyKIX+0ikXZqmewRARYPGQY6eXzkbkh s98U2vPLFsvPz6rqPph4WJVirXYVeMyIuWi3joAqMRGS3W9fVoGE7dw4XmDsSJir+gH6 eyT7ZTu9YTDuPFDCGTw3Z+CkXKswAD6PgJ1JZRWybOUuH9reEhoqXkJS58bXvBGPmWl/ yd3qigKF4Zm0y2Bm26mCcOVWyfmFAh6AtPbEa5qMIlpPIBw+6alhzD9pC90viD/kdh0U c3/KW+QRxQjgshpy3AdId12J2dLDcC5r9OvAIpciIJc4ixhGKro2eeI4PWBmlecNiht6 2VxQ==
X-Gm-Message-State: AKS2vOyFr3RMy6qD70A/Iqwb2IfxnhD3P+NIJSPouzDW3x3V9MA/dTlf 8MBlBhVGIRZ6kyP2DPaeIuRvi1Sz80yc
X-Received: by 10.37.75.5 with SMTP id y5mr3993025yba.257.1498088374921; Wed, 21 Jun 2017 16:39:34 -0700 (PDT)
MIME-Version: 1.0
Received: by 10.129.91.70 with HTTP; Wed, 21 Jun 2017 16:39:34 -0700 (PDT)
In-Reply-To: <aa23c37c-7e6a-fa18-29fb-0ae19be28b3c@comcast.net>
References: <149765179618.24205.7010922835529746440.idtracker@ietfa.amsl.com> <CAA4Mczvi-+sCYHO9NO48wE8o6YTvZ-a216d1L_yLvgwFS9O3zw@mail.gmail.com> <aa23c37c-7e6a-fa18-29fb-0ae19be28b3c@comcast.net>
From: "Ali C. Begen" <ali.begen@networked.media>
Date: Thu, 22 Jun 2017 02:39:34 +0300
Message-ID: <CAA4Mczu8a48rqQ7RRFKVOs83+9pK0DPcbbYdLa9y-NHOUt8B2w@mail.gmail.com>
To: Paul Kyzivat <paul.kyzivat@comcast.net>
Cc: mmusic <mmusic@ietf.org>
Content-Type: multipart/alternative; boundary="94eb2c0941766e027b055280e167"
Archived-At: <https://mailarchive.ietf.org/arch/msg/mmusic/zICJVX7vqJpVFfgeVanhwR-7Jcs>
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: Wed, 21 Jun 2017 23:39:38 -0000

Hi Paul

Thanks for catching the error. It is fixed and submitted now.

-acbegen

On Mon, Jun 19, 2017 at 6:02 PM, Paul Kyzivat <paul.kyzivat@comcast.net>
wrote:

> 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-rfc4566b
>> is-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
>
> _______________________________________________
> mmusic mailing list
> mmusic@ietf.org
> https://www.ietf.org/mailman/listinfo/mmusic
>