[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.)