Re: Gen-ART LC review of draft-thaler-v6ops-teredo-extensions-07

Jari Arkko <jari.arkko@piuha.net> Thu, 26 August 2010 04:47 UTC

Return-Path: <jari.arkko@piuha.net>
X-Original-To: ietf@core3.amsl.com
Delivered-To: ietf@core3.amsl.com
Received: from localhost (localhost [127.0.0.1]) by core3.amsl.com (Postfix) with ESMTP id 3C6E43A6A74; Wed, 25 Aug 2010 21:47:58 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -102.358
X-Spam-Level:
X-Spam-Status: No, score=-102.358 tagged_above=-999 required=5 tests=[AWL=0.241, BAYES_00=-2.599, USER_IN_WHITELIST=-100]
Received: from mail.ietf.org ([64.170.98.32]) by localhost (core3.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id zB5jAA8V9CuC; Wed, 25 Aug 2010 21:47:56 -0700 (PDT)
Received: from p130.piuha.net (p130.piuha.net [IPv6:2001:14b8:400::130]) by core3.amsl.com (Postfix) with ESMTP id 7C5693A6A79; Wed, 25 Aug 2010 21:47:56 -0700 (PDT)
Received: from localhost (localhost [127.0.0.1]) by p130.piuha.net (Postfix) with ESMTP id 15E4B2CC34; Thu, 26 Aug 2010 07:48:28 +0300 (EEST)
X-Virus-Scanned: amavisd-new at piuha.net
Received: from p130.piuha.net ([127.0.0.1]) by localhost (p130.piuha.net [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id Tc7gTID4Ort9; Thu, 26 Aug 2010 07:48:27 +0300 (EEST)
Received: from [IPv6:::1] (unknown [IPv6:2001:14b8:400::130]) by p130.piuha.net (Postfix) with ESMTP id 0470B2CC32; Thu, 26 Aug 2010 07:48:25 +0300 (EEST)
Message-ID: <4C75F21B.8070601@piuha.net>
Date: Thu, 26 Aug 2010 00:48:27 -0400
From: Jari Arkko <jari.arkko@piuha.net>
User-Agent: Thunderbird 2.0.0.24 (X11/20100411)
MIME-Version: 1.0
To: Roni Even <ron.even.tlv@gmail.com>
Subject: Re: Gen-ART LC review of draft-thaler-v6ops-teredo-extensions-07
References: <4c591abb.cf7d0e0a.3128.ffff8faf@mx.google.com>
In-Reply-To: <4c591abb.cf7d0e0a.3128.ffff8faf@mx.google.com>
Content-Type: text/plain; charset="windows-1252"; format="flowed"
Content-Transfer-Encoding: 8bit
Cc: 'General Area Review Team' <gen-art@ietf.org>, 'IETF-Discussion list' <ietf@ietf.org>, draft-thaler-v6ops-teredo-extensions.all@tools.ietf.org
X-BeenThere: ietf@ietf.org
X-Mailman-Version: 2.1.9
Precedence: list
List-Id: IETF-Discussion <ietf.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/listinfo/ietf>, <mailto:ietf-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/ietf>
List-Post: <mailto:ietf@ietf.org>
List-Help: <mailto:ietf-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/ietf>, <mailto:ietf-request@ietf.org?subject=subscribe>
X-List-Received-Date: Thu, 26 Aug 2010 04:47:58 -0000

Thanks for your review, Roni.

> 1. In section 4.1 defines type-length-value (TLV) encoded trailers. 
> Can new trailers be defined later and if yes what is the procedure to 
> do it and assign type numbers. If new trailers are not allowed it will 
> be good to state it also
>

There has been earlier discussion on this. I wrote in my AD review:

> 8.  IANA Considerations
>  
>     [RFC Editor: please remove this section prior to publication.]
>  
>     This document has no IANA Actions.
>  
>  I do not understand how this can be. The document introduces new formats and
>  new fields, e.g., DiscoveryType field in Section 4.4. How are new values
>  allocated?

Dave responded:


> No change.  The new fields are not delegated to IANA to maintain, they're
> simply defined in this document and hence can only be updated by future
> RFC action that Updates this document.  In this sense, they're no different 
> from how, say, the flags bits in IPv6 RA's are allocated.

and I replied:

> I'm fine with no IANA actions, but then you should at least state in 
> the draft what you stated in the e-mail. In any case, that is a small 
> thing that we can deal with later. 

In other words, I believe the document still needs a change to describe what the procedure is to extend the TLVs. Dave?


> 2. In section 5.6.4.1 it says that “the Alternate Address Trailer MUST 
> include the node's local address/port in the Alternate Address/Port 
> list. If the UPnP Mapped Address/Port is non-zero, the Alternate 
> Address Trailer MUST also include it in the list.” I understood from 
> 3.5 and 6.5 that UPnP support on the NAT is required in which case the 
> alternate address trailer must have a UPnP address or is the 
> requirement for UPnP is needed only when the two peers are connected 
> via more than one NAT in the private address space as section 3.5 and 
> 6.5 describe. Please clarify.
>

Good question. I can't answer it, Dave?

Jari