[MMUSIC] Shepherd's review of draft-ietf-msrp-usage-data-channel-17

Bo Burman <bo.burman@ericsson.com> Tue, 09 June 2020 23:24 UTC

Return-Path: <bo.burman@ericsson.com>
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 900E93A0BF2; Tue, 9 Jun 2020 16:24:34 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.101
X-Spam-Level:
X-Spam-Status: No, score=-2.101 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIMWL_WL_HIGH=-0.001, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, HTML_MESSAGE=0.001, RCVD_IN_MSPIKE_H2=-0.001, SPF_PASS=-0.001, URIBL_BLOCKED=0.001] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (1024-bit key) header.d=ericsson.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 fHmoljyvZQNX; Tue, 9 Jun 2020 16:24:32 -0700 (PDT)
Received: from EUR05-VI1-obe.outbound.protection.outlook.com (mail-vi1eur05on2079.outbound.protection.outlook.com [40.107.21.79]) (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 3C10F3A0C35; Tue, 9 Jun 2020 16:24:31 -0700 (PDT)
ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=DG9RhvCO8lzuUejZ8GA4+FHnuxlrdJGsNRsNQ6ADVfoqd5LMweXPcYY4EhinuI1yZ1QQC6UH2kiWWqyu5092HMRV6+nE7fuWxQuV/lJmp3jVnqlM7Utg7eDmp12ygFhaepU6SJFSIO6GD/oaBoaFCbGmLZAsxJtxk4yR3TwfVWHsMSdLl+OTQW4bule3CA7orhDDFVIpGrociaB79gvkSWYMY8V1d4VsxRAbyd9qVdmA82ccSInMb12OlxcSK5pVPm0TFSZ0G1c/PVAb466Xo83X3N6CPKm5CgHLWbMNX8J1w7+UwBk3ZEUgIZAKSxrPCjZe2aD+gglPvMeG4w1yCA==
ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=HDo5X1fyj7Z1bKa8wOR7deO7ceqvLqhAigU+5Wcz9HI=; b=EESCXghw/wgoJbQSei6hVpWckf1ThmFfveSTEMTykUYJdz+vu3mrGrUBEPHIwMmeeVkZgM8z7l1x7mrb8qlrDiUM+nWud4qbCLfCw4D6RxXNWxA+AM4f1ZZZJxh5EoqQacapkD809FTGG4KTzG3zG1djAYDAnERvE6xaMIvr7c9Z8tPMdF0b94wROMM7kpZG4beFCFRxxmpFgT264ABZMm19Ig52078W1DYjt8X0dcBMcGkorbGjRY6retjrBnOwFrnxOxLqltbsO7te0hYt6BW1ZTU35g2o5S/tCsf3tfSfjma6rLXb/8Ijtqw9KIOboQ09hE6avo/D66wG4pF6xQ==
ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=ericsson.com; dmarc=pass action=none header.from=ericsson.com; dkim=pass header.d=ericsson.com; arc=none
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ericsson.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=HDo5X1fyj7Z1bKa8wOR7deO7ceqvLqhAigU+5Wcz9HI=; b=fwv0dC3s/154e0/pdInOwzSC5qN5FTpuZUhpH+W9lGcEOpBNDm498AM6b1TRmuTEFo725s6XN71aizWYoqFhTmtwcL3bu1OfOYN+sThGpPkGYPkn3UN7j1jyx16HbkpU+akR4/243kjvgKx3tm723DnEV5ywLrfueVfo0i2EMNE=
Received: from HE1PR07MB4426.eurprd07.prod.outlook.com (2603:10a6:7:a1::15) by HE1PR0701MB2665.eurprd07.prod.outlook.com (2603:10a6:3:97::8) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3088.15; Tue, 9 Jun 2020 23:24:29 +0000
Received: from HE1PR07MB4426.eurprd07.prod.outlook.com ([fe80::8de4:f9d4:1e81:6c79]) by HE1PR07MB4426.eurprd07.prod.outlook.com ([fe80::8de4:f9d4:1e81:6c79%5]) with mapi id 15.20.3088.018; Tue, 9 Jun 2020 23:24:29 +0000
From: Bo Burman <bo.burman@ericsson.com>
To: "draft-ietf-mmusic-msrp-usage-data-channel.authors@ietf.org" <draft-ietf-mmusic-msrp-usage-data-channel.authors@ietf.org>, mmusic <mmusic@ietf.org>
CC: "mmusic-chairs@ietf.org" <mmusic-chairs@ietf.org>
Thread-Topic: Shepherd's review of draft-ietf-msrp-usage-data-channel-17
Thread-Index: AdY+tQCYaK/SvE0yRXW9AL9PPo+jrQ==
Date: Tue, 09 Jun 2020 23:24:29 +0000
Message-ID: <HE1PR07MB4426977EA914E7253488C5D08D820@HE1PR07MB4426.eurprd07.prod.outlook.com>
Accept-Language: sv-SE, en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
authentication-results: ietf.org; dkim=none (message not signed) header.d=none;ietf.org; dmarc=none action=none header.from=ericsson.com;
x-originating-ip: [77.53.238.202]
x-ms-publictraffictype: Email
x-ms-office365-filtering-correlation-id: be4959d8-e200-476a-4d0c-08d80ccc4212
x-ms-traffictypediagnostic: HE1PR0701MB2665:
x-microsoft-antispam-prvs: <HE1PR0701MB2665A49FB89647CBC613983F8D820@HE1PR0701MB2665.eurprd07.prod.outlook.com>
x-ms-oob-tlc-oobclassifiers: OLM:8273;
x-forefront-prvs: 042957ACD7
x-ms-exchange-senderadcheck: 1
x-microsoft-antispam: BCL:0;
x-microsoft-antispam-message-info: AiINeHMERpfEzLhTZPx4+9L6q5oRafxfreknua/QCvrRRxiRMg61BuQTIeYU/KbsQLK0oEbBVdNbg381NG4MwAxi39fqcZMFSFsVr8NWI7Qo+oqefjXKcJoJhShY7DOohyg7yfqCdTKSb0filwVdUJ20shPrR3rc4k33zJ1CVaYYjcM97hZGPI3PHVio97hoWZrUou4SL3RW6T0v1/MyBbZ+cdpGMlkf0oZs0/tYncyR22LJOtieXF2AR9/Zj1g6P3uSoCL5fHdrxFrqen4wBS6tZzhVNZ8qsKmVS1NMH8dUF6LxDrgdfft2HqPRk1v9/Ti+TNljmHYEPkvn04sLB9IIqP0Eu1/tgUZ1lrPr8BDRiQ8LrQ2XNY8vQzW4E62uegNR0ROqpitQLf1uXkTWOg==
x-forefront-antispam-report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:HE1PR07MB4426.eurprd07.prod.outlook.com; PTR:; CAT:NONE; SFTY:; SFS:(4636009)(39860400002)(376002)(396003)(346002)(136003)(366004)(7696005)(478600001)(52536014)(8936002)(71200400001)(55016002)(2906002)(26005)(316002)(9686003)(110136005)(450100002)(4326008)(66556008)(6506007)(66446008)(76116006)(66946007)(64756008)(83380400001)(66476007)(186003)(33656002)(44832011)(86362001)(8676002)(166002)(5660300002); DIR:OUT; SFP:1101;
x-ms-exchange-antispam-messagedata: lm2XIOEonIrHjOb7LmrMKTCGO933E1T6Hs9RYsQixlVJ036EqNzxUv9nQNAHOjKWgemRCm8kt8caeq8dIyYyaRz4lYyUlgIJI0GdehTflmYvtk+is3Dt0KeqzpNewGCDBZfnfuoGY2cyxniZ5pMK4gQqQ8zU/TqYLZytM6h7uL6o+T1KHRYx/e6XNLc6+zML1o2gBZ/XIUNcj5kfoUv60e/VecmRdZKyg1Vot051tZlrH7+qGW/qW/FoPR+ZwoCds8qlccu6BXN2vUaiAdK2MQPRtF0b9dzRlUhqg/jbwrR+UWDwQyxzixCRH9SVGwsq5pb8vN467kuYeth72OIHFjIJtgo3E+oU1KslzuaHSWos0zfUL7+v1mRHXx+nDnsMLf5s7Qe3Y9dBY/WPVR9UlJpivhCU9HNJ8fIjLbSvzdvSj/j3D52Jh2+28DWaPTbbpZOklmEPpau+EmTlwjEuX0XIUeIRz3IbEqOjT1b22D2Zs/ovag/XfNLNFuIGbL1N
x-ms-exchange-transport-forked: True
Content-Type: multipart/alternative; boundary="_000_HE1PR07MB4426977EA914E7253488C5D08D820HE1PR07MB4426eurp_"
MIME-Version: 1.0
X-OriginatorOrg: ericsson.com
X-MS-Exchange-CrossTenant-Network-Message-Id: be4959d8-e200-476a-4d0c-08d80ccc4212
X-MS-Exchange-CrossTenant-originalarrivaltime: 09 Jun 2020 23:24:29.2223 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 92e84ceb-fbfd-47ab-be52-080c6b87953f
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: /jFKLU7DLmxJaf2Yz66R5DRwxr6p7X7cPa3AaG+LWcVEL9Z3qf3pdsSWnfr5D3WgtG3YZC97UDVVDe6L/6l45g==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: HE1PR0701MB2665
Archived-At: <https://mailarchive.ietf.org/arch/msg/mmusic/6Y2Pzjh6WFIJjMjouKq3jivlmVQ>
Subject: [MMUSIC] Shepherd's review of draft-ietf-msrp-usage-data-channel-17
X-BeenThere: mmusic@ietf.org
X-Mailman-Version: 2.1.29
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: Tue, 09 Jun 2020 23:24:35 -0000

WG, authors,



These are my shepherd's review comments:

1.      Please point more clearly to which IANA registries that are targeted in sections 9.1 and 9.3.
2.      In section 4.4, you are discussing dcsa use of setup, path, msrp-cema, accept-types, accept-wrapped-types, max-size, sendonly, recvonly, inactive, sendrecv, and "all the parameters related to MSRP file transfer" (section 4.7), but only setup has a corresponding IANA section (9.3), asking to register the attribute on dcsa level. Please add IANA information for all of the other specified attributes as well (as separate sections or extend section 9.3, as you see fit).
3.      In section 3.1 change sub-protocol value in the body text from "MSRP" to "msrp", for consistency.



Editorial comments/nits:

i)                    Consider aligning all occurrences of "a MSRP" to "an MSRP".



Idnits<https://tools.ietf.org/tools/idnits/>:

a.      The document has no formal "updates" tag in the file header, it just mentions it in the abstract. Please use the formal title page header format, also for the provided XML.
b.      The abstract contains RFC4975 as a reference, which cannot be part of an abstract and must be replaced with a plain-text mention of that RFC.



Thanks,

Bo

MMUSIC co-chair