Re: [Gen-art] Gen-ART review of draft-ietf-pcp-upnp-igd-interworking-07

<mohamed.boucadair@orange.com> Thu, 11 April 2013 05:43 UTC

Return-Path: <mohamed.boucadair@orange.com>
X-Original-To: gen-art@ietfa.amsl.com
Delivered-To: gen-art@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 9BE9D21F8D8D; Wed, 10 Apr 2013 22:43:58 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.791
X-Spam-Level:
X-Spam-Status: No, score=-1.791 tagged_above=-999 required=5 tests=[AWL=-0.342, BAYES_00=-2.599, HELO_EQ_FR=0.35, SARE_SUB_RAND_LETTRS4=0.799, UNPARSEABLE_RELAY=0.001]
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 YojbI92zc7zW; Wed, 10 Apr 2013 22:43:57 -0700 (PDT)
Received: from relais-inet.francetelecom.com (relais-ias92.francetelecom.com [193.251.215.92]) by ietfa.amsl.com (Postfix) with ESMTP id 9DE5621F8D79; Wed, 10 Apr 2013 22:43:57 -0700 (PDT)
Received: from omfedm07.si.francetelecom.fr (unknown [xx.xx.xx.3]) by omfedm09.si.francetelecom.fr (ESMTP service) with ESMTP id 3185A2DC72E; Thu, 11 Apr 2013 07:43:56 +0200 (CEST)
Received: from PUEXCH71.nanterre.francetelecom.fr (unknown [10.101.44.33]) by omfedm07.si.francetelecom.fr (ESMTP service) with ESMTP id 11A2E4C070; Thu, 11 Apr 2013 07:43:56 +0200 (CEST)
Received: from PUEXCB1B.nanterre.francetelecom.fr ([10.101.44.7]) by PUEXCH71.nanterre.francetelecom.fr ([10.101.44.33]) with mapi; Thu, 11 Apr 2013 07:43:55 +0200
From: mohamed.boucadair@orange.com
To: Peter Yee <peter@akayla.com>, "draft-ietf-pcp-upnp-igd-interworking.all@tools.ietf.org" <draft-ietf-pcp-upnp-igd-interworking.all@tools.ietf.org>
Date: Thu, 11 Apr 2013 07:43:53 +0200
Thread-Topic: Gen-ART review of draft-ietf-pcp-upnp-igd-interworking-07
Thread-Index: AQGru2VgIxcOQENUYzhlyQVOZ0pm1wHHP5CkAxM512IB11gsjJjfEHeAgADYYbA=
Message-ID: <94C682931C08B048B7A8645303FDC9F36EC1AE7D6E@PUEXCB1B.nanterre.francetelecom.fr>
References: <054201ce34fa$0746ec80$15d4c580$@akayla.com> <94C682931C08B048B7A8645303FDC9F36EC1AE76B6@PUEXCB1B.nanterre.francetelecom.fr> <059701ce3554$235aa0b0$6a0fe210$@akayla.com> <94C682931C08B048B7A8645303FDC9F36EC1AE796C@PUEXCB1B.nanterre.francetelecom.fr> <004c01ce360b$03fdade0$0bf909a0$@akayla.com>
In-Reply-To: <004c01ce360b$03fdade0$0bf909a0$@akayla.com>
Accept-Language: fr-FR
Content-Language: fr-FR
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
acceptlanguage: fr-FR
Content-Type: text/plain; charset="iso-8859-1"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-PMX-Version: 5.6.1.2065439, Antispam-Engine: 2.7.2.376379, Antispam-Data: 2013.4.11.43316
Cc: "gen-art@ietf.org" <gen-art@ietf.org>, "ietf@ietf.org" <ietf@ietf.org>
Subject: Re: [Gen-art] Gen-ART review of draft-ietf-pcp-upnp-igd-interworking-07
X-BeenThere: gen-art@ietf.org
X-Mailman-Version: 2.1.12
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: Thu, 11 Apr 2013 05:43:58 -0000

Dear Peter,

