Re: [rtcweb] [MMUSIC] Default proto transport in JSEP

Christer Holmberg <christer.holmberg@ericsson.com> Tue, 04 December 2018 07:05 UTC

Return-Path: <christer.holmberg@ericsson.com>
X-Original-To: rtcweb@ietfa.amsl.com
Delivered-To: rtcweb@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 2D09E130DF7 for <rtcweb@ietfa.amsl.com>; Mon, 3 Dec 2018 23:05:06 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -5.747
X-Spam-Level:
X-Spam-Status: No, score=-5.747 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIMWL_WL_HIGH=-1.46, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, HTML_MESSAGE=0.001, RCVD_IN_DNSWL_MED=-2.3, RCVD_IN_MSPIKE_H3=0.001, RCVD_IN_MSPIKE_WL=0.001, SPF_PASS=-0.001, T_KAM_HTML_FONT_INVALID=0.01, URIBL_BLOCKED=0.001] autolearn=unavailable autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (1024-bit key) header.d=ericsson.com header.b=HDgGj0zU; dkim=pass (1024-bit key) header.d=ericsson.com header.b=Ii3DFAwC
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 jqagvt4mL7zy for <rtcweb@ietfa.amsl.com>; Mon, 3 Dec 2018 23:05:03 -0800 (PST)
Received: from sesbmg22.ericsson.net (sesbmg22.ericsson.net [193.180.251.48]) (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 6C1AF130DFB for <rtcweb@ietf.org>; Mon, 3 Dec 2018 23:05:00 -0800 (PST)
DKIM-Signature: v=1; a=rsa-sha256; d=ericsson.com; s=mailgw201801; c=relaxed/simple; q=dns/txt; i=@ericsson.com; t=1543907093; x=1546499093; h=From:Sender:Reply-To:Subject:Date:Message-ID:To:CC:MIME-Version:Content-Type: Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:In-Reply-To:References:List-Id: List-Help:List-Unsubscribe:List-Subscribe:List-Post:List-Owner:List-Archive; bh=kW30SeQVV0iLRu6mGsms3kYeKNWnTNixXxfKqw55OEw=; b=HDgGj0zUIGXSx/qmDgDDIWX+8fDOGMf5X/6Mv3dcWHL8E+CUnTeTg5/uhEUzj5QA EnNHfT/bQa/+Qw9Jl/DyZMqDGh7Je+VWACdSqzlh8cOP0ZjnOAJR7tEotgyxeJ3n JZXrJbQfxYDEEdcWGIUtP2fWx0WOPNnV0LA4jfsaurI=;
X-AuditID: c1b4fb30-39c4e9e0000043c4-0c-5c0627152985
Received: from ESESSMB501.ericsson.se (Unknown_Domain [153.88.183.119]) by sesbmg22.ericsson.net (Symantec Mail Security) with SMTP id 08.99.17348.517260C5; Tue, 4 Dec 2018 08:04:53 +0100 (CET)
Received: from ESESBMB501.ericsson.se (153.88.183.168) by ESESSMB501.ericsson.se (153.88.183.162) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.1466.3; Tue, 4 Dec 2018 08:04:50 +0100
Received: from EUR02-AM5-obe.outbound.protection.outlook.com (153.88.183.157) by ESESBMB501.ericsson.se (153.88.183.168) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.1466.3 via Frontend Transport; Tue, 4 Dec 2018 08:04:50 +0100
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=kW30SeQVV0iLRu6mGsms3kYeKNWnTNixXxfKqw55OEw=; b=Ii3DFAwCSZzj4amanwpEmIRJX/HzyIYd08fsCySCVSxBMS6wqqjxzhdhh03b0LlLFol6s+WF5pOckzji0xwutKiU0ywW+pB4lzeWMMboUUF8s7r8bfFsuPc+VZGivobhPLpFBbh+nabfOcs1EMEXT13cmbfZMSOpZG7xMpcFetk=
Received: from AM6PR07MB5621.eurprd07.prod.outlook.com (20.178.91.14) by AM6PR07MB4486.eurprd07.prod.outlook.com (20.177.38.77) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.1404.7; Tue, 4 Dec 2018 07:04:50 +0000
Received: from AM6PR07MB5621.eurprd07.prod.outlook.com ([fe80::a5dd:4302:feec:e113]) by AM6PR07MB5621.eurprd07.prod.outlook.com ([fe80::a5dd:4302:feec:e113%3]) with mapi id 15.20.1382.023; Tue, 4 Dec 2018 07:04:50 +0000
From: Christer Holmberg <christer.holmberg@ericsson.com>
To: Roman Shpount <roman@telurix.com>, Adam Roach - SIPCORE Chair <adam@nostrum.com>
CC: RTCWeb IETF <rtcweb@ietf.org>, mmusic WG <mmusic@ietf.org>
Thread-Topic: [rtcweb] [MMUSIC] Default proto transport in JSEP
Thread-Index: AQHUhquP7oWmB9W6S0+oK/sz+g4vAaVlZEyAgAAFLoCAABfmgIAABUCAgAf/CNyAAAM1xIAAqHN9
Date: Tue, 4 Dec 2018 07:04:49 +0000
Message-ID: <AM6PR07MB5621629CDF15C89D5AE048E893AF0@AM6PR07MB5621.eurprd07.prod.outlook.com>
References: <CA+9kkMADnZJBaV0hfLuwGU0bGBEP5tCPZ=8Zd_85Dgzi37ghAQ@mail.gmail.com> <CAD5OKxsNFFmER__H0+5Mzts58yn9cWLMEADhSnLR4nreKD9WAQ@mail.gmail.com> <9B9B741B-622F-4565-899B-700636408F6C@iii.ca> <CAD5OKxv9r08RLvMSM4h11A6sXU9E=u_8Qvy-TBfjNcwkhcqf3w@mail.gmail.com> <54ebb208-e7b3-a0f1-6a5c-4745d3a56447@cisco.com> <CAD5OKxut5Lr+Bmyc20y+vV=+_RESw+h72DYLnt3G1_BjS6sTVA@mail.gmail.com> <1346FE48-5D61-48B7-BF37-3D7BAA930DB0@iii.ca> <CAD5OKxv0N+TF3L3bB9KPm4vqQdPZKE=1zkdw1PaV7CpNJ2kYaQ@mail.gmail.com> <110dc822-b3be-7bc2-dcc5-9e6c8277e0d1@nostrum.com>, <CAD5OKxtKOLovNCi0cJiEiHD+M3tCda7ZSecU8EJKxVPuFs7maQ@mail.gmail.com>, <AM6PR07MB5621291E0EA9E72A8065380A93AE0@AM6PR07MB5621.eurprd07.prod.outlook.com>, <AM6PR07MB562184CD3351856DBC45A71B93AE0@AM6PR07MB5621.eurprd07.prod.outlook.com>
In-Reply-To: <AM6PR07MB562184CD3351856DBC45A71B93AE0@AM6PR07MB5621.eurprd07.prod.outlook.com>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
x-originating-ip: [2001:14bb:44:c7e:7d15:40e6:4a11:7e26]
x-ms-publictraffictype: Email
x-microsoft-exchange-diagnostics: 1; AM6PR07MB4486; 6:KPt9W5fxsCQgsRg9NtFvKMXC6gpoDEmS+jgwZoQfVhXOygqToys5hkK2H6qDz4WaqzD/dsiroSuTpApedgxGKLZOiP5rDH3NboUj0+t+W1Y/hfYnue6LwUiA/tHoapgWQq6g6pGebnU+eE1fhyjajocXbVSSYoM+S4/UrVNJgjovbxDJ0MJVpMji0paGZF3uySFOys/f2ojYbeDyEu+V6tcpYFGKZysEe/J3xxOma5Xn/zGuA7Tn/ShGfZr4PbqGmjWVSTNncz5A74E3PkYFaPxbMdmNWkB5lnmuB0NClr0JECO6BWVEdxJCSnKnb1vO8QE9/hMbg58IJVRuBt3Fic4QqaHJ0cOeEIky3pSjtUqtfaRtN+riw9OZVU3+hPWwo1TFsQDgYo/avDlLl7FFGRBaNbxd8hLRYUu/C3ceovClmF28qQN9yLXj/zmIjOdj3pjkgCWhYdfgo8XodrpbdQ==; 5:+DM+pXm09zh3RrCgIxSR+d51K0WyfWXsYETfwRLqWlD2pPqdMU4rKHtjf0GsUyqbdA+6AY7fMkg2iVCSGiTumnxqY4ZlsnPv+W2r6YQNhkVT32ETKsvklZnvuBov2Qxx5t1C33EkSwVPRg/Ahq92ldM7+4pds8C5XuZNubw2JWU=; 7:QrwgzF2bwFScmFFQ7nfY52/oA0ILjMN8Q/cQVOMJE9AD/WsXe8BjdcRcoFE61PnRHjC19wtWTyXXN3CGQ6izVu0PJOjGar0GtIhtHWyr0PNtGiEnavmgjH6t8ZDVyJxX6sW9vVOw845VShNC8SB56w==
x-ms-exchange-antispam-srfa-diagnostics: SOS;
x-ms-office365-filtering-correlation-id: 930080dd-f0b8-448c-39aa-08d659b6c86c
x-microsoft-antispam: BCL:0; PCL:0; RULEID:(2390098)(7020095)(4652040)(8989299)(5600074)(711020)(4534185)(4627221)(201703031133081)(201702281549075)(8990200)(2017052603328)(7153060)(7193020); SRVR:AM6PR07MB4486;
x-ms-traffictypediagnostic: AM6PR07MB4486:
authentication-results: spf=none (sender IP is ) smtp.mailfrom=christer.holmberg@ericsson.com;
x-microsoft-antispam-prvs: <AM6PR07MB4486B3F9F74D2CA7B78A02F993AF0@AM6PR07MB4486.eurprd07.prod.outlook.com>
x-ms-exchange-senderadcheck: 1
x-exchange-antispam-report-cfa-test: BCL:0; PCL:0; RULEID:(8211001083)(6040522)(2401047)(8121501046)(5005006)(93006095)(93001095)(3231455)(999002)(944501493)(52105112)(3002001)(10201501046)(148016)(149066)(150057)(6041310)(20161123562045)(201703131423095)(201702281528075)(20161123555045)(201703061421075)(201703061406153)(20161123560045)(20161123564045)(20161123558120)(201708071742011)(7699051)(76991095); SRVR:AM6PR07MB4486; BCL:0; PCL:0; RULEID:; SRVR:AM6PR07MB4486;
x-forefront-prvs: 0876988AF0
x-forefront-antispam-report: SFV:NSPM; SFS:(10009020)(136003)(376002)(346002)(396003)(39860400002)(366004)(199004)(189003)(54896002)(486006)(6306002)(110136005)(4326008)(55016002)(446003)(11346002)(5660300001)(476003)(44832011)(9686003)(71200400001)(71190400001)(6436002)(6606003)(81166006)(966005)(68736007)(236005)(81156014)(316002)(97736004)(54906003)(606006)(8676002)(5070765005)(33656002)(86362001)(256004)(14444005)(53936002)(6116002)(99286004)(6246003)(74316002)(25786009)(186003)(8936002)(46003)(7696005)(7736002)(106356001)(105586002)(93886005)(19627405001)(478600001)(76176011)(53546011)(6506007)(14454004)(102836004)(2906002)(229853002); DIR:OUT; SFP:1101; SCL:1; SRVR:AM6PR07MB4486; H:AM6PR07MB5621.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-microsoft-antispam-message-info: 4XyFwMa7JiudQ2cEhOiYKiKw17QZkShuksXbX1fwGvbKWsYG77P0uYzZkVBRrsXcBdFspvd1RhzbBgj3l2sHiX9Lp8/W/AQ7ouOaD9opjWf6+rQCIpVBj5n63tqJIkT1rahaDW0/xn5KQtOpjYu8Oj++tRnUwTbdC1KhbaKwmLc3GWub3ZgXRQblsgoyJdQftsXFSwXEDnWCE1aEvMpuYwS5Yl3dEMMaMGzlqBvjrLsc/ATgr7B7hCRMFvBApSDnmnVSu/1kCg1hGRSFm0YThaGMH6ZkV9wVNXTaXsKAQ1kMIpAsoODWnHZEkEICvF9nazSmuE61J8qBGVGBvOs0/JCr9TLv+V96OjuDn/X2DxU=
spamdiagnosticoutput: 1:99
spamdiagnosticmetadata: NSPM
Content-Type: multipart/alternative; boundary="_000_AM6PR07MB5621629CDF15C89D5AE048E893AF0AM6PR07MB5621eurp_"
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-Network-Message-Id: 930080dd-f0b8-448c-39aa-08d659b6c86c
X-MS-Exchange-CrossTenant-originalarrivaltime: 04 Dec 2018 07:04:49.9698 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 92e84ceb-fbfd-47ab-be52-080c6b87953f
X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM6PR07MB4486
X-OriginatorOrg: ericsson.com
X-Brightmail-Tracker: H4sIAAAAAAAAA03Se0hTYRQA8L7du+26nNzmzIORxAoWvhWlS4UlQdgfiRU9GItceX3gnOte s+wv0bLa8pE0muIjc2mYiqmlomSK/pFhWmHGEJlMEDXznaU1c34L/O93vnO+c84HH0XI2oU+ VLIuneV0Gq1CJCGLL7XeDPRSitQh5fn+TOffZ2LGVGMnGfOQiWDqHffEx8loi+W3ILqkfYKM tj7WxxIqydF4VpucwXLBkXGSpOW8N4S+vhXd6jYWEFnoUQUyIDcK6HCY7OgTOy2jexH0NnoY kGTTKwiyFuZEOKgSgMGQI3YGJF1IQHNHqRhnCgSQa1glcTCO4O67UoEBUZSIZsDo8Hf2ldMX oHF9YWseQUdCn3lE4LTnpsu+3CdxzTHoNmULsVUw09RIOE3SB6ClroZwtpTSavhgO4hHVYrA fMeyVe9GX4Zek23LiN4Nq/11AjzLG6wTFQL8ThosnYMEthdM2R1CbAYGcm1i7L3wucKInAOA /ioCe/WiCCcCYd5kcl0+DV25xUJcNITAkd3i6uQHJdZ+Md5CA121NteFFPjTVuayL9TmjZPY gwRsrCUUouCSbctip0H9RumWpfQueF88QeLzEJj7WEFg+0N15YzLwfBqeQBtP3+KxLXIi2f5 q6mJYWFBLJd8jefTdEE6Nr0JbX6o7pb1kDY0NRnVg2gKKdylM7+EaplQk8FnpvYgoAiFXKq/ Tqpl0nhN5m2WS7vC3dCyfA/aQ5EKbykT06yS0YmadDaFZfUs9z8roNx8shC3eHawINaTieic 3hEzrPQkD0ny15hTLze0ad9OKqeT2lamRC/Kx0aGHxZdXPcea9nfgPZV9k2nn4krKRpTLr3N CucCgqJWf+QFUFVhliOS2VEPFfP6sHx+OdS6EtRw3pzTNfNTovWJefAp9Nzs8xOj+icJEaRR 7st5LH3faXdXkHySJtSP4HjNP/PFuFZMAwAA
Archived-At: <https://mailarchive.ietf.org/arch/msg/rtcweb/2FPDjq6pO0V29HayXOTOBxwIJWU>
Subject: Re: [rtcweb] [MMUSIC] Default proto transport in JSEP
X-BeenThere: rtcweb@ietf.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: Real-Time Communication in WEB-browsers working group list <rtcweb.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/rtcweb>, <mailto:rtcweb-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/rtcweb/>
List-Post: <mailto:rtcweb@ietf.org>
List-Help: <mailto:rtcweb-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/rtcweb>, <mailto:rtcweb-request@ietf.org?subject=subscribe>
X-List-Received-Date: Tue, 04 Dec 2018 07:05:06 -0000

Hi,

Also, as far as I understand, JSEP only defines an API, not a network protocol, so if you want to be standards compliant on the wire the JS app would have to modify the SDP before sending it to the network, and I do not think we want that.

Regards,

Christer

________________________________
From: rtcweb <rtcweb-bounces@ietf.org> on behalf of Christer Holmberg <christer.holmberg@ericsson.com>
Sent: Monday, December 3, 2018 11:01 PM
To: Roman Shpount; Adam Roach - SIPCORE Chair
Cc: RTCWeb IETF; mmusic WG
Subject: Re: [rtcweb] [MMUSIC] Default proto transport in JSEP


Hi,


In addition, even from a pure WebRTC perspective, there is no guarantee that both endpoints are implementing JSEP, so any WebRTC-specific ICE modification would probably have to be done in a separate document.


Regards,


Christer


________________________________
From: rtcweb <rtcweb-bounces@ietf.org> on behalf of Christer Holmberg <christer.holmberg@ericsson.com>
Sent: Monday, December 3, 2018 10:58 PM
To: Roman Shpount; Adam Roach - SIPCORE Chair
Cc: RTCWeb IETF; mmusic WG
Subject: Re: [rtcweb] [MMUSIC] Default proto transport in JSEP



Hi,

I am jumping late into the circle, but I share Roman's concern regarding the suggested change to 5.2.2. I think the WHOLE m= line (including the protocol) shall match the currently used properties. You may also end up having SDP attributes that are not defined the for the indicated protocol (since you are using another protocol). That is asking for interoperability problems, in my opinion.

I've probably missed it, but what is the "catastrophic flaw" (to use Adam's words) that would require this change, and misalign the JSEP ICE procedures from all the documents Roman listed?

