IETF Logo Mail Archive

[apps-discuss] AppsDir review of draft-ietf-emu-chbind-14

Yoshiro YONEYA <yoshiro.yoneya@jprs.co.jp> Mon, 16 April 2012 09:50 UTC

Return-Path: <yoshiro.yoneya@jprs.co.jp>
X-Original-To: apps-discuss@ietfa.amsl.com
Delivered-To: apps-discuss@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 4876721F8633; Mon, 16 Apr 2012 02:50:27 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -102.599
X-Spam-Level:
X-Spam-Status: No, score=-102.599 tagged_above=-999 required=5 tests=[BAYES_00=-2.599, USER_IN_WHITELIST=-100]
Received: from mail.ietf.org ([12.22.58.30]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id FNkt0OA0LkS6; Mon, 16 Apr 2012 02:50:26 -0700 (PDT)
Received: from off-send01.tyo.jprs.co.jp (off-send01.tyo.jprs.co.jp [IPv6:2001:df0:8:17::10]) by ietfa.amsl.com (Postfix) with ESMTP id 8D82321F8650; Mon, 16 Apr 2012 02:50:26 -0700 (PDT)
Received: from off-sendsmg01.tyo.jprs.co.jp (off-sendsmg01.tyo.jprs.co.jp [172.18.8.32]) by off-send01.tyo.jprs.co.jp (8.13.8/8.13.8) with ESMTP id q3G9oOes029034; Mon, 16 Apr 2012 18:50:24 +0900
X-AuditID: ac120820-b7f4d6d000000ccc-93-4f8beb5f625a
Received: from NOTE550 (off-cpu04.tyo.jprs.co.jp [172.18.4.14]) by off-sendsmg01.tyo.jprs.co.jp (Symantec Messaging Gateway) with SMTP id 8B.7E.03276.F5BEB8F4; Mon, 16 Apr 2012 18:50:24 +0900 (JST)
Date: Mon, 16 Apr 2012 18:50:10 +0900
From: Yoshiro YONEYA <yoshiro.yoneya@jprs.co.jp>
To: apps-discuss@ietf.org, draft-ietf-emu-chbind.all@tools.ietf.org
Message-Id: <20120416185010.ee76b398.yoshiro.yoneya@jprs.co.jp>
X-Mailer: Sylpheed 3.1.3 (GTK+ 2.10.14; i686-pc-mingw32)
Mime-Version: 1.0
Content-Type: text/plain; charset="US-ASCII"
Content-Transfer-Encoding: 7bit
X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFjrFIsWRmVeSWpSXmKPExsWyRoiFTzfhdbe/wd4HTBarX65gs/i3o53Z YsaficwWXW2bWRxYPJYs+cnk8ff+O1aPL5c/swUwR3HZpKTmZJalFunbJXBlbLrfzlywX6Di 57KLjA2M53i6GDk5JARMJCZ1LmKHsMUkLtxbz9bFyMUhJHCcUeLvp2msXYwcHCwCqhK/f9iD 1LAJGEj8WvabCcQWEXCWaJ7bzwJiMwsISjS9fwVmCwsYSzw9vpEVxOYVsJe42X2VGWK+hcTy vp9sICN5ger/7hCGaNWSePjrFtQYeYntb+cwT2DknYVQNQtJ1SwkVQsYmVcxyuSnpekWp+al FOemGxjqlVTm62UVFBXrJYPoTYzgoONQ2ME445TBIUYBDkYlHl6ejG5/IdbEsuLK3EOMkhxM SqK8HC+AQnxJ+SmVGYnFGfFFpTmpxYcYJTiYlUR4Y2SBcrwpiZVVqUX5MClpDhYlcd7jZ3f4 CQmkJ5akZqemFqQWwWRlODiUJHhnvwJqFCxKTU+tSMvMKUFIM3FwggznARrO/xpkeHFBYm5x ZjpE/hSjpJQ47yGQZgGQREZpHlzvK0ZxoBeEeaeCZHmACQSu6xXQQCaggfElXSADSxIRUlIN jOsvS3z227WbR/ual8aN5uDkCVzfjXLlJd92ZTjttX/XPrP7X/9XU23rOb3J65gv7rymej7c PkRCgPX2XF0+7pvtc3ewXHdgOb1mE6vRmmW8QSlOj/3u8Db82OWVVqsiZ3+Zte+WEldPR2rI XePd0yOZt8o+YK4+WsDb3G2RaPIwKUL2e7+UEktxRqKhFnNRcSIARAAjmt0CAAA=
Cc: iesg@ietf.org
Subject: [apps-discuss] AppsDir review of draft-ietf-emu-chbind-14
X-BeenThere: apps-discuss@ietf.org
X-Mailman-Version: 2.1.12
Precedence: list
List-Id: General discussion of application-layer protocols <apps-discuss.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/apps-discuss>, <mailto:apps-discuss-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/apps-discuss>
List-Post: <mailto:apps-discuss@ietf.org>
List-Help: <mailto:apps-discuss-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/apps-discuss>, <mailto:apps-discuss-request@ietf.org?subject=subscribe>
X-List-Received-Date: Mon, 16 Apr 2012 09:50:27 -0000

I have been selected as the Applications Area Directorate reviewer for 
this draft (for background on appsdir, please see  
http://trac.tools.ietf.org/area/app/trac/wiki/ApplicationsAreaDirectorate).

Please resolve these comments along with any other Last Call comments 
you may receive. Please wait for direction from your document shepherd 
or AD before posting a new version of the draft.

Document: draft-ietf-emu-chbind-14.txt
Title: Channel Binding Support for EAP Methods
Reviewer: Yoshiro Yoneya
Review Date: 16 April 2012
IETF Last Call Date: 29 March 2012
IESG Telechat Date: 26 April 2012
Summary: This draft is almost ready for publication as an Standards Track
         RFC but has a few issues that should be fixed before publication.

Major Issues:

  Section 5.1 [Page 13]
    Arrival order of i2 and i1 to the EAP server is not specified. 
    What happens if i1 arrives before i2 arrival?  What happens if i2 
    arrives but i1 never?  Latter case could be potential DoS attack 
    to the EAP server if the authenticator is malicious.

Minor Issues:

  Section 5.3 [Page 17]
    In description of NSID: The word "RADIUS" is the first appearance in 
    this document and it should have reference to RADIUS standard.

  Section 5.3.3 [Page 18]
    The word "AVP" is the first appearance in this document and its full 
    wording should be dentoted here.

  Section 7 [Page 20]
    The word "TLV" is the first appearance in this document and its full 
    wording should be denoted here.

Nits:

  Section 3 [Page 6]
    First bullet: virtual Lads (VLANs) => virtual LANs (VLANs)

  Section 3 [Page 7]
    Second bullet: The EAP GSS-API mechanism [I-D.ietf-abfab-gss-eap] mechanism =>
                   The EAP GSS-API mechanism [I-D.ietf-abfab-gss-eap]

  Section 7.2 [Page 21]
    "Type" in packet format and "The code" in last paragraph seems to 
    be the same thing, but the name is differ.  Should use the same word.

  Section 12 [Page 27]
    Sam hartman's => Sam Hartman's

-- 
Yoshiro YONEYA <yoshiro.yoneya@jprs.co.jp>

v2.23.0 | Report a Bug | By Email