Re: [rtcweb] Ambiguities with BUNDLE when used with ICE?
Taylor Brandstetter <deadbeef@google.com> Wed, 30 August 2017 18:44 UTC
Return-Path: <deadbeef@google.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 13878132930 for <rtcweb@ietfa.amsl.com>; Wed, 30 Aug 2017 11:44:52 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.7
X-Spam-Level:
X-Spam-Status: No, score=-2.7 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, HTML_MESSAGE=0.001, RCVD_IN_DNSWL_LOW=-0.7, SPF_PASS=-0.001] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (2048-bit key) header.d=google.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 QG5rRxfd70MB for <rtcweb@ietfa.amsl.com>; Wed, 30 Aug 2017 11:44:47 -0700 (PDT)
Received: from mail-wm0-x229.google.com (mail-wm0-x229.google.com [IPv6:2a00:1450:400c:c09::229]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id D97F91326EA for <rtcweb@ietf.org>; Wed, 30 Aug 2017 11:44:46 -0700 (PDT)
Received: by mail-wm0-x229.google.com with SMTP id u26so15979917wma.0 for <rtcweb@ietf.org>; Wed, 30 Aug 2017 11:44:46 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20161025; h=mime-version:in-reply-to:references:from:date:message-id:subject:to :cc; bh=FNX+Ov/H7+GFy9wNj/DYMIM3Zk06Ujpt3sMBD23ZRfc=; b=EGaJgOm/3fqcg44yAnuqftH8uU988okaxBua8CD2fCKaW6bwtU0IsFePVZbE53q2cp AFdq0pywKid+JyO9KrSTokzwDOAiHTaiQzZeyA4+twjxInwfrpe4GR7IQHTuuLbm1OMX OG5DSAF4o0uwg1SKO0krpHKXuxvlsT8Z5MTr2xzFZJL/e/ZpfQ0RFoPOD5qN22eChp91 YOTIzwnjywc3XLufQWv/0TfwacX9ilNiwKe36lFyhmwJYjd+CPJQ7/qatiAVDjyP8ssl 8sD6h7KXfkyvvirDjjyzEEZldNUMNopDayDerhwFKQKPlAEO+2xhXh9w3I7G4p6b2cEe zXxQ==
X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:in-reply-to:references:from:date :message-id:subject:to:cc; bh=FNX+Ov/H7+GFy9wNj/DYMIM3Zk06Ujpt3sMBD23ZRfc=; b=UfCNgJrfw8YyCf2/g8AxWL9YW90/g4JBVwbRuMRulOdwaUw6Zs+amzPrtxGL9eelxK vpJh1uOzVvdhmUpp/d7+QL3KpJ4sasu3OLOyvPNeTvBt1h9fwSEfiKtggLXvXmvYJ37c oyaNXLlM+uWqEBL7blAuEjXtHb6BGdeixgGCkFmruByFGYAd8S9Ecz0HMiPbNU11/4hu GrCPz8S69HeRXtxRTzgtYq4ohsbtZRsWjspK3k29+SMb77xr7A+cWedPO8brpaYvg+xF CmtKgvkTbvtGcEjbNxJSPUgqgjTYN4nrf4Zh1FxpCsRSbmAm4i3sdxc2jM/8uClUvMia OEQg==
X-Gm-Message-State: AHYfb5h1w5uUIHV2xVOn11xWuyNaoTdMYGMf8kPcKp2n/1+pyOnY64Io OSfXb3aO4zD27ltm/80bsI4wDW4YJkCj
X-Google-Smtp-Source: ADKCNb4vNbN2PdSxCv5tWLFHDigFPkZ0306BhaiRSnuaHLY4KIZnK0qz352UC3VyukTQPMZJG8yELXN1vDaZcRPHDAs=
X-Received: by 10.28.16.133 with SMTP id 127mr1677699wmq.75.1504118685080; Wed, 30 Aug 2017 11:44:45 -0700 (PDT)
MIME-Version: 1.0
Received: by 10.223.154.55 with HTTP; Wed, 30 Aug 2017 11:44:44 -0700 (PDT)
In-Reply-To: <7594FB04B1934943A5C02806D1A2204B4CD0B071@ESESSMB109.ericsson.se>
References: <CAK35n0ZvBiQ935y3TpEtP83bXMW8kVu_d=jygEbdwHB+Ax0+Tg@mail.gmail.com> <0567f978-4fd6-11b5-1a32-afda996a4cd3@mozilla.com> <D5CC6C34.2093A%christer.holmberg@ericsson.com> <325d3a82-eee9-aad4-5f60-c726b2cd9de5@mozilla.com> <D5CC9711.20AF8%christer.holmberg@ericsson.com> <1e0d3ba2-c8a9-59bd-d94e-38bd68ce0235@mozilla.com> <7594FB04B1934943A5C02806D1A2204B4CD0AFA1@ESESSMB109.ericsson.se> <7594FB04B1934943A5C02806D1A2204B4CD0B071@ESESSMB109.ericsson.se>
From: Taylor Brandstetter <deadbeef@google.com>
Date: Wed, 30 Aug 2017 11:44:44 -0700
Message-ID: <CAK35n0bd+qoMBn0_mbVKCAn9boREwz=y42Pf+anJjN7uvGcubQ@mail.gmail.com>
To: Christer Holmberg <christer.holmberg@ericsson.com>
Cc: Byron Campen <bcampen@mozilla.com>, RTCWeb IETF <rtcweb@ietf.org>
Content-Type: multipart/alternative; boundary="001a1146d950ed15010557fceb98"
Archived-At: <https://mailarchive.ietf.org/arch/msg/rtcweb/7pkP9S1yKK7krjl1Tib7Ac5xZmU>
Subject: Re: [rtcweb] Ambiguities with BUNDLE when used with ICE?
X-BeenThere: rtcweb@ietf.org
X-Mailman-Version: 2.1.22
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: Wed, 30 Aug 2017 18:44:52 -0000
> > Maybe I¹m missing something, but I fail to see what the problem is. > If the BUNDLE-tag section is changed (e.g., because the current BUNDLE-tag > section m- line is rejected) you simply include the TRANSPORT/IDENTICAL > category attributes in the new BUNDLE-tag section m- line. The issue is that it may not be clear whether the offerer intended for the same ICE session to be used, or a new ICE session that happens to use the same attributes (ufrag/password). Another option might be to define an "a=ice-id" just like we have an > "a=dtls-id". Or it could be a generic "a=transport-id", so it would work with any other protocol that may have this problem with BUNDLE. On Wed, Aug 30, 2017 at 7:36 AM, Christer Holmberg < christer.holmberg@ericsson.com> wrote: > Hi, > > Here is the GitHub issue where me and Taylor discussed this: > > https://github.com/cdh4u/draft-sdp-bundle/issues/26 > > The outcome was that we need to say "something", but there is not more > than that, so feel free to suggest :) > > Regards, > > Christer > > -----Original Message----- > From: rtcweb [mailto:rtcweb-bounces@ietf.org] On Behalf Of Christer > Holmberg > Sent: 30 August 2017 16:27 > To: Byron Campen <bcampen@mozilla.com>; Taylor Brandstetter < > deadbeef@google.com>; RTCWeb IETF <rtcweb@ietf.org> > Subject: Re: [rtcweb] Ambiguities with BUNDLE when used with ICE? > > Hi, > > >>> Alright, if we want to write a rule that the transport follows > >>> the bundle, what about this? > >>> > >>> a=group:BUNDLE mid1 mid2 > >>> a=group:BUNDLE mid3 mid4 > >>> > >>> and in a reoffer > >>> > >>> a=group:BUNDLE mid2 mid3 > >>> a=group:BUNDLE mid4 mid1 > >>> > >>> What transport goes where? > >> > >> I think Taylor raised this issue earlier. What you are doing above is > >> moving streams from one BUNDLE group to another, and mixing the > >> BUNDLE groups up, and I don’t think that’s a good idea. > > > > If you want to classify this as a bad idea, we need normative language > > forbidding it. For instance, a rule that once a mid is placed in a > bundle, the only way to remove it is to disable it, as well as a rule to > handle the following: > > > > a=group:BUNDLE mid2 mid3 > > a=mid: mid1 <--- not bundled > >... > >a=mid: mid2 > >... > >a=mid: mid3 > > > >and in a reoffer > > > >a=group mid1 mid2 mid3 > > > > We would need to define which transport is retained; mid1's, or the > bundle's. Or perhaps a rule forbidding previously unbundled mids to be > added to a bundle. > > Me and Taylor did discuss some restriction text. I'll dig in my achieve. > > >There are lots of corner cases here that simply aren't specified right > >now. In the non-trickle-ICE case, this is ok, because the offerer > unambiguously signals what transports it uses where in the c= line. > > I think that's a separate problem. > > I do agree that the BUNDLE group transport is currently bound to the c/m- > line. If ICE is used, we should probably use candidate information instead. > > Regards, > > Christer > > > > >> Best regards, > >> Byron Campen > >> > >> On 8/30/17 5:18 AM, Christer Holmberg wrote: > >>> Hi, > >>> > >>> Maybe I¹m missing something, but I fail to see what the problem is. > >>> > >>> If the BUNDLE-tag section is changed (e.g., because the current > >>> BUNDLE-tag section m- line is rejected) you simply include the > >>> TRANSPORT/IDENTICAL category attributes in the new BUNDLE-tag > >>> section m- line. > >>> > >>> Regards, > >>> > >>> Christer > >>> > >>> > >>> > >>> > >>> > >>> On 29/08/17 19:45, "rtcweb on behalf of Byron Campen" > >>> <rtcweb-bounces@ietf.org on behalf of bcampen@mozilla.com> wrote: > >>> > >>>> On 8/29/17 11:35 AM, Taylor Brandstetter wrote: > >>>>> Some ideas for how to resolve it: > >>>>> > >>>>> 1. Require ufrags to be unique for different ICE sessions (JSEP > >>>>> already requires this), and use the ufrag as the unique identifier. > >>>>> Then the BUNDLE semantics could be used as-is, with "unique/shared > >>>>> ufrag" replacing "unique/shared address". > >>>> I think this is our best option. It should allow trickle ICE > >>>> to work as well as anything else. We might still want to do some of > >>>> the below in addition. > >>>> > >>>>> 2. For ICE, instead of using a "shared address" to mean > >>>>> "guaranteed to use the existing BUNDLE transport", use > >>>>> "a=bundle-only" in subsequent offers. If we were to do this, we > >>>>> may also need to allow the offerer BUNDLE-tag section to be > >>>>> rejected; that's currently not supported (https://github.com/cdh4u/ > draft-sdp-bundle/issues/22). > >>>> This does not help clarify things when the offerer changes > >>>> the BUNDLE tag (disable, remove from BUNDLE, make something else > >>>> the BUNDLE-tag, etc). > >>>> > >>>>> 3. Since the "shared address" rules say "don't associate > >>>>> TRANSPORT/IDENTICAL category attributes with m= sections other > >>>>> than the offerer BUNDLE-tag section", we could interpret "m= > >>>>> section has no transport attributes of its own" as "using shared > address". > >>>> I don't think this helps when the offerer changes the > >>>> BUNDLE-tag either. > >>>> > >>>> Best regards, > >>>> Byron Campen > >>>> > >>>> _______________________________________________ > >>>> rtcweb mailing list > >>>> rtcweb@ietf.org > >>>> https://www.ietf.org/mailman/listinfo/rtcweb > >> > > _______________________________________________ > rtcweb mailing list > rtcweb@ietf.org > https://www.ietf.org/mailman/listinfo/rtcweb >
- [rtcweb] Ambiguities with BUNDLE when used with I… Taylor Brandstetter
- Re: [rtcweb] Ambiguities with BUNDLE when used wi… Byron Campen
- Re: [rtcweb] Ambiguities with BUNDLE when used wi… Peter Thatcher
- Re: [rtcweb] Ambiguities with BUNDLE when used wi… Christer Holmberg
- Re: [rtcweb] Ambiguities with BUNDLE when used wi… Christer Holmberg
- Re: [rtcweb] Ambiguities with BUNDLE when used wi… Byron Campen
- Re: [rtcweb] Ambiguities with BUNDLE when used wi… Christer Holmberg
- Re: [rtcweb] Ambiguities with BUNDLE when used wi… Byron Campen
- Re: [rtcweb] Ambiguities with BUNDLE when used wi… Christer Holmberg
- Re: [rtcweb] Ambiguities with BUNDLE when used wi… Christer Holmberg
- Re: [rtcweb] Ambiguities with BUNDLE when used wi… Taylor Brandstetter
- Re: [rtcweb] Ambiguities with BUNDLE when used wi… Christer Holmberg
- Re: [rtcweb] Ambiguities with BUNDLE when used wi… Justin Uberti
- Re: [rtcweb] Ambiguities with BUNDLE when used wi… Sergio Garcia Murillo
- Re: [rtcweb] Ambiguities with BUNDLE when used wi… Peter Thatcher
- Re: [rtcweb] Ambiguities with BUNDLE when used wi… Christer Holmberg
- Re: [rtcweb] Ambiguities with BUNDLE when used wi… Christer Holmberg
- Re: [rtcweb] Ambiguities with BUNDLE when used wi… Christer Holmberg
- Re: [rtcweb] Ambiguities with BUNDLE when used wi… Byron Campen
- Re: [rtcweb] Ambiguities with BUNDLE when used wi… Christer Holmberg
- Re: [rtcweb] Ambiguities with BUNDLE when used wi… Byron Campen
- Re: [rtcweb] Ambiguities with BUNDLE when used wi… Christer Holmberg
- Re: [rtcweb] Ambiguities with BUNDLE when used wi… Byron Campen
- Re: [rtcweb] Ambiguities with BUNDLE when used wi… Cullen Jennings
- Re: [rtcweb] Ambiguities with BUNDLE when used wi… Christer Holmberg
- Re: [rtcweb] Ambiguities with BUNDLE when used wi… Byron Campen
- Re: [rtcweb] Ambiguities with BUNDLE when used wi… Christer Holmberg
- Re: [rtcweb] Ambiguities with BUNDLE when used wi… Byron Campen
- Re: [rtcweb] Ambiguities with BUNDLE when used wi… Christer Holmberg
- Re: [rtcweb] Ambiguities with BUNDLE when used wi… Byron Campen
- Re: [rtcweb] Ambiguities with BUNDLE when used wi… Christer Holmberg