[clue] Ben Campbell's Discuss on draft-ietf-clue-rtp-mapping-12: (with DISCUSS and COMMENT)

"Ben Campbell" <ben@nostrum.com> Wed, 18 January 2017 20:48 UTC

Return-Path: <ben@nostrum.com>
X-Original-To: clue@ietf.org
Delivered-To: clue@ietfa.amsl.com
Received: from ietfa.amsl.com (localhost [IPv6:::1]) by ietfa.amsl.com (Postfix) with ESMTP id 5E8B6128E18; Wed, 18 Jan 2017 12:48:01 -0800 (PST)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
From: Ben Campbell <ben@nostrum.com>
To: The IESG <iesg@ietf.org>
X-Test-IDTracker: no
X-IETF-IDTracker: 6.40.3
Auto-Submitted: auto-generated
Precedence: bulk
Message-ID: <148477248127.2230.11179332810356559654.idtracker@ietfa.amsl.com>
Date: Wed, 18 Jan 2017 12:48:01 -0800
Archived-At: <https://mailarchive.ietf.org/arch/msg/clue/mOrHjswHeckRRUSCF57T6g-dQ8M>
Cc: clue@ietf.org, clue-chairs@ietf.org, draft-ietf-clue-rtp-mapping@ietf.org
Subject: [clue] Ben Campbell's Discuss on draft-ietf-clue-rtp-mapping-12: (with DISCUSS and COMMENT)
X-BeenThere: clue@ietf.org
X-Mailman-Version: 2.1.17
List-Id: CLUE - ControLling mUltiple streams for TElepresence <clue.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/clue>, <mailto:clue-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/clue/>
List-Post: <mailto:clue@ietf.org>
List-Help: <mailto:clue-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/clue>, <mailto:clue-request@ietf.org?subject=subscribe>
X-List-Received-Date: Wed, 18 Jan 2017 20:48:01 -0000

Ben Campbell has entered the following ballot position for
draft-ietf-clue-rtp-mapping-12: Discuss

When responding, please keep the subject line intact and reply to all
email addresses included in the To and CC lines. (Feel free to cut this
introductory paragraph, however.)


Please refer to https://www.ietf.org/iesg/statement/discuss-criteria.html
for more information about IESG DISCUSS and COMMENT positions.


The document, along with other ballot positions, can be found here:
https://datatracker.ietf.org/doc/draft-ietf-clue-rtp-mapping/



----------------------------------------------------------------------
DISCUSS:
----------------------------------------------------------------------

I plan to ballot "yes" for this, but there's an issue in section 9 that I
think needs to be fixed first:

In the second paragraph, the draft says "CLUE endpoints MUST support RTP/
SAVPF and DTLS-SRTP keying [RFC5764]." But the framework draft goes
further by saying that media MUST be secured, and that DTLS-SRTP SHOULD
be used unless the media is secured by some other mechanism. I think that
readers will expect the mapping spec to be authoritative about that sort
of thing. It's likely to be misleading to have it mention the requirement
to support RTP/SAVPF and DTLS-SRTP without also mentioning the MUST be
secured, SHOULD be used requirements. This can easily be fixed by
mentioning the additional requirements and citing the framework.


----------------------------------------------------------------------
COMMENT:
----------------------------------------------------------------------

Substantive:

-3: The opening paragraph mentions "Point-to-Point, as well as
Media-Mixing
   mixers, Media- Switching mixers, and Selective Forwarding Middleboxs."
The section goes on to discuss the first two, but doesn't mention the
last two.

-5, paragraph 4: "When the media provider
   switches the MC it sends within an MCC, it MUST send the captureID
   value for the MC just switched into the MCC."

Does MUST send mean MUST send both in the RTP header and as an RTCP SDES
item, as in the previous sentence?

-5, paragraph 5:
It would be helpful to add a sentence or two clarifying the difference
between an MCC that carries multiple MCs at the same time, and one that
switch between MCs, but only carry one at a time.


Editorial:

- 1, 2nd paragraph: Please expand SSRC on first use.

-3, 2nd paragraph: Please expand SDES on first use.
-- "specified in CLUE use case [RFC7205]"
should that say "specified in the CLUE use case document [RFC7205]"?
-- "described using MCC."
missing article ("using an MCC."

-3, third paragraph: "If needed by CLUE
   endpoint,"
Missing article.

-4, 2nd paragraph: s/"slide video"/"side video"

-6, first paragraph: s/"made by"/"made up of"  ; or "comprises"

-9, 2nd paragraph: Please don't use 2119 to describe requirements from
other documents, unless in a direct quote (in quotation marks.)