Re: [secdir] Secdir review of draft-ietf-tram-turn-mobility-03

"Tirumaleswar Reddy (tireddy)" <tireddy@cisco.com> Tue, 09 August 2016 04:11 UTC

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>rg>; draft-ietf-tram-turn-
> mobility.all@ietf.org
> Subject: Secdir review of draft-ietf-tram-turn-mobility-03
> 
> I have reviewed this document as part of the security directorate's ongoing
> 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 just
> like any other last call comments.
> 
> Summary: ready with nits.
> 
> This standards track draft describes a mechanism for a Traversal Using Relays
> around NAT (TURN) client to re-associate with a TURN server after the clients
> 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 seamless
> to remote peers.
> 
> I found that the draft clearly articulates the problem it is trying to solve. The
> security considerations seem to be appropriate for the draft.
> 
> The following are minor nits and editorial issues with the draft that would be
> good to address before progressing the draft:
> 
> In section 1, second paragraph, STUN should be spelled out on its first use 

Updated.

> and
> an informative reference to RFC 7635 should be included.

Informative reference to RFC 7635 is already included in the draft.

> 
> In section 2, there is an extra space s/[RFC5245] , and the/[RFC5245], and
> the/. Similar issues exist throughout the document which also need to be
> fixed.

Thanks, fixed.

> 
> The phase "TBD (Mobility Forbidden)" is used in section 3.1.4 and in other
> parts of the document as a placeholder for the 405 Mobility Forbidden STUN
> 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 what is
> expected to be assigned by IANA before the draft progresses.

Yes.

-Tiru

> 
> Regards,
> Dave Waltermire