Regards,

Christer

________________________________
From: rtcweb <rtcweb-bounces@ietf.org> on behalf of Roman Shpount <roman@telurix.com>
Sent: Wednesday, November 28, 2018 8:41 PM
To: Adam Roach - SIPCORE Chair
Cc: RTCWeb IETF; mmusic WG
Subject: Re: [rtcweb] [MMUSIC] Default proto transport in JSEP


Hi Adam,

On Wed, Nov 28, 2018 at 1:22 PM Adam Roach <adam@nostrum.com<mailto:adam@nostrum.com>> wrote:

On 11/28/18 10:57 AM, Roman Shpount wrote:
On Wed, Nov 28, 2018 at 11:38 AM Cullen Jennings <fluffy@iii.ca<mailto:fluffy@iii.ca>> wrote:
On Nov 27, 2018, at 4:46 PM, Roman Shpount <roman@telurix.com<mailto:roman@telurix.com>> wrote:

 I suggest to update JSEP section 5.1.2 to match the rest of the documents to say that "UDP/TLS/RTP/SAVPF" proto MUST be used during ICE restart. When ICE restart is not in progress, "UDP/TLS/RTP/SAVPF" proto MUST be used if default (only) candidate is a UDP candidate and "TCP/TLS/RTP/SAVPF" proto MUST be used if default (only) candidate is TCP candidate.