The new version is now available online. A diff is available here: http://www.ietf.org/rfcdiff?url2=draft-ietf-pcp-upnp-igd-interworking-08.

Thank you again for the review.

Cheers,
Med

>-----Message d'origine-----
>De : Peter Yee [mailto:peter@akayla.com]
>Envoyé : mercredi 10 avril 2013 18:47
>À : BOUCADAIR Mohamed OLNC/OLN; draft-ietf-pcp-upnp-igd-
>interworking.all@tools.ietf.org
>Cc : gen-art@ietf.org; ietf@ietf.org
>Objet : RE: Gen-ART review of draft-ietf-pcp-upnp-igd-interworking-07
>
>Med,
>
>	That looks great.  Thanks for accommodating my concern.
>
>			Kind regards,
>			-Peter
>
>-----Original Message-----
>From: mohamed.boucadair@orange.com [mailto:mohamed.boucadair@orange.com]
>Sent: Wednesday, April 10, 2013 12:49 AM
>To: Peter Yee; draft-ietf-pcp-upnp-igd-interworking.all@tools.ietf.org
>Cc: gen-art@ietf.org; ietf@ietf.org
>Subject: RE: Gen-ART review of draft-ietf-pcp-upnp-igd-interworking-07
>
>Dear Peter,
>
>I changed the text as follows:
>
>OLD:
>
>   If the requested external port is not available, the PCP server will
>   send a CANNOT_PROVIDE_EXTERNAL error response.  If a short lifetime
>   error is returned, the IGD-PCP IWF MAY re-send the same request to
>   the PCP Server after 30 seconds.  If a PCP error response is
>   received, the IGD-PCP IWF relays a negative message to the UPnP
>   Control Point with ConflictInMappingEntry as the error code.
>
>NEW:
>
>   If the requested external port is not available, the PCP server will
>   send a CANNOT_PROVIDE_EXTERNAL error response:
>
>   1.  If a short lifetime error is returned, the IGD-PCP IWF MAY resend
>       the same request to the PCP Server after 30 seconds without
>       relaying the error to the UPnP Control Point.  The IGD-PCP IWF
>       MAY repeat this process until a positive answer is received or
>       some maximum retry limit is reached.  When the maximum retry
>       limit is reached, the IGD-PCP IWF relays a negative message to
>       the UPnP Control Point with ConflictInMappingEntry as the error
>       code.
>
>   2.  If a long lifetime error is returned, the IGD-PCP IWF relays a
>       negative message to the UPnP Control Point with
>       ConflictInMappingEntry as the error code.
>
>Better?
>
>Cheers,
>Med
>
>>-----Message d'origine-----
>>De : Peter Yee [mailto:peter@akayla.com] Envoyé : mardi 9 avril 2013
>>20:58 À : BOUCADAIR Mohamed OLNC/OLN; draft-ietf-pcp-upnp-igd-
>>interworking.all@tools.ietf.org Cc : gen-art@ietf.org; ietf@ietf.org
>>Objet : RE: Gen-ART review of draft-ietf-pcp-upnp-igd-interworking-07
>>
>>Med,
>>	Thanks for the swift response to my review.  See my one reply
>inline.
>>
>>		Kind regards,
>>		-Peter
>>
>>>>Page 13, 1st paragraph, 3rd sentence: what's meant here is if any PCP
>>>>error other than a short-lifetime error, or in the case of a failed
>>>>resend, any PCP error at all.  The wording makes it seem like the
>>>>short-lifetime errors are somehow not PCP errors and is therefore
>>>>confusing.  It also doesn't explicitly deal with how many repeats
>>>>should
>>be done on a resend.
>>
>>>[Med] The basic behavior is to relay the received error to the UPnP CP.
>>For
>>the short-lifetime errors, the IWF may decide to resend the request and
>>not relay those errors immediately to the UPnP CP. The number of
>>repeats is not specified here as it can be implementation-specific.
>>
>>Your explanation is fine.  I just found the wording "If a PCP  error
>>response is received" to sound ambiguously as if the short-lifetime
>>errors were not a subset of PCP errors.
>>
>