Re: [MMUSIC] Draft new version: t140-data-channel-usage-03 - the pull request
Christer Holmberg <christer.holmberg@ericsson.com> Wed, 18 September 2019 12:24 UTC
Return-Path: <christer.holmberg@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 82A6A120232 for <mmusic@ietfa.amsl.com>; Wed, 18 Sep 2019 05:24:22 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.027
X-Spam-Level:
X-Spam-Status: No, score=-2.027 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, RCVD_IN_DNSWL_NONE=-0.0001, RCVD_IN_MSPIKE_H2=-0.026, 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 fBpOjhLh48_S for <mmusic@ietfa.amsl.com>; Wed, 18 Sep 2019 05:24:20 -0700 (PDT)
Received: from EUR01-DB5-obe.outbound.protection.outlook.com (mail-eopbgr150054.outbound.protection.outlook.com [40.107.15.54]) (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 53C2D12004D for <mmusic@ietf.org>; Wed, 18 Sep 2019 05:24:18 -0700 (PDT)
ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=GrEM3Eh9moN6vteePJwswGW8YTpav4ksgP7PdICYrSINz8AjcIkC/BWzBCS2Yv7/UF3B8V4Nt/a2ubTvnrQhlLNE3pn5RMW/xkR0LyY9O3+QaA23Xpf/iHe5h5ieqCRSSOKavz47a4TUhISn+m46nna/0EtFjAp96J2aRjilD99ILjiexeaji4FJtUlikTnoN1b2+7RMmVTVyvT2ez7cQQXVMJWm32eG9KDDkejMIKhFBIzkuE3Fh3HpQdJnqGQMop+zM9odTtkyNr74DcjqDCf97sIk+k7wEENIPI0oLokNyhq7fp8HxRD0mcPup9avOHQGtiV5r5QLVQKTm9edPw==
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=LP3KP0lsXxqxswLfkIuHTsznUOoQhTdsBiJ4sWA7WzE=; b=CIMqCkJQ1TgoVLSiTTeuTAx/TrW4JjU7cXrzyFIOIV6eb4uQQ5mNGEQXbUYlwjkfFTq8AqaR6NO5RnxkFwwEq19SDh2F8oUTvNyFdu79OPF9StnHrmn7hKKX5yMib/eomwmbvwR5xqASe8exI3yEfmOqHvShrRVja8UoPIVY1jKGMt9sm2+VvVXehw6LFZe9dfCQq7iFw0WwNheOgos9UivJHohYpDERO4cs1d3IyjcxaOtKvECpcp7RH2I1usKkjpwlDWs0ynuuhrgRL8ptNDggeYlDMLepXrcq2ILNM/VJurQDaQcTpIMxkLqsGHq44J+Dj8WaCyF3V+wM2NbfBg==
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=selector2; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=LP3KP0lsXxqxswLfkIuHTsznUOoQhTdsBiJ4sWA7WzE=; b=KvglBm+2vKwpbzNS/xTr6tWpPqUGmOdqBYwwwUvnEKxQPEEOby8DYyJf8OAkVobmKXdV1EPhN/B0BfZk/62AQBxORhth/JIKIOnmqH/h5YgKtZ/UafExytsInebH+HGVAU9v0F3Qt6Z3VwoUy3aoyA5TxOp/EcIGy7ReqYx2DJ0=
Received: from HE1PR07MB3161.eurprd07.prod.outlook.com (10.170.245.23) by HE1PR07MB3131.eurprd07.prod.outlook.com (10.170.245.17) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.2284.17; Wed, 18 Sep 2019 12:24:13 +0000
Received: from HE1PR07MB3161.eurprd07.prod.outlook.com ([fe80::f0a1:2199:7816:ff8d]) by HE1PR07MB3161.eurprd07.prod.outlook.com ([fe80::f0a1:2199:7816:ff8d%6]) with mapi id 15.20.2284.009; Wed, 18 Sep 2019 12:24:13 +0000
From: Christer Holmberg <christer.holmberg@ericsson.com>
To: Paul Kyzivat <pkyzivat@alum.mit.edu>, "mmusic@ietf.org" <mmusic@ietf.org>
Thread-Topic: [MMUSIC] Draft new version: t140-data-channel-usage-03 - the pull request
Thread-Index: AQHVbhv7tx6JxDbGl0msXZVobgH4/g==
Date: Wed, 18 Sep 2019 12:24:13 +0000
Message-ID: <0A055523-E70F-4557-9029-6A326D41CAED@ericsson.com>
Accept-Language: en-US
Content-Language: en-GB
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
user-agent: Microsoft-MacOutlook/10.1b.0.190715
authentication-results: spf=none (sender IP is ) smtp.mailfrom=christer.holmberg@ericsson.com;
x-originating-ip: [89.166.49.243]
x-ms-publictraffictype: Email
x-ms-office365-filtering-correlation-id: 29811be3-1f3b-4a0b-b366-08d73c331def
x-microsoft-antispam: BCL:0; PCL:0; RULEID:(2390118)(7020095)(4652040)(8989299)(5600167)(711020)(4605104)(1401327)(4534185)(4627221)(201703031133081)(201702281549075)(8990200)(2017052603328)(7193020); SRVR:HE1PR07MB3131;
x-ms-traffictypediagnostic: HE1PR07MB3131:
x-ms-exchange-purlcount: 2
x-microsoft-antispam-prvs: <HE1PR07MB31312711BB31D5CE05360DA0938E0@HE1PR07MB3131.eurprd07.prod.outlook.com>
x-ms-oob-tlc-oobclassifiers: OLM:10000;
x-forefront-prvs: 01644DCF4A
x-forefront-antispam-report: SFV:NSPM; SFS:(10009020)(4636009)(346002)(396003)(39860400002)(376002)(136003)(366004)(199004)(189003)(186003)(81166006)(6306002)(6486002)(6116002)(66446008)(6512007)(25786009)(6246003)(33656002)(229853002)(476003)(76116006)(2501003)(6436002)(2616005)(99286004)(8676002)(36756003)(66066001)(3846002)(2171002)(81156014)(64756008)(66556008)(66476007)(8936002)(14454004)(66946007)(478600001)(6506007)(7736002)(86362001)(305945005)(316002)(14444005)(966005)(71190400001)(71200400001)(2906002)(256004)(58126008)(102836004)(26005)(110136005)(44832011)(486006)(5660300002); DIR:OUT; SFP:1101; SCL:1; SRVR:HE1PR07MB3131; H:HE1PR07MB3161.eurprd07.prod.outlook.com; FPR:; SPF:None; LANG:en; PTR:InfoNoRecords; A:1; MX:1;
received-spf: None (protection.outlook.com: ericsson.com does not designate permitted sender hosts)
x-ms-exchange-senderadcheck: 1
x-microsoft-antispam-message-info: TOYXwu54hamt+mDD7zZW2XX4txJ6GEqWWmObIp5OjSenXx1Y+r1JQHCBLXdExpdd/ZmLck/mHdQOkYYVYVmbdC3JvkT/EYJ90uwX08yEgtcEIkFrGsZhrft3ygHCalDE6d641hnoLLbnhydxjXw5Ua1YPtVcAMG9XmDfT3IIPDVuDlAS1h9SvJum/+BgAiNTZnfXBLXOfbyQscxpYIT9VpMBtwStB0sjKXcSIuf119HusS67PaV76ZpRzfbc1br82E27P6FQicExipKInf9dwM1VqtqVTp4CQgwkJkXM6XDXWbhp7oBTHTawcX4ohEnk5/Rry7SvyEdhVwKVghKcWPPhS4oW1la3LASZU72u+VgfILhoP4iQL0vnBwpgLeL3JVPGtm3k9p3QNWkjt3vMYQqHdGuvgkKr2sk5Xnj7VlM=
x-ms-exchange-transport-forked: True
Content-Type: text/plain; charset="utf-8"
Content-ID: <DD279BBB2066304782C8D6FC6CA6266F@eurprd07.prod.outlook.com>
Content-Transfer-Encoding: base64
MIME-Version: 1.0
X-OriginatorOrg: ericsson.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 29811be3-1f3b-4a0b-b366-08d73c331def
X-MS-Exchange-CrossTenant-originalarrivaltime: 18 Sep 2019 12:24:13.8282 (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: 86cyHNWHRFAoivW+yywPk5f8H/8wPfSYv5D+IMaE7XfQm3kymQxWkh+M1Gyp04sgcwb7PUzgMyXO8AMp12ABL5DIFyUzHvhKHyiva3QZ7f8=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: HE1PR07MB3131
Archived-At: <https://mailarchive.ietf.org/arch/msg/mmusic/oz7FeIWSZaDUKewWllb4J0C3O1I>
Subject: Re: [MMUSIC] Draft new version: t140-data-channel-usage-03 - the pull request
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: Wed, 18 Sep 2019 12:24:23 -0000
The pull request: https://github.com/cdh4u/draft-datachannel-t140/pull/32 Regards, Christer On 17/09/2019, 18.27, "mmusic on behalf of Paul Kyzivat" <mmusic-bounces@ietf.org on behalf of pkyzivat@alum.mit.edu> wrote: The changes made in -03 seem fine to me. Then I realized that I have only been looking at diffs for awhile, and haven't carefully reviewed the full document for awhile. So I did that, and found some other thing that I think are worthy of consideration: * Section 4: s/proceudres/procedures/ * Section 4.2: The subsections 4.2.n describe some specific SDP attributes that may be used with T.140 data channels. But it isn't definitive about other attributes. I think you should be explict whether other attributes maybe included. In any case, I think it would be prudent to specify that if other attributes are *received* and are not understood then they are to be ignored, in order to be consistent with SDP treatment of attributes. * Section 4.2.1: This section is a little hard to follow. The use of fmtp with data channels is complicated, since the value part of fmtp is dependent on the fmt in the m= line. You do reference RFC4103, but it is still a leap to put this all together. I suggest you first discuss the general use of fmtp with data channels - that the value syntax of fmtp is to follow RFC4103. *Then* you can move on to discuss the use of 'cps'. It isn't clear to me if there are any other meaningful or permitted fmtp parameters. I suggest you make a definitive statement about this. * Section 5: These examples are good. But there is no example for CPS, and it would be nice to have one. Rather than having a separate example for each, I think it would be sufficient to fold multiple attributes into a single example. * Section 5.3: In the following: As described in [T140], buffering can be used to reduce overhead, with the maximum buffering time being 500 ms. It can also be used for staying within the maximum character transmission rate (Section 4.2), if such has been provided by the peer. it is unclear to me whether the 500ms max time is applicable when using buffering to stay within the max transmission rate. I presume not. Can you be more explicit? * Section 5.4: It isn't clear to me whether this is talking about failure and re-establishment of individual data *channels*, or of the containing SCTP association. I guess you mean both. But it would be good to be explicit, since the process for doing so is a bit different. * General: Perhaps replace the references to RFC4566 throughout with references to RFC4566bis. It is already in the editor's queue so shouldn't cause a delay. Since you have no dependency on the updates in RFC4566bis this isn't an essential change. It is just a nice-to-have. Thanks, Paul _______________________________________________ mmusic mailing list mmusic@ietf.org https://www.ietf.org/mailman/listinfo/mmusic
- Re: [MMUSIC] Draft new version: t140-data-channel… Christer Holmberg