I don’t see any real befits to implementations to this change and I don’t think the rtcweb consensus was around the currently solution. Do you see some advantage to implementations to this?

This is what every other document related to ICE, including JSEP section 5.2.2 currently specifies. It was also consensus in MMUSIC. I think RTCWEB need a really good reason why it needs to be different.

It would probably help clarify things if you quoted the parts of the document that you think are in conflict. I can't find any explicit <proto> field handling in 5.2.2.

 I have mentioned this already in the previous message, but I guess this got lost in the traffic.

JSEP-25 in section 5.2.2 says (https://tools.ietf.org/html/draft-ietf-rtcweb-jsep-25#section-5.2.2):

Each "m=" and c=" line MUST be filled in with the port, protocol, and address of the default candidate for the m= section, as described in [I-D.ietf-mmusic-ice-sip-sdp], Section 3.2.1.2.

At the same time section 5.1.2 says (https://tools..ietf.org/html/draft-ietf-rtcweb-jsep-25#section-5.1.2<https://tools.ietf.org/html/draft-ietf-rtcweb-jsep-25#section-5.1.2>)2>):

For media m= sections, JSEP implementations MUST support the "UDP/TLS/RTP/SAVPF" profile specified in [RFC5764], and MUST indicate this profile for each media m= line they produce in an offer. For data m= sections, implementations MUST support the "UDP/DTLS/SCTP" profile and MUST indicate this profile for each data m= line they produce in an offer.

So, section 5.2.2 says m= line should be filled with currently used protocol, which means "TCP/TLS/RTP/SAVPF" or "TCP/DTLS/SCTP" if default candidate is TCP based, but section 5.1.2 says it must be "UDP/TLS/RTP/SAVPF" or "UDP/DTLS/SCTP", even if default candidate is TCP based. I thought that section 5.2.2, since it is more specific, overwrites 5.1.2, which I assumed only applies to ICE restart. Authors disagree and want to update the document.

In terms of changing technical aspects of JSEP: the only reason the document is out of the RFC Editor's queue right now is to address issues arising from rationalizing the reference to RFC 8445 within Cluster 238. This is not an opportunity to re-litigate previously settled consensus decisions. Technical issues such as the one at hand should have been raised during WG development, WG last call, or -- in extremis, since you're a regular RTCWEB participant -- during IETF last call. It's up to the chairs what to allow, but I wouldn't expect anything other than catastrophic flaws to be open for change at this time.

I am not the one who opened this can of worms. I am fine if the current draft version is not changed. This is why I did not comment during the WG last call. Draft authors are introducing the new change in https://github.com/rtcweb-wg/jsep/pull/857, which makes JSEP incompatible with ice-sip-sdp. I oppose this change. If the group considers that a change to clarify things is necessary, I would suggest that section 5.1.2 should be changed instead to that it only applies during ICE restart, so that JSEP is compatible with ice-sip-sdp.

Regards,
______________
Roman Shpount