Re: [MMUSIC] RID update (draft-pthatcher-mmusic-rid-01,> )

Adam Roach <adam@nostrum.com> Fri, 16 October 2015 16:42 UTC

Return-Path: <adam@nostrum.com>
X-Original-To: mmusic@ietfa.amsl.com
Delivered-To: mmusic@ietfa.amsl.com
Received: from localhost (ietfa.amsl.com [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id F097A1ACD0A for <mmusic@ietfa.amsl.com>; Fri, 16 Oct 2015 09:42:04 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.91
X-Spam-Level:
X-Spam-Status: No, score=-1.91 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, T_RP_MATCHES_RCVD=-0.01] autolearn=ham
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 Jpz4mD9bGjEV for <mmusic@ietfa.amsl.com>; Fri, 16 Oct 2015 09:42:02 -0700 (PDT)
Received: from nostrum.com (raven-v6.nostrum.com [IPv6:2001:470:d:1130::1]) (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 43B711ACCDE for <mmusic@ietf.org>; Fri, 16 Oct 2015 09:42:02 -0700 (PDT)
Received: from Orochi.local (99-152-145-110.lightspeed.dllstx.sbcglobal.net [99.152.145.110]) (authenticated bits=0) by nostrum.com (8.15.2/8.14.9) with ESMTPSA id t9GGfvKB063178 (version=TLSv1.2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128 verify=NO); Fri, 16 Oct 2015 11:41:58 -0500 (CDT) (envelope-from adam@nostrum.com)
X-Authentication-Warning: raven.nostrum.com: Host 99-152-145-110.lightspeed.dllstx.sbcglobal.net [99.152.145.110] claimed to be Orochi.local
To: Paul Kyzivat <pkyzivat@alum.mit.edu>, mmusic@ietf.org
References: <56202075.3000202@nostrum.com> <5620692D.5030703@alum.mit.edu>
From: Adam Roach <adam@nostrum.com>
Message-ID: <562128D5.1040503@nostrum.com>
Date: Fri, 16 Oct 2015 11:41:57 -0500
User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.10; rv:38.0) Gecko/20100101 Thunderbird/38.3.0
MIME-Version: 1.0
In-Reply-To: <5620692D.5030703@alum.mit.edu>
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Transfer-Encoding: 7bit
Archived-At: <http://mailarchive.ietf.org/arch/msg/mmusic/WLvV3RKr3b7s3iR9Ct2wvkTqQIw>
Subject: Re: [MMUSIC] RID update (draft-pthatcher-mmusic-rid-01,> )
X-BeenThere: mmusic@ietf.org
X-Mailman-Version: 2.1.15
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: Fri, 16 Oct 2015 16:42:05 -0000

On 10/15/15 22:04, Paul Kyzivat wrote:
> * Section 7.1:
>
> In:
>
>  1. It MUST generate a 'rid-identifier' that is unique within a media
>     description
>
> IIUC, the rid-identifier must be unique within an RTP session, 
> including all the media descriptions pertaining to that RTP session.

Added:

     Note that 'rid's are only required to be unique within a media
     section ("m-line"); they do not necessarily need to be unique within
     an entire RTP session.  In traditional usage, each media section is
     sent on its own unique 5-tuple, which provides an unambiguous scope.
     Similarly, when using BUNDLE
     [I-D.ietf-mmusic-sdp-bundle-negotiation], MID values associate RTP
     streams uniquely to a single media description.


>
> * Section 7.2.2, item 2:
>
> Restating my earlier comment, why not simply check the PT against 
> those listed in the m-line? There could be default PTs in use that 
> have no rtpmap or fmtp. Those are only defined for audio, but there 
> *could* be rid constraints applied to audio. (I think max-br would 
> potentially be applicable. Or new ones might be defined that apply.) 
> At least it seems harmless to define it this way.

Updated to:

      ...payload types is verified against the list of valid payload types
         for the media section (that is, those listed on the "m=" line).
         If there is no match for the Payload Type listed in the "a=rid"
         line, then remove the "a=rid" line.

>
>
> * Section 7.4, item 4:
>
> I think there should be a check that the PTs in the answer are a 
> subset of those in the offer.

Added:

     4.  If the "a=rid" line contains a "pt=" parameter, the offerer
         verifies that the list of payload types is a subset of those sent
         in the corresponding "a=rid" line in the offer.


> (And, while it can be inferred, it might be good to state somewhere 
> that in cases where the answerer m-line PT values differ from those in 
> the offer, the PTs in the answer rid use the offerer's PT values.)

Added:

     Note: in the case that the answerer uses different PT values to
     represent a codec than the offerer did, the "a=rid" values in the
     answer use the PT values that were sent in the offer.

/a