Return-Path: <tireddy@cisco.com>
X-Original-To: secdir@ietfa.amsl.com
Delivered-To: secdir@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1])
 by ietfa.amsl.com (Postfix) with ESMTP id A59F412D74E;
 Mon,  8 Aug 2016 21:11:01 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -15.768
X-Spam-Level: 
X-Spam-Status: No, score=-15.768 tagged_above=-999 required=5
 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1,
 DKIM_VALID_AU=-0.1, RCVD_IN_DNSWL_HI=-5, RCVD_IN_MSPIKE_H3=-0.01,
 RCVD_IN_MSPIKE_WL=-0.01, RP_MATCHES_RCVD=-1.247, SPF_PASS=-0.001,
 USER_IN_DEF_DKIM_WL=-7.5] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (1024-bit key)
 header.d=cisco.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 Wf0wTiNC3nTk; Mon,  8 Aug 2016 21:11:00 -0700 (PDT)
Received: from rcdn-iport-5.cisco.com (rcdn-iport-5.cisco.com [173.37.86.76])
 (using TLSv1.2 with cipher DHE-RSA-SEED-SHA (128/128 bits))
 (No client certificate requested)
 by ietfa.amsl.com (Postfix) with ESMTPS id C8F2312D728;
 Mon,  8 Aug 2016 21:10:59 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple;
 d=cisco.com; i=@cisco.com; l=2294; q=dns/txt; s=iport;
 t=1470715859; x=1471925459;
 h=from:to:subject:date:message-id:references:in-reply-to:
 content-transfer-encoding:mime-version;
 bh=qDAfKxp2y0ielNRG42U4yQVt/adqE4mL0vY2W4I+BkY=;
 b=NnLn6Uo+RsQWfuTlza+FcjK7kYvwaMX94/F2SfN5aCh7nDaGB/c94NBg
 EMufxYErQMa293WAkb9ULCLdvyOcvyiVtxtpJgvX7M1zOi/hhLl0JeM5L
 nb+BgZeDCUHqjbu70BzBMNQN3famf3HTfKTd3nNXHCmcjBJKrJTTcHocm Y=;
X-IronPort-Anti-Spam-Filtered: true
X-IronPort-Anti-Spam-Result: =?us-ascii?q?A0AQAgA5V6lX/5pdJa1dg0WBUge5GIF9h?=
 =?us-ascii?q?h0CgUU4FAEBAQEBAQFdJ4ReAQEEATpEBwQCAQgRBAEBHwkHMhQJCAEBBAESCIg?=
 =?us-ascii?q?hCMMqAQEBAQEBAQEBAQEBAQEBAQEBAQEBHIYqhE2EKoVxBZN1hUQBjwKBcoRbi?=
 =?us-ascii?q?H2MNIN3AR42g3puhXd/AQEB?=
X-IronPort-AV: E=Sophos;i="5.28,493,1464652800"; d="scan'208";a="135733727"
Received: from rcdn-core-3.cisco.com ([173.37.93.154])
 by rcdn-iport-5.cisco.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384;
 09 Aug 2016 04:10:58 +0000
Received: from XCH-RCD-020.cisco.com (xch-rcd-020.cisco.com [173.37.102.30])
 by rcdn-core-3.cisco.com (8.14.5/8.14.5) with ESMTP id u794Awas032389
 (version=TLSv1/SSLv3 cipher=AES256-SHA bits=256 verify=FAIL);
 Tue, 9 Aug 2016 04:10:59 GMT
Received: from xch-rcd-017.cisco.com (173.37.102.27) by XCH-RCD-020.cisco.com
 (173.37.102.30) with Microsoft SMTP Server (TLS) id 15.0.1210.3;
 Mon, 8 Aug 2016 23:10:58 -0500
Received: from xch-rcd-017.cisco.com ([173.37.102.27]) by
 XCH-RCD-017.cisco.com ([173.37.102.27]) with mapi id 15.00.1210.000; Mon, 8
 Aug 2016 23:10:58 -0500
