[Gen-art] Gen-ART review of draft-ietf-tram-stun-origin-05

"Black, David" <david.black@emc.com> Sat, 21 March 2015 03:12 UTC

Return-Path: <david.black@emc.com>
X-Original-To: gen-art@ietfa.amsl.com
Delivered-To: gen-art@ietfa.amsl.com
Received: from localhost (ietfa.amsl.com [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id AD1D91A8904; Fri, 20 Mar 2015 20:12:42 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -4.311
X-Spam-Level:
X-Spam-Status: No, score=-4.311 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_MED=-2.3, SPF_PASS=-0.001, T_RP_MATCHES_RCVD=-0.01] autolearn=ham
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 45VWdpTbs6SW; Fri, 20 Mar 2015 20:12:41 -0700 (PDT)
Received: from mailuogwhop.emc.com (mailuogwhop.emc.com [168.159.213.141]) (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 03E591A8900; Fri, 20 Mar 2015 20:12:40 -0700 (PDT)
Received: from maildlpprd05.lss.emc.com (maildlpprd05.lss.emc.com [10.253.24.37]) by mailuogwprd01.lss.emc.com (Sentrion-MTA-4.3.1/Sentrion-MTA-4.3.0) with ESMTP id t2L3CX1n021044 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NO); Fri, 20 Mar 2015 23:12:36 -0400
X-DKIM: OpenDKIM Filter v2.4.3 mailuogwprd01.lss.emc.com t2L3CX1n021044
DKIM-Signature: v=1; a=rsa-sha1; c=relaxed/relaxed; d=emc.com; s=jan2013; t=1426907557; bh=PE0hOaJEYezvSzF2nygEXoFAvgM=; h=From:To:CC:Subject:Date:Message-ID:Content-Type: Content-Transfer-Encoding:MIME-Version; b=YOA5YJ14/5NvUv2VsQ6y9boJLmXG4DQTF/r/Jn8B4MEP5nXNTU7Ahx/QvM6/ESAqG +QqtfCUpV60uHMjanZ1cqPZrmaXYhdyLDowzfMhmW8LUBqj/7NVGx6GbhjW0PMXZYO wb8LEY7Mp9ndRENYJJJ2u43Y7kpbENAd95U3vbwE=
X-DKIM: OpenDKIM Filter v2.4.3 mailuogwprd01.lss.emc.com t2L3CX1n021044
Received: from mailusrhubprd02.lss.emc.com (mailusrhubprd02.lss.emc.com [10.253.24.20]) by maildlpprd05.lss.emc.com (RSA Interceptor); Fri, 20 Mar 2015 23:12:18 -0400
Received: from mxhub11.corp.emc.com (mxhub11.corp.emc.com [10.254.92.106]) by mailusrhubprd02.lss.emc.com (Sentrion-MTA-4.3.1/Sentrion-MTA-4.3.0) with ESMTP id t2L3CNVv003989 (version=TLSv1 cipher=AES128-SHA bits=128 verify=FAIL); Fri, 20 Mar 2015 23:12:24 -0400
Received: from MXHUB203.corp.emc.com (10.253.68.29) by mxhub11.corp.emc.com (10.254.92.106) with Microsoft SMTP Server (TLS) id 8.3.327.1; Fri, 20 Mar 2015 23:12:23 -0400
Received: from MX104CL02.corp.emc.com ([169.254.8.93]) by MXHUB203.corp.emc.com ([10.253.68.29]) with mapi id 14.03.0224.002; Fri, 20 Mar 2015 23:12:23 -0400
From: "Black, David" <david.black@emc.com>
To: "alan.b.johnston@gmail.com" <alan.b.johnston@gmail.com>, "justin@uberti.name" <justin@uberti.name>, "yoakum@avaya.com" <yoakum@avaya.com>, "General Area Review Team (gen-art@ietf.org)" <gen-art@ietf.org>
Thread-Topic: Gen-ART review of draft-ietf-tram-stun-origin-05
Thread-Index: AdBjhNiYXvzTSPo6RjeFsz+Pyn+RaA==
Date: Sat, 21 Mar 2015 03:12:22 +0000
Message-ID: <CE03DB3D7B45C245BCA0D2432779493641B3BE@MX104CL02.corp.emc.com>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
x-originating-ip: [10.250.48.35]
Content-Type: text/plain; charset="iso-8859-1"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-Sentrion-Hostname: mailusrhubprd02.lss.emc.com
X-RSA-Classifications: public, Resumes
Archived-At: <http://mailarchive.ietf.org/arch/msg/gen-art/k7yjOXqWezWEM-y3lC054cgBACU>
Cc: "ietf@ietf.org" <ietf@ietf.org>, "tram@ietf.org" <tram@ietf.org>
Subject: [Gen-art] Gen-ART review of draft-ietf-tram-stun-origin-05
X-BeenThere: gen-art@ietf.org
X-Mailman-Version: 2.1.15
Precedence: list
List-Id: "GEN-ART: General Area Review Team" <gen-art.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/gen-art>, <mailto:gen-art-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/gen-art/>
List-Post: <mailto:gen-art@ietf.org>
List-Help: <mailto:gen-art-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/gen-art>, <mailto:gen-art-request@ietf.org?subject=subscribe>
X-List-Received-Date: Sat, 21 Mar 2015 03:12:42 -0000

I am the assigned Gen-ART reviewer for this draft. For background on
Gen-ART, please see the FAQ at

<http://wiki.tools.ietf.org/area/gen/trac/wiki/GenArtfaq>.

Please resolve these comments along with any other Last Call comments
you may receive.

Document: draft-ietf-tram-stun-origin-05
Reviewer: David L. Black
Review Date: March 20, 2015
IETF LC End Date: March 17, 2015

Summary: This draft is on the right track, but has open issues
 		described in the review.

This draft describes the addition of a web origin attribute to STUN and
usage of that attribute in several protocol contexts.  The draft is well-
written and easy to read.  I found one minor issue which may be editorial.

Major issues: None.

Minor issues:

Section 2.7 discusses use of multiple STUN origins with Web RTC and
concludes by imposing a "MUST" requirement on use of multiple STUN
origins with HTTP in general (use first origin, ignore others).  While
Web RTC may be the predominant or only current use of STUN and TURN with
HTTP, this "MUST" could foreclose the use of STUN origins with other
uses of HTTP.  I'm not sure what those possible future uses might be,
but at a minimum this draft ought to more tightly scope its discussion
of use of STUN origins with HTTP to limit that usage to Web RTC.  If
there's a good way for a STUN or TURN server to detect Web RTC usage,
requiring STUN and TURN servers to look for Web RTC as the use of
HTTP, and only impose this "MUST" requirement if Web RTC is detected
would better align that requirement with the discussion in this draft.

Nits/editorial comments:

idnits 2.13.01 turned up a reference problem:

  == Unused Reference: 'RFC7350' is defined on line 490, but no explicit
     reference was found in the text

That RFC should be cited somewhere.  In addition, there are no RFCs cited
or referenced for TLS and DTLS - they should be added (I believe that
RFC 5246 and RFC 6347 are appropriate, respectively). 

Thanks,
--David
----------------------------------------------------
David L. Black, Distinguished Engineer
EMC Corporation, 176 South St., Hopkinton, MA  01748
+1 (508) 293-7953             FAX: +1 (508) 293-7786
david.black@emc.com        Mobile: +1 (978) 394-7754
----------------------------------------------------