Re: [tram] I-D Action: draft-ietf-tram-turnbis-18.txt

Justin Uberti <juberti@google.com> Fri, 01 June 2018 23:57 UTC

Return-Path: <juberti@google.com>
X-Original-To: tram@ietfa.amsl.com
Delivered-To: tram@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 14BBE12E050 for <tram@ietfa.amsl.com>; Fri, 1 Jun 2018 16:57:05 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -18.21
X-Spam-Level:
X-Spam-Status: No, score=-18.21 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, ENV_AND_HDR_SPF_MATCH=-0.5, HTML_MESSAGE=0.001, RCVD_IN_DNSWL_LOW=-0.7, SPF_PASS=-0.001, T_DKIMWL_WL_MED=-0.01, USER_IN_DEF_DKIM_WL=-7.5, USER_IN_DEF_SPF_WL=-7.5] 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 4r_7GypS1BRp for <tram@ietfa.amsl.com>; Fri, 1 Jun 2018 16:57:03 -0700 (PDT)
Received: from mail-it0-x234.google.com (mail-it0-x234.google.com [IPv6:2607:f8b0:4001:c0b::234]) (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 C576012E048 for <tram@ietf.org>; Fri, 1 Jun 2018 16:57:02 -0700 (PDT)
Received: by mail-it0-x234.google.com with SMTP id c3-v6so3858793itj.4 for <tram@ietf.org>; Fri, 01 Jun 2018 16:57:02 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20161025; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=MQF6qODNhrh3+AzTKy701bG+XUmRqzmdpacv3oeTsc0=; b=BUiHO6nAYCtskyuwrQn4bcx6QHE8KWOIZT1FgM/OzAvmK5LGqpvuzQ8zAZ6p/7tGVf XJroBimRH4Ow5iA26XxTZ46Jc7jyT9q9boiZTrmf0R2a4hWyAI1P5GyT0xh6qbqfdqlx 0sGmPeSOuG7LfSe2O9r1ja8wd1PJKalwfxW7gNY9QJ91dHm9C/Lb3TYGuiaCoIbhZbM+ u5OyuxwUpRSGuVogVdowz3iyjMip5ExBRA5pLeoOy2P/u62DMK1fMcohzZGOlohjNmgS eLDc7saigxdxEu9gJykD45Uxk/HXmpYRTmHDaK5AWqHCGp0YBBBmUSBoKzhOEABfRnf1 ljPw==
X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=MQF6qODNhrh3+AzTKy701bG+XUmRqzmdpacv3oeTsc0=; b=OLBpmYfWr7MjQ0vJGCxsRsdli5ix+59xM8ZyiLUXInzXAJkYrBM/pifZRK6+1878sg j6kDvOpp3Qy7zzbHhYNd5O7axixgvjZbaIeVspYUhtjeYCx4As0FeQW7Av3fTnMhTDmZ yyVmG8evSrrOZb+qlNVx5N1C6NIJbEZDK7gydKJmnYqYnvfAPFiu0HVXHy/S/vIy6/Wn GqHx8R7P+8i8D7fyNhr00O7M922nUJCyqgtkuMGaKwmRxl2Qm9xuh4U0ZnWI6ta8MaHW XjrhrnvUw9csz4o9ycpL9aLdJYd7fKSg+TIpt9JwbiGPjYWlwtCNhHO4qSZAbyG+ZFOo /C4g==
X-Gm-Message-State: APt69E3mLGnPEP/k3mV9Q9Dg+bumo3ZKcwSWDsikpqIiXLnfOj5ejAWy Ocz7x+Y6tmARzvBS9/FB2zjX438UjiuQIpJ7T42nJg==
X-Google-Smtp-Source: ADUXVKKY0Xg0u0UvqllTBUfAJ67bB7HK8X54MvbTPuFm8DIJBXtuPqgmWQTxu+Y82CBuYi3iELOEI6ebTvqoUQgr1D0=
X-Received: by 2002:a24:5901:: with SMTP id p1-v6mr6158214itb.119.1527897421643; Fri, 01 Jun 2018 16:57:01 -0700 (PDT)
MIME-Version: 1.0
References: <152759086979.25527.10597739101423022507@ietfa.amsl.com> <BN6PR16MB1425B25D7809C9D6B2144B65EA6D0@BN6PR16MB1425.namprd16.prod.outlook.com> <4B3DB45B-4358-43FB-A9B0-01F919E434EF@mozilla.com>
In-Reply-To: <4B3DB45B-4358-43FB-A9B0-01F919E434EF@mozilla.com>
From: Justin Uberti <juberti@google.com>
Date: Fri, 1 Jun 2018 16:56:50 -0700
Message-ID: <CAOJ7v-36KAZXcedsOQS1aU8spJa0V5FrAh=xXCUmOZFKg4=FRw@mail.gmail.com>
To: Nils Ohlmeier <nohlmeier@mozilla.com>
Cc: tram@ietf.org
Content-Type: multipart/alternative; boundary="00000000000012ca36056d9d576d"
Archived-At: <https://mailarchive.ietf.org/arch/msg/tram/JQhxtpOCWBk5ZptXEbrk4P3rgf8>
Subject: Re: [tram] I-D Action: draft-ietf-tram-turnbis-18.txt
X-BeenThere: tram@ietf.org
X-Mailman-Version: 2.1.22
Precedence: list
List-Id: "Discussing the creation of a Turn Revised And Modernized \(TRAM\) WG, which goal is to consolidate the various initiatives to update TURN and STUN." <tram.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/tram>, <mailto:tram-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/tram/>
List-Post: <mailto:tram@ietf.org>
List-Help: <mailto:tram-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/tram>, <mailto:tram-request@ietf.org?subject=subscribe>
X-List-Received-Date: Fri, 01 Jun 2018 23:57:05 -0000

I reviewed the diff as well and have the following comments:
- In the initial discussion of allowing STUN packets, it mentions that STUN
checks can occur while the client is preparing permission requests. I think
this could be worded more clearly, e.g.
"... addressing the race condition that exists when the remote peer sends
connectivity checks before the local client has had a chance to create
permissions in the TURN server for the remote peer IP addresses."

- Nit: In Section 19, 'atrributes' typo

On Fri, Jun 1, 2018 at 11:47 AM Nils Ohlmeier <nohlmeier@mozilla.com> wrote:

> I reviewed the diff between version 17 and 18 and the changes to allow a
> TURN server to forward inbound STUN messages if configured so look good to
> me.
>
> Best
>   Nils Ohlmeier
>
> > On May 29, 2018, at 03:52, Konda, Tirumaleswar Reddy
> <TirumaleswarReddy_Konda@McAfee.com> wrote:
> >
> > This revision incorporates editorial changes proposed by Brandon to
> allow the TURN server to forward inbound STUN connectivity checks without
> explicit permissions.
> > More comments and suggestions are welcome.
> >
> > Cheers,
> > -Tiru
> >
> >> -----Original Message-----
> >> From: tram [mailto:tram-bounces@ietf.org] On Behalf Of internet-
> >> drafts@ietf.org
> >> Sent: Tuesday, May 29, 2018 4:18 PM
> >> To: i-d-announce@ietf.org
> >> Cc: tram@ietf.org
> >> Subject: [tram] I-D Action: draft-ietf-tram-turnbis-18.txt
> >>
> >>
> >>
> >>
> >> A New Internet-Draft is available from the on-line Internet-Drafts
> directories.
> >> This draft is a work item of the TURN Revised and Modernized WG of the
> IETF.
> >>
> >>        Title           : Traversal Using Relays around NAT (TURN):
> Relay Extensions
> >> to Session Traversal Utilities for NAT (STUN)
> >>        Authors         : Tirumaleswar Reddy
> >>                          Alan Johnston
> >>                          Philip Matthews
> >>                          Jonathan Rosenberg
> >>      Filename        : draft-ietf-tram-turnbis-18.txt
> >>      Pages           : 84
> >>      Date            : 2018-05-29
> >>
> >> Abstract:
> >>   If a host is located behind a NAT, then in certain situations it can
> >>   be impossible for that host to communicate directly with other hosts
> >>   (peers).  In these situations, it is necessary for the host to use
> >>   the services of an intermediate node that acts as a communication
> >>   relay.  This specification defines a protocol, called TURN (Traversal
> >>   Using Relays around NAT), that allows the host to control the
> >>   operation of the relay and to exchange packets with its peers using
> >>   the relay.  TURN differs from some other relay control protocols in
> >>   that it allows a client to communicate with multiple peers using a
> >>   single relay address.
> >>
> >>   The TURN protocol was designed to be used as part of the ICE
> >>   (Interactive Connectivity Establishment) approach to NAT traversal,
> >>   though it also can be used without ICE.
> >>
> >>   This document obsoletes RFC 5766 and RFC 6156.
> >>
> >>
> >> The IETF datatracker status page for this draft is:
> >> https://datatracker.ietf.org/doc/draft-ietf-tram-turnbis/
> >>
> >> There are also htmlized versions available at:
> >> https://tools.ietf.org/html/draft-ietf-tram-turnbis-18
> >> https://datatracker.ietf.org/doc/html/draft-ietf-tram-turnbis-18
> >>
> >> A diff from the previous version is available at:
> >> https://www.ietf.org/rfcdiff?url2=draft-ietf-tram-turnbis-18
> >>
> >>
> >> Please note that it may take a couple of minutes from the time of
> submission
> >> until the htmlized version and diff are available at tools.ietf.org.
> >>
> >> Internet-Drafts are also available by anonymous FTP at:
> >> ftp://ftp.ietf.org/internet-drafts/
> >>
> >> _______________________________________________
> >> tram mailing list
> >> tram@ietf.org
> >> https://www.ietf.org/mailman/listinfo/tram
> >
> > _______________________________________________
> > tram mailing list
> > tram@ietf.org
> > https://www.ietf.org/mailman/listinfo/tram
>
> _______________________________________________
> tram mailing list
> tram@ietf.org
> https://www.ietf.org/mailman/listinfo/tram
>