From: "Tirumaleswar Reddy (tireddy)" <tireddy@cisco.com>
To: "Waltermire, David A. (Fed)" <david.waltermire@nist.gov>,
 "secdir@ietf.org" <secdir@ietf.org>, "'iesg@ietf.org'" <iesg@ietf.org>,
 "draft-ietf-tram-turn-mobility.all@ietf.org"
 <draft-ietf-tram-turn-mobility.all@ietf.org>
Thread-Topic: Secdir review of draft-ietf-tram-turn-mobility-03
Thread-Index: AdHxlZAf0viCeDMrR7KVHzNfcwAdaAAW8uLw
Date: Tue, 9 Aug 2016 04:10:57 +0000
Message-ID: <97ae9a43544a4144aaddde0c476f73d5@XCH-RCD-017.cisco.com>
References: <MWHPR09MB14407DC07A9F24514A49754AF01B0@MWHPR09MB1440.namprd09.prod.outlook.com>
In-Reply-To: <MWHPR09MB14407DC07A9F24514A49754AF01B0@MWHPR09MB1440.namprd09.prod.outlook.com>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach: 
X-MS-TNEF-Correlator: 
x-ms-exchange-transport-fromentityheader: Hosted
x-originating-ip: [10.65.32.8]
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
Archived-At: <https://mailarchive.ietf.org/arch/msg/secdir/_mMOkyubXwNE73A-h4BpAXAMtTM>
Subject: Re: [secdir] Secdir review of draft-ietf-tram-turn-mobility-03
X-BeenThere: secdir@ietf.org
X-Mailman-Version: 2.1.17
Precedence: list
List-Id: Security Area Directorate <secdir.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/secdir>,
 <mailto:secdir-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/secdir/>
List-Post: <mailto:secdir@ietf.org>
List-Help: <mailto:secdir-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/secdir>,
 <mailto:secdir-request@ietf.org?subject=subscribe>
X-List-Received-Date: Tue, 09 Aug 2016 04:11:01 -0000

Thanks for the review. Please see inline

> -----Original Message-----
> From: Waltermire, David A. (Fed) [mailto:david.waltermire@nist.gov]
> Sent: Monday, August 8, 2016 10:26 PM
> To: secdir@ietf.org; 'iesg@ietf.org' <iesg@ietf.org>; draft-ietf-tram-tur=
n-
> mobility.all@ietf.org
> Subject: Secdir review of draft-ietf-tram-turn-mobility-03
>=20
> I have reviewed this document as part of the security directorate's ongoi=
ng
> effort to review all IETF documents being processed by the IESG.  These
> comments were written primarily for the benefit of the security area
> directors.  Document editors and WG chairs should treat these comments ju=
st
> like any other last call comments.
>=20
> Summary: ready with nits.
>=20
> This standards track draft describes a mechanism for a Traversal Using Re=
lays
> around NAT (TURN) client to re-associate with a TURN server after the cli=
ents
> IP address and/or port changes allowing previous allocations to be kept. =
This
> helps to support IP address mobility in a way that is transparent and sea=
mless
> to remote peers.
>=20
> I found that the draft clearly articulates the problem it is trying to so=
lve. The
> security considerations seem to be appropriate for the draft.
>=20
> The following are minor nits and editorial issues with the draft that wou=
ld be
> good to address before progressing the draft:
>=20
> In section 1, second paragraph, STUN should be spelled out on its first u=
se=20

Updated.

> and
> an informative reference to RFC 7635 should be included.

Informative reference to RFC 7635 is already included in the draft.

>=20
> In section 2, there is an extra space s/[RFC5245] , and the/[RFC5245], an=
d
> the/. Similar issues exist throughout the document which also need to be
> fixed.

Thanks, fixed.

>=20
> The phase "TBD (Mobility Forbidden)" is used in section 3.1.4 and in othe=
r
> parts of the document as a placeholder for the 405 Mobility Forbidden STU=
N
> Error Code requested in the IANA considerations. While the actions to be
> taken by IANA are clear, the TBD placeholders should be filled in with wh=
at is
> expected to be assigned by IANA before the draft progresses.

Yes.

-Tiru

>=20
> Regards,
> Dave Waltermire

